Request 555262: Include column number in the XML output

build.xml

@@ -187,6 +187,7 @@
                      includes="**/*.java"
                      excludes="**/Generated*.java"/>
             <formatter type="plain"/>
+            <formatter type="xml" toFile="classes/cs_errors.xml"/>
         </checkstyle>
     </target>
 

docs/releasenotes.html

@@ -26,6 +26,7 @@
     <td valign="top" align="left" nowrap="true" bgcolor="#EEEEEE">
       <p>Checkstyle 2</p>
       <ul>
+        <li><a href="#release2_3">Release 2.3</a></li>
         <li><a href="#release2_2">Release 2.2</a></li>
         <li><a href="#release2_1">Release 2.1</a></li>
         <li><a href="#release2_0">Release 2.0</a></li>
@@ -35,6 +36,24 @@
     <!--Content-->
     <td width="100%" valign="top" align="left">
 
+      <a name="release2_3"></a>
+      <h2>Release 2.3</h2>
+      <p class="body">
+        New features:
+        <ul>
+          <!--(Feature request 555262)-->
+          <li class="body">Include column number in the XML output (request 555262).</li>
+        </ul>
+      </p>
+
+      <p class="body">
+        Resolved bugs:
+        <ul>
+          <li class="body">Bugs! What bugs?</li>
+        </ul>
+      </p>
+
+
       <a name="release2_2"></a>
       <h2>Release 2.2</h2>
       <p class="body">

src/checkstyle/com/puppycrawl/tools/checkstyle/XMLLogger.java

@@ -109,9 +109,11 @@ public class XMLLogger
     /** @see AuditListener **/
     public void addError(AuditEvent aEvt)
     {
-        mWriter.println("<error " +
-                        "line=\"" + aEvt.getLine() + "\" " +
-                        "message=\"" + encode(aEvt.getMessage()) + "\"/>");
+        mWriter.print("<error" + " line=\"" + aEvt.getLine() + "\"");
+        if (aEvt.getColumn() > 0) {
+            mWriter.print(" column=\"" + aEvt.getColumn() + "\"");
+        }
+        mWriter.println(" message=\"" + encode(aEvt.getMessage()) + "\"/>");
     }
 
     /** @see AuditListener **/