Merge pull request #184 from olafos/master

Fix for #183

Author: ashawley

shared/src/main/scala/scala/xml/Utility.scala

@@ -201,7 +201,7 @@ object Utility extends AnyRef with parsing.TokenTests {
     minimizeTags: MinimizeMode.Value = MinimizeMode.Default): StringBuilder =
     {
       x match {
-        case c: Comment if !stripComments => c buildString sb
+        case c: Comment                   => if (!stripComments) c buildString sb; sb
         case s: SpecialNode               => s buildString sb
         case g: Group                     =>
           for (c <- g.nodes) serialize(c, g.scope, sb, stripComments, decodeEntities, preserveWhitespace, minimizeTags); sb

shared/src/test/scala/scala/xml/UtilityTest.scala

@@ -56,4 +56,11 @@ class UtilityTest {
     assertEquals("<node><leaf/></node>", Utility.serialize(x, minimizeTags = MinimizeMode.Always).toString)
   }
 
+  @Test
+  def issue183: Unit = {
+    val x = <node><!-- comment  --></node>
+    assertEquals("<node></node>", Utility.serialize(x, stripComments = true).toString)
+    assertEquals("<node><!-- comment  --></node>", Utility.serialize(x, stripComments = false).toString)
+  }
+
 }