summaryrefslogtreecommitdiff
path: root/vcl/source/gdi
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-10-06 08:55:26 +0200
committerStephan Bergmann <sbergman@redhat.com>2017-10-06 10:05:00 +0200
commit4fc52078f6afa4368b2f4de3cd700e474b7a417f (patch)
treeeaf349ac004cc81df8c03d3f71a49ae8c2305356 /vcl/source/gdi
parent192a0152ff56890b406fbacfe365c452caba7fc7 (diff)
Improve performance of loplugin:commaoperator
...by avoiding calls to parentStmt, thereby also improving the precision of exactly which comma operators to ignore (which turned up a handful more finds). Also added tests. Change-Id: Ie74f824fd7f54131aab09b59086452fb4f3ff827 Reviewed-on: https://gerrit.libreoffice.org/43181 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'vcl/source/gdi')
-rw-r--r--vcl/source/gdi/svmconverter.cxx5
1 files changed, 4 insertions, 1 deletions
diff --git a/vcl/source/gdi/svmconverter.cxx b/vcl/source/gdi/svmconverter.cxx
index 5a07927216f4..57c47fd3d85c 100644
--- a/vcl/source/gdi/svmconverter.cxx
+++ b/vcl/source/gdi/svmconverter.cxx
@@ -988,7 +988,10 @@ void SVMConverter::ImplConvertFromSVM1( SvStream& rIStm, GDIMetaFile& rMtf )
pDXAry.reset(new long[nDXAryLen]);
for (sal_Int32 j = 0; j < nAryLen; ++j)
- rIStm.ReadInt32( nTmp ), pDXAry[ j ] = nTmp;
+ {
+ rIStm.ReadInt32( nTmp );
+ pDXAry[ j ] = nTmp;
+ }
// #106172# Add last DX array elem, if missing
if( nAryLen != nStrLen )