diff options
author | Evgeny Mandrikov <Godin@users.noreply.github.com> | 2017-04-03 19:35:02 +0200 |
---|---|---|
committer | Marc R. Hoffmann <hoffmann@mountainminds.com> | 2017-04-03 19:35:02 +0200 |
commit | e94c7af58000b3421bc63a252ee054c08a863568 (patch) | |
tree | 385b540d48297b3b66947e349f61fb2f4cd72030 /org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java | |
parent | 7f719c8365b3ac8853cfc8b62fd48d2eabddfc4c (diff) | |
download | jacoco-e94c7af58000b3421bc63a252ee054c08a863568.tar.gz |
Use new filtering API for existing filter of methods in enums (#512)
Diffstat (limited to 'org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java')
-rw-r--r-- | org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java | 29 |
1 files changed, 2 insertions, 27 deletions
diff --git a/org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java b/org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java index 38037be5..fa3517b6 100644 --- a/org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java +++ b/org.jacoco.core/src/org/jacoco/core/internal/analysis/ClassAnalyzer.java @@ -63,12 +63,8 @@ public class ClassAnalyzer extends ClassProbesVisitor { InstrSupport.assertNotInstrumented(name, coverage.getName()); - if (isMethodFiltered(coverage.getName(), coverage.getSuperName(), name, - desc)) { - return null; - } - - return new MethodAnalyzer(stringPool.get(name), stringPool.get(desc), + return new MethodAnalyzer(coverage.getName(), coverage.getSuperName(), + stringPool.get(name), stringPool.get(desc), stringPool.get(signature), probes) { @Override public void visitEnd() { @@ -82,27 +78,6 @@ public class ClassAnalyzer extends ClassProbesVisitor { }; } - /** - * @return <code>true</code> if method should not be analyzed - */ - // TODO: Use filter hook in future - private boolean isMethodFiltered(final String className, - final String superClassName, final String name, final String desc) { - if ("java/lang/Enum".equals(superClassName)) { - // filter out methods that compiler creates for enums - if ("values".equals(name) - && ("()[L" + className + ";").equals(desc)) { - return true; - } - if ("valueOf".equals(name) - && ("(Ljava/lang/String;)L" + className + ";") - .equals(desc)) { - return true; - } - } - return false; - } - @Override public FieldVisitor visitField(final int access, final String name, final String desc, final String signature, final Object value) { |