svn commit: r734797 - /openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/AbstractCFMetaDataFactory.java

[mikedd]

Author: mikedd
Date: Thu Jan 15 12:21:45 2009
New Revision: 734797

URL: http://svn.apache.org/viewvc?rev=734797&view=rev
Log:
OPENJPA-860 improve trace. Entity class names are printed when found in orm.xml

Modified:
    
openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/AbstractCFMetaDataFactory.java

Modified: 
openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/AbstractCFMetaDataFactory.java
URL: 
http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/AbstractCFMetaDataFactory.java?rev=734797&r1=734796&r2=734797&view=diff
==============================================================================
--- 
openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/AbstractCFMetaDataFactory.java
 (original)
+++ 
openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/meta/AbstractCFMetaDataFactory.java
 Thu Jan 15 12:21:45 2009
@@ -627,7 +627,7 @@
         throws IOException {
         ClassArgParser cparser = newClassArgParser();
         String[] clss;
-        Set names = new HashSet();
+        Set<String> names = new HashSet<String>();
         if (files != null) {
             File file;
             for (Iterator itr = files.iterator(); itr.hasNext();) {
@@ -655,9 +655,10 @@
                         log.trace(_loc.get("scanning-file", file));
                     clss = cparser.parseTypeNames(new FileMetaDataIterator
                         (file));
+                    List<String> newNames = Arrays.asList(clss);
                     if (log.isTraceEnabled())
-                        log.trace(_loc.get("scan-found-names", clss, file));
-                    names.addAll(Arrays.asList(clss));
+                        log.trace(_loc.get("scan-found-names", newNames, 
file));
+                    names.addAll(newNames);
                     File f = AccessController
                         .doPrivileged(J2DoPrivHelper
                             .getAbsoluteFileAction(file));
@@ -714,9 +715,10 @@
                     if (log.isTraceEnabled())
                         log.trace(_loc.get("scanning-url", url));
                     clss = cparser.parseTypeNames(new 
URLMetaDataIterator(url));
+                    List<String> newNames = Arrays.asList(clss);
                     if (log.isTraceEnabled())
-                        log.trace(_loc.get("scan-found-names", clss, url));
-                    names.addAll(Arrays.asList(clss));
+                        log.trace(_loc.get("scan-found-names", newNames, url));
+                    names.addAll(newNames);
                     mapPersistentTypeNames(url, clss);
                 }
             }
@@ -752,9 +754,10 @@
                         url = (URL) mitr.next();
                         clss = cparser.parseTypeNames(new URLMetaDataIterator
                             (url));
+                        List<String> newNames = Arrays.asList(clss);
                         if (log.isTraceEnabled())
                             log.trace(_loc.get("scan-found-names", clss, 
rsrc));
-                        names.addAll(Arrays.asList(clss));
+                        names.addAll(newNames);
                         mapPersistentTypeNames(url, clss);
                     }
                     mitr.close();