GitLab

Fixes #917

Found with readability-else-after-return
Signed-off-by: Rosen Penev <rosenp@gmail.com>

Fix invocation of clang-format in CMakeLists and apply clang-format.

* Removed an expression which is always true
* The second expression (ch is space) is removed because the first one contains space 0x20
* nextEmptyLine is always false so it is removed from the expression

This improves performance on the test.yaml attached to #158 by about
25% on my machine as compared to the previous commit (0.25s -> 0.20s),
as measured by `time build/util/parse < test.yaml > /dev/null`.

Fixed folding bug (detecting indentation, example 8.2), and clipping/stripping empty strings (example 8.6)

Moved all the includes to a subfolder yaml-cpp so they don't interfere with other libraries' includes

Fixed flow folding, and made the separation slightly cleaner (but the whole scanscalar thing could use a major refactoring)

(Actually) fixed the folding newline bug, but it's a bit messy, and we don't accurately make the distinction between block folding and flow folding

Converted indexing to std::size_t, and fixed the Node templated overloads to properly index any index type (determining what is an index type is a bit of a hack - it should be is_convertible<T, std::size_t> (I think), but I just explicitly wrote down a list)

In particular:
1. Windows CR/LF weren't read properly (issue #11)
2. Scanning wasn't reading EOF properly
3. Documents may be empty (this was old, I think)
Also fixed some VS2008 warnings on /W4.

We were getting memory leaks (as told by the CRT detectors, which I also added), and there's really no reason (as long as we're careful) to use pointers there.

Combined the myriad ScannerExceptions and ParserExceptions to a single ParserException class that has a message and a line/column position in the file where the error occurred.

Fixed a bug in the last newline of a block folded scalar.

Moved the three scalar token scanning functions back to scantoken.cpp, so scanscalar.cpp now only has the main scalar scanning function.
Renamed ScanScalarInfo to ScanScalarParams.

Renamed the valid/possible tokens to a single variable status with enums valid, invalid, and unverified.

Moved the input stream, together with line/column info, into its own class, which allowed some other stuff just to pass the stream, and not have to be a member of Scanner.

No commit message