You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by ol...@apache.org on 2009/11/19 23:01:06 UTC

svn commit: r882317 - /maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java

Author: olamy
Date: Thu Nov 19 22:01:06 2009
New Revision: 882317

URL: http://svn.apache.org/viewvc?rev=882317&view=rev
Log:
add a comment on class loading hack for maven 2.x

Modified:
    maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java

Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java?rev=882317&r1=882316&r2=882317&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java Thu Nov 19 22:01:06 2009
@@ -74,6 +74,7 @@
         // checkstyle will always use the context classloader in order
         // to load resources (dtds),
         // so we have to fix it
+        // olamy this hack is not anymore needed in maven 3.x
         ClassLoader checkstyleClassLoader = PackageNamesLoader.class.getClassLoader();
         Thread.currentThread().setContextClassLoader( checkstyleClassLoader );