Skip to content

Commit f85878f

Browse files
committed
Use byte buddy saveIn API instead of my own implementation.
1 parent b53c178 commit f85878f

File tree

1 file changed

+1
-22
lines changed

1 file changed

+1
-22
lines changed

apm-sniffer/apm-agent/src/main/java/org/skywalking/apm/agent/InstrumentDebuggingClass.java

Lines changed: 1 addition & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
package org.skywalking.apm.agent;
2020

2121
import java.io.File;
22-
import java.io.FileOutputStream;
2322
import java.io.IOException;
2423
import net.bytebuddy.description.type.TypeDescription;
2524
import net.bytebuddy.dynamic.DynamicType;
@@ -59,30 +58,10 @@ public void log(TypeDescription typeDescription, DynamicType dynamicType) {
5958
}
6059
}
6160

62-
File newClassFileFolder = new File(debuggingClassesRootPath, typeDescription.getPackage().getActualName().replaceAll("\\.", "/"));
63-
if (!newClassFileFolder.exists()) {
64-
newClassFileFolder.mkdirs();
65-
}
66-
File newClassFile = new File(newClassFileFolder, typeDescription.getSimpleName() + ".class");
67-
FileOutputStream fos = null;
6861
try {
69-
if (newClassFile.exists()) {
70-
newClassFile.delete();
71-
}
72-
newClassFile.createNewFile();
73-
fos = new FileOutputStream(newClassFile);
74-
fos.write(dynamicType.getBytes());
75-
fos.flush();
62+
dynamicType.saveIn(debuggingClassesRootPath);
7663
} catch (IOException e) {
7764
logger.error(e, "Can't save class {} to file." + typeDescription.getActualName());
78-
} finally {
79-
if (fos != null) {
80-
try {
81-
fos.close();
82-
} catch (IOException e) {
83-
84-
}
85-
}
8665
}
8766
} catch (Throwable t) {
8867
logger.error(t, "Save debugging classes fail.");

0 commit comments

Comments
 (0)