Wrong LF count at the end

[goforit22123-netizen]

I have following code:

YAML::Emitter out;
out << YAML::BeginMap;
out << YAML::Key << "data" << YAML::Value << YAML::Literal << "hello\nworld\n\n";
out << YAML::EndMap;
std::string tmp(out.c_str());

size_t tmp_cnt = std::count(tmp.begin(), tmp.end(), '\n');
std::cout << "lf count before save: " << tmp_cnt << std::endl;

auto node = YAML::Load(tmp);
auto result = node["data"].as<std::string>();

size_t cnt = std::count(result.begin(), result.end(), '\n');
std::cout << "lf count after load: " << cnt << std::endl;

and I expect 3 LF's in result string but it contains only 2.
I think it happens because yaml-cpp uses | instead of |+ for the literal content.
AFAIK, this problem is fixed in master branch but there is no new release that contains fix.
So, I decided to switch to the master version of yamp-cpp to get this update.
But in the master, we have another issues related to LF at the end.

Here is description of issues and code sample:

1. If literal_value equals to helloworld, LF count is 0 after load, it's OK
2. If literal_value equals to "helloworld\n" LF count is 0 after load, it's NOT OK (count should be 1)
3. If literal_value equals to helloworld\n\n LF count is 1 after load, it's NOT OK (count should be 2)

For some obscure reasons, yaml-cpp removes one LF when data is readed back through YAML::Load.

Here is code:

        YAML::Emitter out;
        std::string literal_value = "helloworld\n\n";
        out << YAML::BeginMap;
        out << YAML::Key << "data" << YAML::Value << YAML::Literal << literal_value;
        out << YAML::EndMap;
        std::string tmp(out.c_str());

        size_t tmp_cnt = std::count(tmp.begin(), tmp.end(), '\n');
        std::cout << "LF count before save: " << tmp_cnt << std::endl;

        auto node = YAML::Load(tmp);
        auto result = node["data"].as<std::string>();

        size_t cnt = std::count(result.begin(), result.end(), '\n');
        std::cout << "LF count after load: " << cnt << std::endl;

        std::cout << tmp;

So, currently I need both functionality:

1. Block Chomping Indicator
2. Correct LF count at the end

I apologize for asking, but my project deadline is approaching, and I would like to know roughly when to expect a new version of the library that addresses these issues (or at least fix in master branch). If the release fits within my deadline, I will wait; otherwise, I will switch to a different library to ensure I can complete my tasks on time.

Thanks in advance for your reply.

[goforit22123-netizen]

I think problem may be here:

1. https://github.com/jbeder/yaml-cpp/blob/master/src/emitterutils.cpp#L416
2. https://github.com/jbeder/yaml-cpp/blob/master/src/emitterutils.cpp#L419

From comments in the code:

// The chomping depends on the number of new lines.
// The output following this 'WriteLiteralString' call will add an additional '\n',
// because of this we need to remove one in the case of 'strip' and 'keep'.

in the sources the size for | and |- equals to size - 1.
This might have worked previously because the emitter added an extra \n.
However, now the emitter does not add an additional \n (in the master version), apparently this is the cause of the bug.