diff options
author | Grzegorz Kossakowski <grzegorz.kossakowski@gmail.com> | 2012-07-26 11:40:13 +0200 |
---|---|---|
committer | Grzegorz Kossakowski <grzegorz.kossakowski@gmail.com> | 2012-07-26 11:54:47 +0200 |
commit | 9e04e2365a7210f7e671137d8f70e475688b0885 (patch) | |
tree | 6ff6517a65bcd4f954c916129822e755d3a340b3 /src | |
parent | ad08f24448729009fc8d5ff0acf307a43b4cfe0a (diff) | |
download | scala-9e04e2365a7210f7e671137d8f70e475688b0885.tar.gz scala-9e04e2365a7210f7e671137d8f70e475688b0885.tar.bz2 scala-9e04e2365a7210f7e671137d8f70e475688b0885.zip |
Instrument all classes in `instrumented` package.
Extend instrumenting infrastructure to instrument
classes in `instrumented` package. This is useful
because very often you need to put your classes
into non-empty package. E.g. inliner doesn't work
properly with empty package at the moment so in
order to test any behaviour we need to put classes
in some other package that is instrumented.
Added testing code for that to instrumentation
test-case.
Review by @phaller.
Diffstat (limited to 'src')
-rw-r--r-- | src/partest/scala/tools/partest/javaagent/ASMTransformer.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/partest/scala/tools/partest/javaagent/ASMTransformer.java b/src/partest/scala/tools/partest/javaagent/ASMTransformer.java index 643c683002..09cd485d6b 100644 --- a/src/partest/scala/tools/partest/javaagent/ASMTransformer.java +++ b/src/partest/scala/tools/partest/javaagent/ASMTransformer.java @@ -21,7 +21,9 @@ public class ASMTransformer implements ClassFileTransformer { // we instrument all classes from empty package (!className.contains("/") || // we instrument all classes from scala package - className.startsWith("scala/")); + className.startsWith("scala/") || + // we instrument all classes from `instrumented` package + className.startsWith("instrumented/")); } public byte[] transform(ClassLoader loader, String className, Class<?> classBeingRedefined, ProtectionDomain protectionDomain, byte[] classfileBuffer) { |