From 558b5a1183e840007e554a05082ade0b8e3e3f92 Mon Sep 17 00:00:00 2001 From: Bogdan Buzea Date: Sun, 20 Oct 2024 21:11:50 +0200 Subject: 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 --- xmloff/source/draw/ximpshap.cxx | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'xmloff') 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(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 -- cgit