Skip to content

Do not instrument classes loaded by platform class loader by default#2155

Merged
Godin merged 11 commits into
jacoco:masterfrom
Godin:platform_class_loader
Jun 12, 2026
Merged

Do not instrument classes loaded by platform class loader by default#2155
Godin merged 11 commits into
jacoco:masterfrom
Godin:platform_class_loader

Conversation

@Godin

@Godin Godin commented Jun 5, 2026

Copy link
Copy Markdown
Member

Closes #2152

  • add integration test
  • add unit tests
  • fix
  • update documentation
  • update changelog

@Godin Godin added this to the 0.8.16 milestone Jun 5, 2026
@Godin Godin self-assigned this Jun 5, 2026
@Godin
Godin force-pushed the platform_class_loader branch from 9b9b2ff to 268bbf1 Compare June 5, 2026 10:05
@Godin

Godin commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

@marchof could you please pre-review approach before I'll work on update of documentation?

Comment thread org.jacoco.ant.test/src/org/jacoco/ant/CoverageTaskTest.xml Outdated
Comment thread org.jacoco.agent.rt/src/org/jacoco/agent/rt/internal/CoverageTransformer.java Outdated
@Godin
Godin force-pushed the platform_class_loader branch from f992fc7 to 54d6875 Compare June 12, 2026 20:31
@Godin Godin changed the title Do not instrument classes loaded by platform class loader by default Require inclbootstrapclasses=true to instrument classes loaded by platform Jun 12, 2026
@Godin Godin changed the title Require inclbootstrapclasses=true to instrument classes loaded by platform Do not instrument classes loaded by platform classloader by default Jun 12, 2026
@Godin Godin changed the title Do not instrument classes loaded by platform classloader by default Do not instrument classes loaded by platform class loader by default Jun 12, 2026
@Godin
Godin force-pushed the platform_class_loader branch from 21d4a43 to 7ce157d Compare June 12, 2026 21:30
@Godin
Godin marked this pull request as ready for review June 12, 2026 21:53
@Godin
Godin enabled auto-merge (squash) June 12, 2026 21:54
@Godin
Godin merged commit 3457f84 into jacoco:master Jun 12, 2026
31 checks passed
@Godin
Godin deleted the platform_class_loader branch June 12, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Do not instrument classes loaded by platform class loader by default

2 participants