GitLab

Removed some extra stuff in the comment/newline in flow map tests, which really should be illegal (since implicit keys can't span multiple lines). It would be impossible to fix if we keep the immediate-output we're doing now - the only way to prevent it would be to hold on to a key's text until we got to the value token to make sure it could be an implicit key

Added explicit conversion from an iterator value to a Node. This conversion was always allowed (since the iterator value is derived from Node, but since Node has a templated constructor, that would take precedence over the derived-to-base conversion. This didn't seem to be a problem in gcc or clang, but MSVC seems to have trouble. (new API)

Fixed compiler error in iterator_base friend forward declaration in node on clang (and I hope MSVC), plus warnings on clang

Set the default operator >> to not compile unless there is a scalar conversion, so it doesn't interfere with user-defined types

Switched the utf bom checking to putback in the stream (instead of keeping a secondary buffer), which fixes a bug when there's only one ascii character

Split the yaml.h file into new/old API, which we'll then copy to yaml.h at build time (so the right one gets installed)

Set the default build to the old API, and removed the duplicate spec test implementation for the old api