Author: vsiveton
Date: Mon May 28 14:56:50 2007
New Revision: 542331

URL: http://svn.apache.org/viewvc?view=rev&rev=542331
Log:
MCHANGELOG-58: Make the report easier to read by bolding the actual file name
Submitted by: Jacob Robertson
Reviewed by: Vincent Siveton

o patch applied

Modified:
    
maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java

Modified: 
maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java?view=diff&rev=542331&r1=542330&r2=542331
==============================================================================
--- 
maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java
 (original)
+++ 
maven/plugins/trunk/maven-changelog-plugin/src/main/java/org/apache/maven/plugin/changelog/ChangeLogReport.java
 Mon May 28 14:56:50 2007
@@ -1143,12 +1143,12 @@
         if ( linkFile != null )
         {
             sink.link( linkFile );
-            sink.text( name );
+            sinkFileName( name, sink );
             sink.link_();
         }
         else
         {
-            sink.text( name );
+            sinkFileName( name, sink );
         }
 
         sink.text( " " );
@@ -1163,6 +1163,33 @@
         {
             sink.text( "v " + revision );
         }
+    }
+
+    /**
+     * Encapsulates the logic for rendering the name with a bolded markup.
+     *
+     * @param name filename of the file that was changed
+     * @param sink the report formatting tool
+     */
+    private void sinkFileName( String name, Sink sink )
+    {
+        name = name.replaceAll( "\\\\", "/" );
+        int pos = name.lastIndexOf( '/' );
+
+        String head;
+        String tail;
+        if (pos < 0) {
+            head = "";
+            tail = name;
+        } else {
+            head = name.substring( 0, pos ) + "/";
+            tail = name.substring( pos + 1 );
+        }
+
+        sink.text( head );
+        sink.bold();
+        sink.text( tail );
+        sink.bold_();
     }
 
     /**


Reply via email to