github-repos/config
1
0
Fork 0
mirror of https://github.com/lightbend/config.git synced 2025-03-23 07:40:25 +08:00
Code Issues Packages Projects Releases Wiki Activity

Don't add multiple spaces in front of comments, fixes #87

Browse Source
This commit is contained in:
Havoc Pennington 2013-09-19 11:41:58 -04:00
parent c545910761
commit 1653cc8e88
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
config/src/main/java/com/typesafe/config/impl/SimpleConfigObject.java
View File

@ -390,7 +390,9 @@ final class SimpleConfigObject extends AbstractConfigObject implements Serializa
if (options.getComments()) {
for (String comment : v.origin().comments()) {
indent(sb, indent + 1, options);
sb.append("# ");
sb.append("#");
if (!comment.startsWith(" "))
sb.append(' ');
sb.append(comment);
sb.append("\n");
}

9
config/src/test/scala/com/typesafe/config/impl/ConfigTest.scala
View File

@ -1059,6 +1059,15 @@ class ConfigTest extends TestUtils {
throw e
}
}
// rendering repeatedly should not make the file different (e.g. shouldn't make it longer)
// unless the debug comments are in there
if (!renderOptions.getOriginComments()) {
val renderedAgain = resolvedParsed.root.render(renderOptions)
// TODO the strings should be THE SAME not just the same length,
// but there's a bug right now that sometimes object keys seem to
// be re-ordered. Need to fix.
assertEquals("render changed, resolved options=" + renderOptions, resolvedRender.length, renderedAgain.length)
}
}
}
}