name | [configuration] Fix multi-line strings handling |
---|---|
revisions | #9a8f910ac6f9 [configuration] Fix multi-line strings handling, #86e00b30862a [configuration] Fix multi-line strings handling |
ticket | #3691182 generate_config mess up multi-line strings [validation pending] |
committer | <not specified> |
reviewer | <not specified> |
Comments
-
2016/01/18 08:37, written by cdevienne
-
2016/01/19 08:19, written by dlaxalde
add commentValues are stripped before being written to the ouput, meaning a leading
\n
would be dropped no matter what (and it always has been like this).What we could do if we want consistency is to strip value when reading, so that any leading
\n
is ignored.The other option is to "fix" the initial behavior and not drop leading
\n
when writing, as a part of the multi-line support. I guess it makes sens to do so.I didn't go into all the details of this code but was just wondering about this "roundtrip" consistency. So any solution that would make the configuration pass through the roundtrip would be fine IMO.