svn commit: r831173 - in /tapestry/tapestry4/trunk/tapestry-framework/src: java/org/apache/tapestry/markup/MarkupUtils.java test/org/apache/tapestry/html/TestRelation.java

View: New views
1 Messages — Rating Filter:   Alert me  

svn commit: r831173 - in /tapestry/tapestry4/trunk/tapestry-framework/src: java/org/apache/tapestry/markup/MarkupUtils.java test/org/apache/tapestry/html/TestRelation.java

by andyhot-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Author: andyhot
Date: Fri Oct 30 00:33:49 2009
New Revision: 831173

URL: http://svn.apache.org/viewvc?rev=831173&view=rev
Log:
turns out there was no reason to leave so much whitespace for conditional comments (related to TAPESTRY-2750)

Modified:
    tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/markup/MarkupUtils.java
    tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/html/TestRelation.java

Modified: tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/markup/MarkupUtils.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/markup/MarkupUtils.java?rev=831173&r1=831172&r2=831173&view=diff
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/markup/MarkupUtils.java (original)
+++ tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/markup/MarkupUtils.java Fri Oct 30 00:33:49 2009
@@ -27,14 +27,11 @@
 
     public static void beginConditionalComment(IMarkupWriter writer, String condition)
     {
-        writer.println();
-        writer.printRaw("<!--[if " + condition + "]>");
-        writer.println();
+        writer.printRaw("<!--[if " + condition + "]>");        
     }
 
     public static void endConditionalComment(IMarkupWriter writer)
     {
-        writer.println();
         writer.printRaw("<![endif]-->");
         writer.println();
     }

Modified: tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/html/TestRelation.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/html/TestRelation.java?rev=831173&r1=831172&r2=831173&view=diff
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/html/TestRelation.java (original)
+++ tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/html/TestRelation.java Fri Oct 30 00:33:49 2009
@@ -183,8 +183,8 @@
         assertBuffer("");
 
         // now check what Shell has gathered
-        assertEquals(shellOutput.toString(), SYSTEM_NEWLINE + "<!--[if IE]>" + SYSTEM_NEWLINE +
-                "<style type=\"text/css\">Some css rules</style>" + SYSTEM_NEWLINE +
+        assertEquals(shellOutput.toString(), "<!--[if IE]>" +
+                "<style type=\"text/css\">Some css rules</style>" +
                 "<![endif]-->" + SYSTEM_NEWLINE);
         verify();
     }