ldh0826 / snakeyaml

Automatically exported from code.google.com/p/snakeyaml
Apache License 2.0
0 stars 0 forks source link

Add split lines option to DumperOptions to allow line splitting to be disabled #206

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Development is done.

The tests are incomplete.
Test coverage for Emitter should not go down.
To see the coverage run 'mvn site'

Original issue reported on code.google.com by py4fun@gmail.com on 12 Feb 2015 at 4:58

GoogleCodeExporter commented 8 years ago
I've pushed the additional unit tests to my cloned repository here:

https://code.google.com/r/vkoskela-spltlines/source/detail?r=9a04ba54432f07155fb
48cc9fcc3a5d4b7e440ed

It is also attached as a patch on top of my previous changes.

This should complete line coverage in Emitter for changes introduced by split 
lines. Please let me know if I missed anything!

Cheers,
Ville

Original comment by vkosk...@groupon.com on 14 Feb 2015 at 4:39

Attachments:

GoogleCodeExporter commented 8 years ago
The change is taken from the repo.
One deviation:
Emitter was changed to remove one occurrence of 'splitLines' because it was 
related to complex keys: when a key is itself a map. Since this is not an issue 
for Java (Map does not implement Comparable) I have removed the check.
Please review the change. If it is Ok for you it will be released.

Original comment by py4fun@gmail.com on 16 Feb 2015 at 12:57

GoogleCodeExporter commented 8 years ago
Delivered in version 1.15

Original comment by py4fun@gmail.com on 20 Feb 2015 at 11:03