svn commit: r618830 - /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraXML.java

[dennisl]

Author: dennisl
Date: Tue Feb  5 15:58:29 2008
New Revision: 618830

URL: http://svn.apache.org/viewvc?rev=618830&view=rev
Log:
o Improve the XML parser in a similar way that was done in MCHANGES-66.

Modified:
    
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraXML.java

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraXML.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraXML.java?rev=618830&r1=618829&r2=618830&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraXML.java
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraXML.java
 Tue Feb  5 15:58:29 2008
@@ -41,7 +41,7 @@
 {
     private List issueList;
 
-    private String currentElement;
+    private StringBuffer currentElement = new StringBuffer( 1024 );
 
     private String currentParent = "";
 
@@ -87,65 +87,60 @@
         }
         else if ( qName.equals( "key" ) )
         {
-            issue.setKey( currentElement );
+            issue.setKey( currentElement.toString().trim() );
         }
         else if ( qName.equals( "summary" ) )
         {
-            issue.setSummary( currentElement );
+            issue.setSummary( currentElement.toString().trim() );
         }
         else if ( qName.equals( "type" ) )
         {
-            issue.setType( currentElement );
+            issue.setType( currentElement.toString().trim() );
         }
         else if ( qName.equals( "link" ) && currentParent.equals( "item" ) )
         {
-            issue.setLink( currentElement );
+            issue.setLink( currentElement.toString().trim() );
         }
         else if ( qName.equals( "priority" ) )
         {
-            issue.setPriority( currentElement );
+            issue.setPriority( currentElement.toString().trim() );
         }
         else if ( qName.equals( "status" ) )
         {
-            issue.setStatus( currentElement );
+            issue.setStatus( currentElement.toString().trim() );
         }
         else if ( qName.equals( "resolution" ) )
         {
-            issue.setResolution( currentElement );
+            issue.setResolution( currentElement.toString().trim() );
         }
         else if ( qName.equals( "assignee" ) )
         {
-            issue.setAssignee( currentElement );
+            issue.setAssignee( currentElement.toString().trim() );
         }
         else if ( qName.equals( "reporter" ) )
         {
-            issue.setReporter( currentElement );
+            issue.setReporter( currentElement.toString().trim() );
         }
         else if ( qName.equals( "version" ) )
         {
-            issue.setVersion( currentElement );
+            issue.setVersion( currentElement.toString().trim() );
         }
         else if ( qName.equals( "fixVersion" ) )
         {
-            issue.setFixVersion( currentElement );
+            issue.setFixVersion( currentElement.toString().trim() );
         }
         else if ( qName.equals( "component" ) )
         {
-            issue.setComponent( currentElement );
+            issue.setComponent( currentElement.toString().trim() );
         }
 
-        currentElement = "";
+        currentElement.setLength( 0 );
     }
 
     public void characters( char[] buf, int offset, int len )
         throws SAXException
     {
-        String s = new String( buf, offset, len );
-
-        if ( !s.trim().equals( "" ) )
-        {
-            currentElement = currentElement + s.trim() + "\n";
-        }
+        currentElement.append( buf, offset, len );
     }
 
     public List getIssueList()