Bug #19318 for YAML-Syck: Incorrect parsing of newlines

Wed May 17 17:28:47 2006 kevinj [...] cpan.org - Ticket created

Subject:

Incorrect parsing of newlines

This is the same as bug 19313 which I reported in JSON::Syck perl -MYAML::Syck -le 'print YAML::Syck::Dump({a => "hello\nworld\n"})' should produce: --- a: "hello\nworld\n" but instead produces --- a: "hello\n\ world\n" \n characters are being translated into a \n followed by a literal newline and two spaces.

Fri May 26 16:51:49 2006 kevinj [...] cpan.org - Correspondence added

Here's my patch which fixes this bug... It doesn't break any unit tests in this package, JSON::Syck, nor why's libsyck

p4 edit //depot/users/kevinj/YAML-Syck-0.44/emitter.c Index: emitter.c --- emitter.c.~1~ Fri May 26 13:38:31 2006 +++ emitter.c Fri May 26 13:38:31 2006 @@ -885,7 +885,7 @@ case '\n': end = mark + 1; syck_emitter_write( e, "\\n", 2 ); - do_indent = 2; + //do_indent = 2; start = mark + 1; if ( start < str + len && ( *start == ' ' || *start == '\n' ) ) { do_indent = 0; p4 edit //depot/users/kevinj/YAML-Syck-0.44/t/json-basic.t Index: t/json-basic.t --- t/json-basic.t.~1~ Fri May 26 13:38:31 2006 +++ t/json-basic.t Fri May 26 13:38:31 2006 @@ -41,6 +41,7 @@ '{"foo":""}', '["\"://\""]', '"~foo"', + '{"hw":"Hello\nWorld\n"}', ); plan tests => scalar @tests * (1 + $HAS_JSON) * 2; End of Patch.

Fri May 26 16:51:52 2006 The RT System itself - Status changed from 'new' to 'open'

Sat Feb 16 07:01:54 2008 cpan [...] audreyt.org - Correspondence added

Marking as resolved (as of YAML 1.01).

Sat Feb 16 07:01:59 2008 cpan [...] audreyt.org - Status changed from 'open' to 'resolved'