Fix position on MXParser.

Author: joehni

src/main/java/org/codehaus/plexus/util/xml/pull/MXParser.java

@@ -464,7 +464,7 @@ private void reset()
         // System.out.println("reset() called");
         location = null;
         lineNumber = 1;
-        columnNumber = 0;
+        columnNumber = 1;
         seenRoot = false;
         reachedEnd = false;
         eventType = START_DOCUMENT;
@@ -3103,7 +3103,7 @@ else if ( !seenInnerTag )
                     {
                         // seenPITarget && !seenQ
                         throw new XmlPullParserException( "processing instruction started on line " + curLine
-                            + " and column " + curColumn + " was not closed", this, null );
+                            + " and column " + (curColumn -2) + " was not closed", this, null );
                     }
                 }
                 else if ( ch == '<' )

src/test/java/org/codehaus/plexus/util/xml/pull/MXParserTest.java

@@ -351,6 +351,35 @@ public void testValidCharacterReferenceDecimal()
      *
      * @throws java.lang.Exception if any.
      */
+    @Test
+    public void testParserPosition()
+            throws Exception
+    {
+        String input = "<?xml version=\"1.0\" encoding=\"UTF-8\"?><!-- A --> \n <!-- B --><test>\tnnn</test>\n<!-- C\nC -->";
+
+        MXParser parser = new MXParser();
+        parser.setInput( new StringReader( input ) );
+
+        assertEquals( XmlPullParser.PROCESSING_INSTRUCTION, parser.nextToken() );
+        assertPosition( 1, 39, parser );
+        assertEquals( XmlPullParser.COMMENT, parser.nextToken() );
+        assertPosition( 1, 49, parser );
+        assertEquals( XmlPullParser.IGNORABLE_WHITESPACE, parser.nextToken() );
+        assertPosition( 2, 3, parser ); // end when next token starts
+        assertEquals( XmlPullParser.COMMENT, parser.nextToken() );
+        assertPosition( 2, 12, parser );
+        assertEquals( XmlPullParser.START_TAG, parser.nextToken() );
+        assertPosition( 2, 18, parser );
+        assertEquals( XmlPullParser.TEXT, parser.nextToken() );
+        assertPosition( 2, 23, parser ); // end when next token starts
+        assertEquals( XmlPullParser.END_TAG, parser.nextToken() );
+        assertPosition( 2, 29, parser );
+        assertEquals( XmlPullParser.IGNORABLE_WHITESPACE, parser.nextToken() );
+        assertPosition( 3, 2, parser ); // end when next token starts
+        assertEquals( XmlPullParser.COMMENT, parser.nextToken() );
+        assertPosition( 4, 6, parser );
+    }
+
     @Test
     public void testProcessingInstruction()
         throws Exception
@@ -622,7 +651,7 @@ public void testMalformedProcessingInstructionNoClosingQuestionMark()
         }
         catch ( XmlPullParserException ex )
         {
-            assertTrue( ex.getMessage().contains( "processing instruction started on line 1 and column 2 was not closed" ) );
+            assertTrue( ex.getMessage().contains( "processing instruction started on line 1 and column 1 was not closed" ) );
         }
     }
 
@@ -655,7 +684,7 @@ public void testSubsequentMalformedProcessingInstructionNoClosingQuestionMark()
         }
         catch ( XmlPullParserException ex )
         {
-            assertTrue( ex.getMessage().contains( "processing instruction started on line 1 and column 13 was not closed" ) );
+            assertTrue( ex.getMessage().contains( "processing instruction started on line 1 and column 12 was not closed" ) );
         }
     }
 
@@ -898,4 +927,8 @@ public void testEncodingISO_8859_1_setInputStream()
         }
     }
 
+    private static void assertPosition(int row, int col, MXParser parser) {
+        assertEquals("Current line", row, parser.getLineNumber());
+        assertEquals("Current column", col, parser.getColumnNumber());
+    }
 }