As we have this snippet now all over the place:

coverage.getInstructionCounter().getTotalCount() == 0

I would like to propose to add a new method to ICoverageNode:

/**
 * Checks whether this is an empty node.
 * 
 * @return <code>true</code> is this node does not contain any instructions
 */
boolean isEmpty();

@marchof added

@Godin Thx for adding isEmpty(). I found three more candidates for using it:

• org.jacoco.core.internal.analysis.ClassAnalyzer.addMethodCoverage(String, String, String, InstructionsBuilder, MethodNode)
• org.jacoco.ant.ReportTask.checkForMissingDebugInformation(ICoverageNode)
• org.jacoco.maven.ReportSupport.logBundleInfo(IBundleCoverage, Collection<IClassCoverage>)

@marchof thanks for spotting this - made changes, hopefully correctly 😉