svn commit: r1230296 - /maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdViolationCheckMojo.java

[dennisl]

Author: dennisl
Date: Wed Jan 11 22:35:26 2012
New Revision: 1230296

URL: http://svn.apache.org/viewvc?rev=1230296&view=rev
Log:
Fix error reported by Checkstyle.

Modified:
    
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdViolationCheckMojo.java

Modified: 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdViolationCheckMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdViolationCheckMojo.java?rev=1230296&r1=1230295&r2=1230296&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdViolationCheckMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/CpdViolationCheckMojo.java
 Wed Jan 11 22:35:26 2012
@@ -117,20 +117,18 @@ public class CpdViolationCheckMojo
                 msgs.put( "text", xpp.getText().trim() );
                 break;
             case XmlPullParser.START_TAG:
+                String nm = xpp.getName();
+                if ( msgs.containsKey( nm ) )
                 {
-                    String nm = xpp.getName();
-                    if ( msgs.containsKey( nm ) )
+                    int cnt = 1;
+                    while ( msgs.containsKey( nm + cnt ) )
                     {
-                        int cnt = 1;
-                        while ( msgs.containsKey( nm + cnt ) )
-                        {
-                            ++cnt;
-                        }
-                        nm = nm + cnt;
+                        ++cnt;
                     }
-                    msgs.put( nm, getErrorDetails( xpp ) );
-                    break;
+                    nm = nm + cnt;
                 }
+                msgs.put( nm, getErrorDetails( xpp ) );
+                break;
             default:
             }
             tp = xpp.next();