Fixing some buffer problems in the default message holder.
authorJohn Dennis Casey <jdcasey@apache.org>
Fri, 9 Jun 2006 00:05:04 +0000 (00:05 +0000)
committerJohn Dennis Casey <jdcasey@apache.org>
Fri, 9 Jun 2006 00:05:04 +0000 (00:05 +0000)
git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@412887 13f79535-47bb-0310-9956-ffa450edef68

pom.xml
src/main/java/org/apache/maven/shared/io/logging/DefaultMessageHolder.java
src/test/java/org/apache/maven/shared/io/logging/DefaultMessageHolderTest.java

diff --git a/pom.xml b/pom.xml
index ba25b3c..ff1aa23 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -43,7 +43,7 @@
     <dependency>
       <groupId>org.apache.maven.wagon</groupId>
       <artifactId>wagon-provider-api</artifactId>
-      <version>[1.0-alpha-6,1.0]</version>
+      <version>1.0-alpha-6</version>
     </dependency>
     <dependency>
       <groupId>org.codehaus.plexus</groupId>
index a2d5e21..25845c1 100644 (file)
@@ -17,7 +17,7 @@ public class DefaultMessageHolder
     public MessageHolder addMessage( CharSequence messagePart, Throwable error )
     {
         newMessage();
-        append( messagePart );
+        append( messagePart.toString() );
         append( error );
         
         return this;
@@ -26,7 +26,7 @@ public class DefaultMessageHolder
     public MessageHolder addMessage( CharSequence messagePart )
     {
         newMessage();
-        append( messagePart );
+        append( messagePart.toString() );
         
         return this;
     }
@@ -46,7 +46,7 @@ public class DefaultMessageHolder
             newMessage();
         }        
         
-        currentMessage.append( messagePart );
+        currentMessage.append( messagePart.toString() );
         
         return this;
     }
@@ -90,7 +90,7 @@ public class DefaultMessageHolder
             if ( content.length() > 0 )
             {
                 buffer.append( '[' ).append( counter++ ).append( "] " );
-                buffer.append( content );
+                buffer.append( content.toString() );
                 
                 if ( it.hasNext() )
                 {
@@ -120,7 +120,7 @@ public class DefaultMessageHolder
         
         public Message append( CharSequence message )
         {
-            this.message.append( message );
+            this.message.append( message.toString() );
             return this;
         }
         
index a460866..ed7b4fe 100644 (file)
@@ -274,4 +274,12 @@ public class DefaultMessageHolderTest
         assertFalse( new DefaultMessageHolder().newMessage().isEmpty() );
     }
     
+    public void testAppendCharSequence()
+    {
+        MessageHolder mh = new DefaultMessageHolder();
+        mh.newMessage().append( new StringBuffer( "This is a test" ) );
+        
+        assertTrue( mh.render().indexOf( "This is a test" ) > -1 );
+    }
+    
 }