From 376d38672b91b389358ec80c83c8527999158fdf Mon Sep 17 00:00:00 2001 From: Henry Coles Date: Fri, 30 Aug 2024 12:31:35 +0100 Subject: [PATCH] fix comment and remove unneccesary logging of common scenario --- .../org/pitest/mutationtest/tooling/MutationCoverage.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/pitest-entry/src/main/java/org/pitest/mutationtest/tooling/MutationCoverage.java b/pitest-entry/src/main/java/org/pitest/mutationtest/tooling/MutationCoverage.java index 648dfac90..c776404a5 100644 --- a/pitest-entry/src/main/java/org/pitest/mutationtest/tooling/MutationCoverage.java +++ b/pitest-entry/src/main/java/org/pitest/mutationtest/tooling/MutationCoverage.java @@ -65,7 +65,6 @@ import java.util.Optional; import java.util.Set; import java.util.function.Predicate; -import java.util.logging.Level; import java.util.logging.Logger; import java.util.stream.Collectors; @@ -377,9 +376,8 @@ private CoverageGenerator coverage() { return this.strategies.coverage(); } - // For reasons not yet understood classes from rt.jar are not resolved for some - // projects during static analysis phase. For now fall back to the classloader when - // a class not provided by project classpath + // Since java 9 rt.jar is no longer on the classpath so jdk classes will not resolve from + // the filesystem and must be pulled out via the classloader private ClassByteArraySource fallbackToClassLoader(final ClassByteArraySource bas) { final ClassByteArraySource clSource = ClassloaderByteArraySource.fromContext(); return clazz -> { @@ -387,7 +385,6 @@ private ClassByteArraySource fallbackToClassLoader(final ClassByteArraySource ba if (maybeBytes.isPresent()) { return maybeBytes; } - LOG.log(Level.FINE, "Could not find " + clazz + " on classpath for analysis. Falling back to classloader"); return clSource.getBytes(clazz); }; }