diff options
| author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-20 21:11:50 +0200 |
|---|---|---|
| committer | David Gilbert <freedesktop@treblig.org> | 2024-10-30 01:36:46 +0100 |
| commit | 558b5a1183e840007e554a05082ade0b8e3e3f92 (patch) | |
| tree | 85f1d9ca1dcd0f09920faf3ab54fedd1c5202447 /xmloff/source/draw/ximpshap.cxx | |
| parent | 167ebdaaf98ba0065081b6fe8319526586e54bbd (diff) | |
tdf#163486: PVS: Identical branches
V1037 Two or more case-branches perform the same actions. Check lines: 836, 890
Change-Id: If6036da666ea46a4e5c2bc0f4547c78aa43a26a9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175270
Tested-by: Jenkins
Reviewed-by: David Gilbert <freedesktop@treblig.org>
Diffstat (limited to 'xmloff/source/draw/ximpshap.cxx')
| -rw-r--r-- | xmloff/source/draw/ximpshap.cxx | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xmloff/source/draw/ximpshap.cxx b/xmloff/source/draw/ximpshap.cxx index 5e4354a5c7ec..79117cc9fe8f 100644 --- a/xmloff/source/draw/ximpshap.cxx +++ b/xmloff/source/draw/ximpshap.cxx @@ -831,10 +831,6 @@ bool SdXMLShapeContext::processAttribute( const sax_fastparser::FastAttributeLis case XML_ELEMENT(DRAW_EXT, XML_LAYER): maLayerName = aIter.toString(); break; - case XML_ELEMENT(DRAW, XML_TRANSFORM): - case XML_ELEMENT(DRAW_EXT, XML_TRANSFORM): - mnTransform.SetString(aIter.toString(), GetImport().GetMM100UnitConverter()); - break; case XML_ELEMENT(DRAW, XML_DISPLAY): case XML_ELEMENT(DRAW_EXT, XML_DISPLAY): mbVisible = IsXMLToken( aIter, XML_ALWAYS ) || IsXMLToken( aIter, XML_SCREEN ); @@ -883,6 +879,8 @@ bool SdXMLShapeContext::processAttribute( const sax_fastparser::FastAttributeLis else if (maSize.Height < 0) maSize.Height = o3tl::saturating_add<sal_Int32>(maSize.Height, -1); break; + case XML_ELEMENT(DRAW, XML_TRANSFORM): + case XML_ELEMENT(DRAW_EXT, XML_TRANSFORM): case XML_ELEMENT(SVG, XML_TRANSFORM): case XML_ELEMENT(SVG_COMPAT, XML_TRANSFORM): // because of #85127# take svg:transform into account and handle like |