Evgeny Mandrikov

Results 252 comments of Evgeny Mandrikov

> Seems now it has build error which is irrelevant to the PR? yes it was unrelated - see #1249

Only classes that were executed are instrumented by Agent. Information about these classes is saved into exec file, list of classes in exec file already available in report on a...

@ggeorgovassilis if you don't have exclusion and inclusion patterns, then you should see all the classes, otherwise even verbose output can't show anything simply because it is not aware of...

> I'd like to figure out why they don't show up in the report. The report goal produced no diagnostic output. This ticket is about printing diagnostics If in absence...

> I assume (?) it then iterates over the entries in the exec file but decides to drop each and every entry. No - it iterates over classes on disk...

@antiloop222 this is described in [documentation of JaCoCo Maven plugin](http://www.jacoco.org/jacoco/trunk/doc/maven.html) on a [page about goal "report-aggregate"](http://www.jacoco.org/jacoco/trunk/doc/report-aggregate-mojo.html), which is also available via `mvn org.jacoco:jacoco-maven-plugin:0.7.8:help` :wink: Contributions are welcomed if you think...

@nathanila first of all thank you for detailed explanations and ideas. ---- > For my use case, removing instrumentation from many base classes will reduce the instrumentation overhead. I would...

Additional thought about `MaxInlineLevel`: we anyway add `$jacocoInit`. --- And here is attempt to write benchmark: ```java import org.openjdk.jmh.annotations.Benchmark; import org.openjdk.jmh.annotations.Scope; import org.openjdk.jmh.annotations.State; @State(Scope.Thread) public class Bench { int x;...

@nathanila > From the results, it appears that you ran this on a machine with 4 physical cores. Yes - this result is from my workstation at work, where use...

@marchof case of interfaces and Java < 8 - is static initializers, so IMO addition of branch in this case not needed.