When we are in this file, maybe tuning of " it needs heavy includes/excludes
tuning."   to contain examples of


69
(http://icedtea.classpath.org/hg/icedtea-web/file/22761e42d5b3/Makefile.am#l69)
export JACOCO_BASE_EXCLUDE=org.junit.*:junit.* 

70
(http://icedtea.classpath.org/hg/icedtea-web/file/22761e42d5b3/Makefile.am#l70)
export JACOCO_ADVANCED_EXCLUDE=:*jacoco*:java.lang.*:java.reflect.*:java.
util.*:sun.reflect.*

from (http://icedtea.classpath.org/hg/icedtea-web/file/22761e42d5b3/
Makefile.am#l69)

may be worthy.

Eg.: "it needs heavy includes/excludes. Most commonly excluded classes are :
*jacoco*:java.lang.*:java.reflect.*:java.util.*:sun.reflect.*. Also the 
tesengine itself mostly needs excluding. Eg for junit it is list of org.
junit.*:junit.*"

Just unblocking nit pick.

thank you again for push request!

J






---------- Původní zpráva ----------
Od: Evgeny Mandrikov <notifications@github.com>
Komu: jacoco/jacoco <jacoco@noreply.github.com>
Datum: 20. 5. 2014 11:23:08
Předmět: Re: [jacoco] Include possibility of covering of boot-classpath (#
49)

"

In jacoco-maven-plugin/src/org/jacoco/maven/AbstractAgentMojo.java:

> @@ -73,6 +73,14 @@
>  	 */
>  	String exclClassLoaders;
>  	/**
> +	 * Specifies whether also classes from the bootstrap classloader should be
> +	 * instrumented. Use this feature with caution, it needs heavy
> +	 * includes/excludes tuning.
> +	 * 
> +	 * @parameter property="jacoco.includebootstrapclasses"


@marchof(https://github.com/marchof) I believe that property name should be 
"jacoco.includeBootstrapClasses" in order to be consistent with naming of 
other properties

—
Reply to this email directly or view it on GitHub
(https://github.com/jacoco/jacoco/pull/49/files#r12834237). 

"