review comments fixed

This commit is contained in:
John Lonergan 2016-08-24 00:48:51 +01:00
parent 2243a011d5
commit 3cadb2756d
3 changed files with 4 additions and 3 deletions

View File

@ -840,7 +840,6 @@ format.
* config-annotation https://github.com/wacai/config-annotation
* PureConfig https://github.com/melrief/pureconfig
* Simple Scala Config https://github.com/ElderResearch/ssc
* konfig https://github.com/vpon/konfig
#### Clojure wrappers for the Java library

View File

@ -481,7 +481,7 @@ final class Tokenizer {
// the open quote has already been consumed
StringBuilder sb = new StringBuilder();
// We need a sec ond string builder to keep track of escape characters.
// We need a second string builder to keep track of escape characters.
// We want to return them exactly as they appeared in the original text,
// which means we will need a new StringBuilder to escape escape characters
// so we can also keep the actual value of the string. This is gross.

View File

@ -428,6 +428,7 @@ abstract trait TestUtils {
"# bar\n", // just a comment with a newline
"# foo\n//bar", // comment then another with no newline
"""{ "foo" = 42 }""", // equals rather than colon
"""{ "foo" = (42) }""", // equals rather than colon
"""{ foo { "bar" : 42 } }""", // omit the colon for object value
"""{ foo baz { "bar" : 42 } }""", // omit the colon with unquoted key with spaces
""" "foo" : 42 """, // omit braces on root object
@ -528,6 +529,7 @@ abstract trait TestUtils {
body
} catch {
case t: Throwable =>
println(t)
val tokens = try {
"tokens: " + tokenizeAsList(s)
} catch {
@ -723,7 +725,7 @@ abstract trait TestUtils {
val f = new File("config/src/test/resources")
if (!f.exists()) {
val here = new File(".").getAbsolutePath
throw new Exception(s"Tests must be run from the root project directory containing ${f.getPath()}, however the current director is $here")
throw new Exception(s"Tests must be run from the root project directory containing ${f.getPath()}, however the current directory is $here")
}
f
}