diff options
| author | Norbert Thiebaud <nthiebaud@gmail.com> | 2013-02-22 01:43:39 -0600 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2013-02-24 23:31:12 +0000 |
| commit | bb717296bf66f0acb84d5f84c38390f3e44c7b62 (patch) | |
| tree | 21f3d89f6e33c6bb0f1d8da9096f7ae922dffe96 /xmloff | |
| parent | df37f7d8e82d8d0854076c3c9be64ee3a155f7bf (diff) | |
coverity#705347 Missing break in switch
Change-Id: Ife49c096722fff204a140be970aa0692281e8b86
Reviewed-on: https://gerrit.libreoffice.org/2327
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'xmloff')
| -rw-r--r-- | xmloff/source/text/txtfldi.cxx | 28 |
1 files changed, 16 insertions, 12 deletions
diff --git a/xmloff/source/text/txtfldi.cxx b/xmloff/source/text/txtfldi.cxx index e7067acc677b..90d23a02502b 100644 --- a/xmloff/source/text/txtfldi.cxx +++ b/xmloff/source/text/txtfldi.cxx @@ -2670,22 +2670,26 @@ void XMLPageVarSetFieldImportContext::ProcessAttribute( { switch (nAttrToken) { - case XML_TOK_TEXTFIELD_ACTIVE: + case XML_TOK_TEXTFIELD_ACTIVE: + { + bool bTmp(false); + if (::sax::Converter::convertBool(bTmp, sAttrValue)) { - bool bTmp(false); - if (::sax::Converter::convertBool(bTmp, sAttrValue)) - { - bActive = bTmp; - } + bActive = bTmp; } - case XML_TOK_TEXTFIELD_PAGE_ADJUST: + break; + } + case XML_TOK_TEXTFIELD_PAGE_ADJUST: + { + sal_Int32 nTmp(0); + if (::sax::Converter::convertNumber(nTmp, sAttrValue)) { - sal_Int32 nTmp; - if (::sax::Converter::convertNumber(nTmp, sAttrValue)) - { - nAdjust = (sal_Int16)nTmp; - } + nAdjust = (sal_Int16)nTmp; } + break; + } + default: + break; } } |