summaryrefslogtreecommitdiff
path: root/writerfilter/source/dmapper/DomainMapper_Impl.cxx
diff options
context:
space:
mode:
authorJustin Luth <justin.luth@collabora.com>2020-09-22 08:48:51 +0300
committerCaolán McNamara <caolanm@redhat.com>2020-09-22 13:53:05 +0200
commita06fe0d7ee665be29f3218f093fcabd601bf99ac (patch)
tree837e9226e7ab29c2821a243076eaaf8037a33294 /writerfilter/source/dmapper/DomainMapper_Impl.cxx
parenta219cb202d6c13c48c315f45796f459443fa6c9a (diff)
tdf#136706 writerfilter: revert my info-gathering assert
The assert was failing in crashTesting for: wget http://bugs.documentfoundation.org/attachment.cgi?id=68205 -O tdf55725-1.docx ./instdir/program/soffice.bin --headless --convert-to docx ./tdf55725-1.docx and also for wget https://bugs.documentfoundation.org/attachment.cgi?id=127288 -O tdf102131-1.docx Both of these documents have SAX errors when loading. Running these interactively, saying "no, do not continue loading" parses a bit more to give the specific error and that is what was running into the assert. Actually importing the document itself does not hit the assert, nor does round-tripping it. Since this is a stupid enough situation, and because I can't think of a good way to word a SAL_WARN, and because I can't say that I have detailed knowledge about this area, I'll just remove the whole thing instead of leaving a warning. Change-Id: If50b06f0103adca348ca314ebc92e7e4045967dc Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103133 Tested-by: Jenkins Reviewed-by: Justin Luth <justin_luth@sil.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'writerfilter/source/dmapper/DomainMapper_Impl.cxx')
-rw-r--r--writerfilter/source/dmapper/DomainMapper_Impl.cxx1
1 files changed, 0 insertions, 1 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index dd13f18c7011..cd985e4fd995 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -679,7 +679,6 @@ void DomainMapper_Impl::PopProperties(ContextType eId)
{
if (m_aPropertyStacks[eId].size() == 1) // tdf#112202 only top level !!!
{
- assert( !IsInFootOrEndnote() && !m_bIsInComments );
m_pLastSectionContext = m_aPropertyStacks[eId].top();
}
}