MemConf: Do not add another new line when serializing (#2354)

2 files changed, 7 insertions, 1 deletions

diff --git a/src/main/scala/firrtl/passes/memlib/ReplaceMemTransform.scala b/src/main/scala/firrtl/passes/memlib/ReplaceMemTransform.scala
index 331dd43e..6f883db2 100644
--- a/src/main/scala/firrtl/passes/memlib/ReplaceMemTransform.scala
+++ b/src/main/scala/firrtl/passes/memlib/ReplaceMemTransform.scala
@@ -69,7 +69,7 @@ case class MemLibOutConfigFileAnnotation(file: String, annotatedMemories: Seq[De
       m.readwriters.length,
       m.maskGran.map(_.toInt)
     ).toString
-  }.mkString("\n").getBytes
+  }.mkString.getBytes
 }
 
 private[memlib] case class AnnotatedMemoriesAnnotation(annotatedMemories: List[DefAnnotatedMemory])
diff --git a/src/test/scala/firrtlTests/ReplSeqMemTests.scala b/src/test/scala/firrtlTests/ReplSeqMemTests.scala
index 414fc6af..9f83bec2 100644
--- a/src/test/scala/firrtlTests/ReplSeqMemTests.scala
+++ b/src/test/scala/firrtlTests/ReplSeqMemTests.scala
@@ -49,6 +49,12 @@ class ReplSeqMemSpec extends SimpleTransformSpec {
     }.get
     // Verify that this does not throw an exception
     val fromConf = MemConf.fromString(text)
+    // Verify that the number of lines in the Conf file is equivalent to the number
+    // of entries (no missing or extra newlines)
+    require(
+      fromConf.size == text.count(_ == '\n'),
+      s"Unexpected newlines in mem.conf file containing ${fromConf.size} confs:\n${text}"
+    )
     // Verify the mems in the conf are the same as the expected ones
     require(
       Set(fromConf: _*) == mems,