Fwd: svn commit: r1564575 - /incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java

[Romain Manni-Bucau <rm...@gmail.com>]

Would be great to have details since this part is here just to avoid the
jvm to break and excludes are handle behind other classes (mainly curiosity
;)

---------- Message transféré ----------
De : <ol...@apache.org>
Date : mercredi 5 février 2014
Objet : svn commit: r1564575 -
/incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java
À : commits@sirona.incubator.apache.org


Author: olamy
Date: Wed Feb  5 00:16:23 2014
New Revision: 1564575

URL: http://svn.apache.org/r1564575
Log:
change to non static and protected so users can override that

Modified:

incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java

Modified:
incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java
URL:
http://svn.apache.org/viewvc/incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java?rev=1564575&r1=1564574&r2=1564575&view=diff
==============================================================================
---
incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java
(original)
+++
incubator/sirona/trunk/agent/javaagent/src/main/java/org/apache/sirona/javaagent/SironaTransformer.java
Wed Feb  5 00:16:23 2014
@@ -35,7 +35,7 @@ public class SironaTransformer implement
         return classfileBuffer;
     }

-    private byte[] doTransform(final String className, final byte[]
classfileBuffer) {
+    protected byte[] doTransform(final String className, final byte[]
classfileBuffer) {
         try {
             final SironaClassVisitor.SironaKeyVisitor keyVisitor = new
SironaClassVisitor.SironaKeyVisitor(className);
             new ClassReader(classfileBuffer).accept(keyVisitor,
ClassReader.SKIP_DEBUG);
@@ -55,7 +55,7 @@ public class SironaTransformer implement
         }
     }

-    private static boolean shouldTransform(final String className, final
ClassLoader loader) {
+    protected boolean shouldTransform(final String className, final
ClassLoader loader) {
         return !(className == null // framework with bug
                 || (loader != null &&
loader.getClass().getName().equals(DELEGATING_CLASS_LOADER))
                 || className.startsWith("sun/reflect")





-- 
*Romain Manni-Bucau*
*Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
*Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
*LinkedIn: **http://fr.linkedin.com/in/rmannibucau
<http://fr.linkedin.com/in/rmannibucau>*
*Github: https://github.com/rmannibucau <https://github.com/rmannibucau>*