newapplesho / NeoYAML / 29704757810

99%

push

github

fix: read YAML with CR and CRLF line breaks (#4)

The scanner and the document splitter both split on LF only, so a
CR-only stream (e.g. lines joined with `String cr`) was scanned as a
single physical line and the following keys were swallowed into the
previous value. CRLF only worked by accident, because the trailing CR
happened to be trimmed off scalar values.

Normalize CRLF and lone CR to LF up front via
`NeoYAMLScanner class>>normalizeLineBreaks:`, called from `scan:` (the
parser path) and `splitDocuments:` (the `---` document-marker path), as
YAML 1.2.2 5.4 mandates. Add reader tests covering CR and CRLF mappings
and a CR literal block scalar.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

24 of 24 new or added lines in 2 files covered. (100.0%)

1490 of 1499 relevant lines covered (99.4%)

1.99 hits per line