Test issue_convert_valid_utf32_to_latin1_2104d5a31440e3ed shows inconsistency in implementations #760
Description
const auto outlen = implementation.latin1_length_from_utf32(data_len);
ASSERT_EQUAL(outlen, 1);
std::vector<char> output(outlen);
const auto r = implementation.convert_valid_utf32_to_latin1(
(const char32_t *)data, data_len, output.data());
/*
* fallback gets 1, the others 0
*/
ASSERT_EQUAL(r, 0);The comment and assertion states something violating our spec. The spec for convert_valid_utf32_to_latin1 says "number of written code units; 0 if conversion is not possible".