[EMAIL PROTECTED] wrote:
Revision
    6408 <http://fisheye.codehaus.org/changelog/mojo/?cs=6408>
Author
    bentmann
Date
    2008-03-09 05:13:31 -0500 (Sun, 09 Mar 2008)


      Log Message

o Fixed bad usage of collections framework


      Modified Paths

    * 
trunk/sandbox/l10n-maven-plugin/src/main/java/org/codehaus/mojo/l10n/L10NStatusReport.java
      
<#trunksandboxl10nmavenpluginsrcmainjavaorgcodehausmojol10nL10NStatusReportjava>


      Diff


        Modified:
        
trunk/sandbox/l10n-maven-plugin/src/main/java/org/codehaus/mojo/l10n/L10NStatusReport.java
        (6407 => 6408)

--- 
trunk/sandbox/l10n-maven-plugin/src/main/java/org/codehaus/mojo/l10n/L10NStatusReport.java
  2008-03-09 09:50:55 UTC (rev 6407)
+++ 
trunk/sandbox/l10n-maven-plugin/src/main/java/org/codehaus/mojo/l10n/L10NStatusReport.java
  2008-03-09 10:13:31 UTC (rev 6408)
@@ -94,7 +94,7 @@
     /**
      * The list of resources that are scanned for properties bundles.
      *
-     * @parameter expression="${project.resources}"
+     * @parameter expression="${project.build.resources}"
      * @required
      */
     private List resources;
@@ -102,16 +102,16 @@
     /**
      * A list of exclude patterns to use. By default no files are excluded.
      *
-     * @parameter expression="${maven.l10n.excludes}"
+     * @parameter
      */

If you remove the expression above and below, doesn't that mean that you can't provide that parameter on the command line anymore? Or is that not possible anyway, because they are collections?


-    private ArrayList excludes;
+    private List excludes;
/**
      * A list of include patterns to use. By default all .java files are 
included.
      *
-     * @parameter expression="${maven.l10n.includes}"
+     * @parameter
      */
-    private ArrayList includes;
+    private List includes;
/**
      * The projects in the reactor for aggregation report.
@@ -432,11 +432,11 @@
                                         in2 = new BufferedInputStream( new 
FileInputStream( locFile ) );
                                         props2.load( in2 );
                                         wr.getProperties().put( loc, props2 );
-                                        HashSet missing = new HashSet( 
props.keySet() );
+                                        Set missing = new HashSet( 
props.keySet() );
                                         missing.removeAll( props2.keySet() );
-                                        HashSet additional = new HashSet( 
props2.keySet() );
+                                        Set additional = new HashSet( 
props2.keySet() );
                                         additional.removeAll( props.keySet() );
-                                        HashSet nonTranslated = new HashSet();
+                                        Set nonTranslated = new HashSet();
                                         Iterator itx = 
props.keySet().iterator();
                                         while ( itx.hasNext() )
                                         {
@@ -572,7 +572,7 @@
                         {
                             locals = new Properties();
                         }
-                        TreeSet missing = new TreeSet( defs.keySet() );
+                        Set missing = new TreeSet( defs.keySet() );
                         missing.removeAll( locals.keySet() );
                         String cell = "";
                         Iterator ms = missing.iterator();
@@ -581,7 +581,7 @@
                             cell = cell + "<tr><td>" + ms.next() + 
"</td></tr>";
                         }
                         tableCell( wrapInTable( okLabel, cell ), true );
-                        TreeSet additional = new TreeSet( locals.keySet() );
+                        Set additional = new TreeSet( locals.keySet() );
                         additional.removeAll( defs.keySet() );
                         Iterator ex = additional.iterator();
                         cell = "";
@@ -590,7 +590,7 @@
                             cell = cell + "<tr><td>" + ex.next() + 
"</td></tr>";
                         }
                         tableCell( wrapInTable( okLabel, cell ), true );
-                        TreeSet nonTranslated = new TreeSet();
+                        Set nonTranslated = new TreeSet();
                         Iterator itnt = defs.keySet().iterator();
                         while ( itnt.hasNext() )
                         {
@@ -643,7 +643,7 @@
private MavenProject proj; - private HashMap properties;
+        private Map properties;
static final String DEFAULT_LOCALE = "Default"; @@ -671,7 +671,7 @@
             return proj;
         }
- public HashMap getProperties()
+        public Map getProperties()
         {
             return properties;
         }

--------------------------------------------------------------------------------

To unsubscribe from this list please visit:

http://xircles.codehaus.org/manage_email



--
Dennis Lundberg

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to