[lmi] count_lines() [Was: Group premium quotes]

[Greg Chicares]

On 2015-08-16 17:07, Greg Chicares wrote:
> First of all, don't read this before September, especially not if you're
> on vacation.

I realize this may not matter much, but:

// Return the number of lines in a possibly multiline string.
std::size_t count_lines(std::string const& s)
{
    return 1u + std::count(s.begin(), s.end(), '\n');
}

It looks like these string arguments:
  "__1__\n__2__"
  "__1__\n__2__\n"
would return different counts. Given the "1u +", I suppose we're assuming
that there's no trailing '\n', in which case we should at least document
that; but if it's worth documenting, it's worth asserting. Untested idea:
  LMI_ASSERT(s.rend() && '\n' != *s.rend());