[Libreoffice-commits] core.git: xmloff/source

[Caolán McNamara (via logerrit)]

 xmloff/source/text/txtparai.cxx |   11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

New commits:
commit e17e3dd6d9c06f0dc25246a4e0fb82970d74ba39
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Fri Aug 7 09:43:14 2020 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Aug 7 21:44:29 2020 +0200

    cid#1465671 Unchecked return value
    
    Change-Id: I796799f2964f70f107acc53d376379a79e044472
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100294
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/xmloff/source/text/txtparai.cxx b/xmloff/source/text/txtparai.cxx
index f957f216fc0e..54a86e01157b 100644
--- a/xmloff/source/text/txtparai.cxx
+++ b/xmloff/source/text/txtparai.cxx
@@ -1937,26 +1937,23 @@ XMLParaContext::XMLParaContext(
             break;
         case XML_TOK_TEXT_P_OUTLINE_CONTENT_VISIBLE:
             {
-                mbOutlineContentVisible = true;
-                ::sax::Converter::convertBool(mbOutlineContentVisible, rValue);
+                bool bBool(false);
+                if (::sax::Converter::convertBool(bBool, rValue))
+                    mbOutlineContentVisible = bBool;
             }
             break;
         case XML_TOK_TEXT_P_IS_LIST_HEADER:
             {
                 bool bBool(false);
-                if( ::sax::Converter::convertBool( bBool, rValue ) )
-                {
+                if (::sax::Converter::convertBool(bBool, rValue))
                     bIsListHeader = bBool;
-                }
             }
             break;
         case XML_TOK_TEXT_P_RESTART_NUMBERING:
             {
                 bool bBool(false);
                 if (::sax::Converter::convertBool(bBool, rValue))
-                {
                     bIsRestart = bBool;
-                }
             }
             break;
         case XML_TOK_TEXT_P_START_VALUE: