Quote:
Originally Posted by KevinH
Okay, I just pushed a change to our github master so that HTMLResource objects will use the QXmlStreamReader itself for well-formed checks instead of the earlier gumbo version (which still exists). I rebuilt with these changes and it fixes the issue by preventing Reports from launching if the xhtml in the book is not well-formed.
That said, the WellFormed checks are used in a number of places so we really need to test this change to make sure that increasing strictness here does not create other issues with Sigil being able to handle very poor html as input.
I have my fingers crossed that there will be no side effects!
KevinH
|
I grabbed the Windows Appveyor build of your latest commit, and it definitely seems to work for the specific test case I mentioned. What areas/features should I focus on to check for problems, do you think? Splitting/merging relies on things being well-formed, right?