Skip to content

Commit dd912e0

Browse files
committed
use array instead of list
1 parent 0317905 commit dd912e0

1 file changed

Lines changed: 14 additions & 13 deletions

File tree

dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/agent/DebuggerTransformer.java

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -96,17 +96,18 @@ public class DebuggerTransformer implements ClassFileTransformer {
9696
private static final String JAVA_IO_TMPDIR = "java.io.tmpdir";
9797
private static final boolean JAVA_AT_LEAST_19 = JavaVirtualMachine.isJavaVersionAtLeast(19);
9898
public static Path DUMP_PATH = Paths.get(SystemProperties.get(JAVA_IO_TMPDIR), "debugger");
99-
private static final List<String> SKIPPED_PACKAGES =
100-
Arrays.asList(
101-
"com/datadog/debugger/agent/",
102-
"com/datadog/debugger/codeorigin/",
103-
"com/datadog/debugger/exception/",
104-
"com/datadog/debugger/instrumentation/",
105-
"com/datadog/debugger/probe/",
106-
"com/datadog/debugger/sink/",
107-
"com/datadog/debugger/symbol/",
108-
"com/datadog/debugger/uploader/",
109-
"com/datadog/debugger/util/");
99+
private static final String[] SKIPPED_PACKAGES =
100+
new String[] {
101+
"com/datadog/debugger/agent/",
102+
"com/datadog/debugger/codeorigin/",
103+
"com/datadog/debugger/exception/",
104+
"com/datadog/debugger/instrumentation/",
105+
"com/datadog/debugger/probe/",
106+
"com/datadog/debugger/sink/",
107+
"com/datadog/debugger/symbol/",
108+
"com/datadog/debugger/uploader/",
109+
"com/datadog/debugger/util/"
110+
};
110111

111112
private final Config config;
112113
private final TransformerDefinitionMatcher definitionMatcher;
@@ -345,8 +346,8 @@ private boolean skipInstrumentation(ClassLoader loader, String classFilePath) {
345346
// skip classes/packages that are part of debugger agent to avoid
346347
// LinkageError: attempted duplicate class definition
347348
// while retransforming a class used by instrumentation
348-
for (int i = 0; i < SKIPPED_PACKAGES.size(); i++) {
349-
if (classFilePath.startsWith(SKIPPED_PACKAGES.get(i))) {
349+
for (int i = 0; i < SKIPPED_PACKAGES.length; i++) {
350+
if (classFilePath.startsWith(SKIPPED_PACKAGES[i])) {
350351
return true;
351352
}
352353
}

0 commit comments

Comments
 (0)