You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by gs...@apache.org on 2010/06/09 22:38:00 UTC

svn commit: r953152 - /ant/ivy/core/trunk/src/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorUpdater.java

Author: gscokart
Date: Wed Jun  9 20:38:00 2010
New Revision: 953152

URL: http://svn.apache.org/viewvc?rev=953152&view=rev
Log:
Fix ConstantNameCheck checkstyle error

Modified:
    ant/ivy/core/trunk/src/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorUpdater.java

Modified: ant/ivy/core/trunk/src/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorUpdater.java
URL: http://svn.apache.org/viewvc/ant/ivy/core/trunk/src/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorUpdater.java?rev=953152&r1=953151&r2=953152&view=diff
==============================================================================
--- ant/ivy/core/trunk/src/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorUpdater.java (original)
+++ ant/ivy/core/trunk/src/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorUpdater.java Wed Jun  9 20:38:00 2010
@@ -162,21 +162,21 @@ public final class XmlModuleDescriptorUp
     private static class UpdaterHandler extends DefaultHandler implements LexicalHandler {
 
         /** standard attributes of ivy-module/info */
-        private static final Collection stdAtts = Arrays.asList(new String[] {"organisation", "module", "branch",
-                "revision", "status", "publication", "namespace"});
+        private static final Collection STD_ATTS = Arrays.asList(new String[] {"organisation",
+                "module", "branch", "revision", "status", "publication", "namespace"});
 
         /** elements that may appear inside ivy-module, in expected order */
-        private static final List moduleElements = Arrays.asList(new String[] {
+        private static final List MODULE_ELEMENTS = Arrays.asList(new String[] {
                 "info", "configurations", "publications", "dependencies", "conflicts"
         });
         /** element position of "configurations" inside "ivy-module" */
-        private static final int CONFIGURATIONS_POSITION = moduleElements.indexOf("configurations");
+        private static final int CONFIGURATIONS_POSITION = MODULE_ELEMENTS.indexOf("configurations");
         /** element position of "dependencies" inside "ivy-module" */
-        private static final int DEPENDENCIES_POSITION = moduleElements.indexOf("dependencies");
+        private static final int DEPENDENCIES_POSITION = MODULE_ELEMENTS.indexOf("dependencies");
 
         /** elements that may appear inside of ivy-module/info */
-        private static final Collection infoElements = Arrays.asList(new String[]{
-                "extends", "ivyauthor", "license", "repository", "description" });
+        private static final Collection INFO_ELEMENTS = Arrays.asList(new String[] {"extends",
+                "ivyauthor", "license", "repository", "description"});
 
         private final ParserSettings settings;
 
@@ -354,7 +354,7 @@ public final class XmlModuleDescriptorUp
                         //if the descriptor already contains a description, don't bother printing
                         //the merged version.
                         hasDescription = true;
-                    } else if (!infoElements.contains(qName)) {
+                    } else if (!INFO_ELEMENTS.contains(qName)) {
                         //according to the XSD, we should write description after all of the other
                         //standard <info> elements but before any extended elements.
                         writeInheritedDescription(merged);
@@ -699,7 +699,7 @@ public final class XmlModuleDescriptorUp
                 extraAttributes = new LinkedHashMap(attributes.getLength());
                 for (int i = 0; i < attributes.getLength(); i++) {
                     String qname = attributes.getQName(i);
-                    if (!stdAtts.contains(qname)) {
+                    if (!STD_ATTS.contains(qname)) {
                         extraAttributes.put(qname, substitute(settings, attributes.getValue(i)));
                     }
                 }
@@ -1047,8 +1047,8 @@ public final class XmlModuleDescriptorUp
                 && !(mergedConfigurations && mergedDependencies)) {
 
                 //calculate the position of the element in ivy-module
-                int position = moduleElement == null ? moduleElements.size()
-                        : moduleElements.indexOf(moduleElement);
+                int position = moduleElement == null ? MODULE_ELEMENTS.size()
+                        : MODULE_ELEMENTS.indexOf(moduleElement);
 
                 ModuleDescriptor merged = options.getMergedDescriptor();