Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trailing whitespace at first EOL when dumping a hash [rt.cpan.org #46983] #38

Open
toddr opened this issue May 11, 2017 · 0 comments
Open

Comments

@toddr
Copy link
Member

toddr commented May 11, 2017

Migrated from rt.cpan.org#46983 (status was 'stalled')

Requestors:

Attachments:

From [email protected] on 2009-06-16 00:52:05:

When dumping a hash, YAML::Syck outputs a trailing whitespace at the end
of the first line, so that the first line is "--- \n".

This causes various problems: trailing whitespace is invisible in most
text editors; other text editors trim trailing whitespace; which ends up
in tests that fail even though the output appears identical.

Test file attached.

From [email protected] on 2010-05-20 11:12:40:

(This is a form-reply that isn't specific to your particular report)

YAML::Syck has just acquired one new maintainer (me), it still doesn't
have anyone that *cares* about it. But I'm willing to help solve your
report & release a new version with the fix if it's easy for me.

It now has a Git repository at:

    http://github.com/avar/YAML-Syck

If your report is a patch that fixes a problem, great. Please remake
the patch against Git by forking that repo and sending me a pull
request on GitHub (or an update to this bug if you prefer
git-format-patch(1) or some other repo provider..). Make sure to
include a test for what you fixed.

If your report is some code that fails (and you have a testcase for
it) a patch against the test suite to demonstrate that failure would
be very useful. It's OK if the test crashes and burns, see
Test::More's docs for how to make TODO tests that fail now, but
shouldn't. Even if it segfaults perl C<system $^X => qw/ -Mblib
-MYAML::Syck .../> or something like that and checking the return
value will do.

From [email protected] on 2010-07-19 20:25:09:

I'm making this a wish list item. patches welcome, but otherwise I don't see this getting fixed. 
Any fix would have to be fully backwards compatible, which I'm not sure this would be.

NOTE I also found that there's a leading space on references to duplicate items too. (&1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant