diff options
| author | Justin Luth <justin.luth@collabora.com> | 2022-12-02 14:59:08 -0500 |
|---|---|---|
| committer | Justin Luth <jluth@mail.com> | 2022-12-05 18:36:29 +0000 |
| commit | fd14dd0e4c6a7ea0866a686f653db90301d664b6 (patch) | |
| tree | 15a17f82b7ba14894dd0081d04f67cf18352edc8 | |
| parent | 9d93e57613fc5e9f67400a166dcce5344c238348 (diff) | |
NFC cleanup docxattributeoutput: use existing function to clear
The function already clears these things,
so use it - which should also prevent bugs
where a clear is put in one place and not the other.
A dead giveaway was that m_bHasId = false
is also done in the function.
Change-Id: Ibb97b8da246404841afa5a03d11514dbcc7157b0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143604
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
| -rw-r--r-- | sw/source/filter/ww8/docxattributeoutput.cxx | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx b/sw/source/filter/ww8/docxattributeoutput.cxx index 6a01be448df8..5eb1c0ba306f 100644 --- a/sw/source/filter/ww8/docxattributeoutput.cxx +++ b/sw/source/filter/ww8/docxattributeoutput.cxx @@ -679,11 +679,7 @@ void SdtBlockHelper::WriteSdtBlock(const ::sax_fastparser::FSHelperPtr& pSeriali // clear sdt status m_nSdtPrToken = 0; - m_pTokenChildren.clear(); - m_pDataBindingAttrs.clear(); - m_pTextAttrs.clear(); - m_aAlias.clear(); - m_bHasId = false; + DeleteAndResetTheLists(); } void SdtBlockHelper::WriteExtraParams(const ::sax_fastparser::FSHelperPtr& pSerializer) @@ -1085,7 +1081,6 @@ void DocxAttributeOutput::EndParagraph( ww8::WW8TableNodeInfoInner::Pointer_t pT //These should be written out to the actual Node and not to the anchor. //Clear them as they will be repopulated when the node is processed. m_aParagraphSdt.m_nSdtPrToken = 0; - m_aParagraphSdt.m_bHasId = false; m_aParagraphSdt.DeleteAndResetTheLists(); } |