[jboss-cvs] JBossAS SVN: r67431 - in projects/aop/branches/joinpoint_graph/aop/src: test/org/jboss/test/aop/memoryleaks and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: flavia.rainone at jboss.com
Date: 2007-11-25 17:39:25 -0500 (Sun, 25 Nov 2007)
New Revision: 67431

Modified:
   projects/aop/branches/joinpoint_graph/aop/src/main/org/jboss/aop/Deployment.java
   projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTestCase.java
   projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTester.java
Log:
[JBAOP-489] Bug fixed.

Modified: projects/aop/branches/joinpoint_graph/aop/src/main/org/jboss/aop/Deployment.java
===================================================================
--- projects/aop/branches/joinpoint_graph/aop/src/main/org/jboss/aop/Deployment.java	2007-11-24 17:22:00 UTC (rev 67430)
+++ projects/aop/branches/joinpoint_graph/aop/src/main/org/jboss/aop/Deployment.java	2007-11-25 22:39:25 UTC (rev 67431)
@@ -112,8 +112,7 @@
          }
          try
          {
-            // use toURI().toURL() because of bug http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4273532
-            URL url = f.toURI().toURL();
+            URL url = f.toURL();
             Iterator it = ArchiveBrowser.getBrowser(url, new ClassFileFilter());
             AspectAnnotationLoader loader = new AspectAnnotationLoader(AspectManager.instance());
             loader.deployInputStreamIterator(it);

Modified: projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTestCase.java
===================================================================
--- projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTestCase.java	2007-11-24 17:22:00 UTC (rev 67430)
+++ projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTestCase.java	2007-11-25 22:39:25 UTC (rev 67431)
@@ -536,7 +536,8 @@
        while (tokenString.hasMoreElements())
        {
           String value = tokenString.nextToken();
-          URL itemLocation = new File(value).toURL();
+          // use toURI().toURL() because of bug http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4273532
+          URL itemLocation = new File(value).toURI().toURL();
           if (!itemLocation.equals(classLocation) && !itemLocation.toString().equals(pathIgnore))
           {
              urls.add(itemLocation);

Modified: projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTester.java
===================================================================
--- projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTester.java	2007-11-24 17:22:00 UTC (rev 67430)
+++ projects/aop/branches/joinpoint_graph/aop/src/test/org/jboss/test/aop/memoryleaks/MemoryLeakTester.java	2007-11-25 22:39:25 UTC (rev 67431)
@@ -91,7 +91,8 @@
        while (tokenString.hasMoreElements())
        {
           String value = tokenString.nextToken();
-          URL itemLocation = new File(value).toURL();
+          // use toURI().toURL() because of bug http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4273532
+          URL itemLocation = new File(value).toURI().toURL();
           if (!itemLocation.equals(classLocation) && !itemLocation.toString().equals(pathIgnore))
           {
              //System.out.println("Location:" + itemLocation);