From 9bdec0831afc45af45ea6d9350ef3372188b45cb Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 1 Nov 2016 13:10:08 +0200 Subject: loplugin:oncevar in sd..svgio Change-Id: I58565460fcc3279c9771c6e1272d0af540b2c87c Reviewed-on: https://gerrit.libreoffice.org/30459 Reviewed-by: Noel Grandin Tested-by: Noel Grandin --- sdext/source/pdfimport/wrapper/wrapper.cxx | 25 ++++++++----------------- 1 file changed, 8 insertions(+), 17 deletions(-) (limited to 'sdext') diff --git a/sdext/source/pdfimport/wrapper/wrapper.cxx b/sdext/source/pdfimport/wrapper/wrapper.cxx index b213e8eed7ca..90d471a23bb8 100644 --- a/sdext/source/pdfimport/wrapper/wrapper.cxx +++ b/sdext/source/pdfimport/wrapper/wrapper.cxx @@ -685,29 +685,20 @@ void Parser::readFont() uno::Sequence Parser::readImageImpl() { - static const char aJpegMarker[] = "JPEG"; - static const char aPbmMarker[] = "PBM"; - static const char aPpmMarker[] = "PPM"; - static const char aPngMarker[] = "PNG"; - static const char aJpegFile[] = "DUMMY.JPEG"; - static const char aPbmFile[] = "DUMMY.PBM"; - static const char aPpmFile[] = "DUMMY.PPM"; - static const char aPngFile[] = "DUMMY.PNG"; - OString aToken = readNextToken(); const sal_Int32 nImageSize( readInt32() ); OUString aFileName; - if( aToken == aPngMarker ) - aFileName = aPngFile; - else if( aToken == aJpegMarker ) - aFileName = aJpegFile; - else if( aToken == aPbmMarker ) - aFileName = aPbmFile; + if( aToken == "PNG" ) + aFileName = "DUMMY.PNG"; + else if( aToken == "JPEG" ) + aFileName = "DUMMY.JPEG"; + else if( aToken == "PBM" ) + aFileName = "DUMMY.PBM"; else { - SAL_WARN_IF(aToken != aPpmMarker,"sdext.pdfimport","Invalid bitmap format"); - aFileName = aPpmFile; + SAL_WARN_IF(aToken != "PPM","sdext.pdfimport","Invalid bitmap format"); + aFileName = "DUMMY.PPM"; } uno::Sequence aDataSequence(nImageSize); -- cgit