DesktopLOKTest::testRedlineWriter is failing

Tomáš Chvátal tomas.chvatal at
Fri Nov 10 09:17:44 UTC 2017

2017-11-06 9:58 GMT+01:00 Stephan Bergmann <sbergman at>:

> Looking at the surrounding code
>     // Get redline info.
>>     boost::property_tree::ptree aTree;
>>     char* pJSON = pDocument->m_pDocumentClass->getCommandValues(pDocument,
>> ".uno:AcceptTrackedChanges");
>>     std::stringstream aStream(pJSON);
>>     free(pJSON);
>>     CPPUNIT_ASSERT(!aStream.str().empty());
>>     boost::property_tree::read_json(aStream, aTree);
>>     // Make sure that pressing a key creates exactly one redline.
>>     CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(1),
>> aTree.get_child("redlines").size());
> what's the binary content of that "char* pJSON" stream?  If that's still
> intact, then it's apparently boost::property_tree::read_json that's
> breaking things.  And if that's already garbled, then it's apparently the
> producer side of that pDocument->m_pDocumentClass->getCommandValues
> (whatever that is) that is breaking it and needs further investigation.
> I checked the read_json function and it was indeed rewritten in between
the boost versions on the distro (and there were some comments about
unicode support). So I had updated the boost function (basically to use
latest boost code) in my testing repo and it still fails:

[ 6427s] /home/abuild/rpmbuild/BUILD/libreoffice-
[ 6427s] equality assertion failed
[ 6427s] - Expected: Insert “t”
[ 6427s] - Actual  : Insert “tâ€

 FWIW I enabled it on the OBS so the full build log is tracked here:

Any idea how to check the getCommandValues behaviour if it is the proper


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the LibreOffice mailing list