Fix an == that should have been equals in ConfigBeanImpl.java
Fix missing equals/hashCode on IgnoredWhitespace token
Add some `<?>` that Xlint:unchecked wanted
Make RenderComparator serializable just to silence findbugs
findbugs still reports some "serializable without void
constructors" but those are not accurate because we replace
the instance with another class before serializing.
At present, we don't make a file() include relative to the
file doing the including. This is discussed in #202.
We should most likely change this behavior, but this
commit documents and tests it.
For broken doc comments in impl, just deleted them for
the most part rather than writing a bunch of useless docs.
"impl" is now included in the docs because javadoc 8 seems
to explode otherwise, but it's well-labeled as internal ABI.
Fixes#176 by ensuring SerializedConfigValue implements the java.lang.Object
contract without exploding. This is a completely internal class, but
-Dsun.io.serialization.extendedDebugInfo=true calls toString on it.
This is a rearrangement of ConfigFactory so that defaultApplication is public.
Also fixes a bug where we were caching the overload that takes parse and resolve
options, without putting those options in the cache key. Moved to only cache
the actual default config (the one that doesn't take any options).
Fixes#196 plus the separate bug about bad caching.
Clean up the lossless tokens code based on feedback.
* Change nextTokenFromIterator() method to
nextTokenIgnoringWhitespace()
* Use StringBuilder in Tokenizer.render()
* Extract Whitespace token creation into a
`createWhitespaceTokenFromSaver` method
* Add DoubleSlashComment and HashComment subclasses of
Comment. Make Comment abstract.
* Make `tokenText()` method virtual and overload it in
subclasses that derive their original token text entirely
from other fields.
* Use `tokenizerTest` method in all relevant Tokenizer tests
* Add extra multi-line comment Tokenizer tests
Keep the original text of all tokens when Tokenizing a config
minus comments. Add a render method to the TokenIterator class
which generates the original text from which the tokens were
parsed.
Fixes#212 that object and list substitutions could not have
whitespace in between.
This patch carries quoted-ness into ConfigString, which is somewhat
bogus, and we may be able to clean it up later if we start to track
the tokens that each value originates from.
Fixes#228 reported by @derrickburns; since we can interpret
numeric keys as lists, we sort the keys numerically to make
it easy to read the list value in order.
