Also consider saved exceptions when terminating parse

Desktop / LibreOffice - Mike Kaganski [collabora.com] - 16 November 2019 19:13 EST

As with previous commit 18ae77a065cb8ae6940d4067f6ab7e99a3f74047, this will start showing parse errors on invalid files which previously just opened without warnings, silently losing the invalid stream part. Any bug bisected to this commit is not a regression from this commit! The real problem was already before, and was just disclosed by this (which is the actual goal).

Also simplify unit test data for tdf#128820, which will now be enough after the change.

A unit test (testN779627) revealed unexpected throws when parsing; this was fixed.

Change-Id: I5a21b9001874ec6e3b8273c10043ef930bf1cc82 Reviewed-on: https://gerrit.libreoffice.org/82981

9fdf8c0a5cc0 Also consider saved exceptions when terminating parse
sax/source/fastparser/fastparser.cxx | 14 +-
sw/qa/extras/ooxmlexport/data/tdf128820.fodt | 1353 +--------------------
writerfilter/source/dmapper/DomainMapper_Impl.cxx | 13 +-
writerfilter/source/dmapper/PropertyMap.cxx | 11 +-
4 files changed, 41 insertions(+), 1350 deletions(-)

Upstream: cgit.freedesktop.org


  • Share