Skip to content

Commit c2a3ba0

Browse files
authored
Remove unnecessary code (#9307)
1 parent 27a4595 commit c2a3ba0

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

javaagent-extension-api/src/main/java/io/opentelemetry/javaagent/extension/matcher/ClassLoaderHasClassesNamedMatcher.java

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,6 @@
99
import io.opentelemetry.javaagent.bootstrap.internal.ClassLoaderMatcherCacheHolder;
1010
import io.opentelemetry.javaagent.bootstrap.internal.InClassLoaderMatcher;
1111
import java.util.BitSet;
12-
import java.util.List;
13-
import java.util.concurrent.CopyOnWriteArrayList;
1412
import java.util.concurrent.atomic.AtomicInteger;
1513
import java.util.concurrent.locks.Lock;
1614
import java.util.concurrent.locks.ReentrantReadWriteLock;
@@ -31,9 +29,6 @@ class ClassLoaderHasClassesNamedMatcher extends ElementMatcher.Junction.Abstract
3129
for (int i = 0; i < resources.length; i++) {
3230
resources[i] = resources[i].replace(".", "/") + ".class";
3331
}
34-
if (useCache) {
35-
Manager.INSTANCE.add(this);
36-
}
3732
}
3833

3934
@Override
@@ -65,28 +60,18 @@ private static boolean hasResources(ClassLoader cl, String... resources) {
6560

6661
private static class Manager {
6762
static final Manager INSTANCE = new Manager();
68-
private final List<ClassLoaderHasClassesNamedMatcher> matchers = new CopyOnWriteArrayList<>();
6963
// each matcher gets a two bits in BitSet, that first bit indicates whether current matcher has
7064
// been run for given class loader and the second whether it matched or not
7165
private final Cache<ClassLoader, BitSet> enabled = Cache.weak();
7266
private final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
7367
private final Lock readLock = lock.readLock();
7468
private final Lock writeLock = lock.writeLock();
75-
private volatile boolean matchCalled = false;
7669

7770
Manager() {
7871
ClassLoaderMatcherCacheHolder.addCache(enabled);
7972
}
8073

81-
void add(ClassLoaderHasClassesNamedMatcher matcher) {
82-
if (matchCalled) {
83-
throw new IllegalStateException("All matchers should be create before match is called");
84-
}
85-
matchers.add(matcher);
86-
}
87-
8874
boolean match(ClassLoaderHasClassesNamedMatcher matcher, ClassLoader cl) {
89-
matchCalled = true;
9075
BitSet set = enabled.computeIfAbsent(cl, (unused) -> new BitSet(counter.get() * 2));
9176
int matcherRunBit = 2 * matcher.index;
9277
int matchedBit = matcherRunBit + 1;

0 commit comments

Comments
 (0)