diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-06-29 12:25:02 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-02 08:47:48 +0200 |
| commit | d98f1e4e0373782ad71b945dcc92c1c3d6dcf6c8 (patch) | |
| tree | a39deddf841d478d7baa5e96829891447383a597 /emfio | |
| parent | 683fa7b68673c8c6563ee493b8b5682bb6fd1b8a (diff) | |
loplugin:unusedfields improve checking for write-only fields
we trade off a little accuracy for finding more possible write-only
fields.
Change-Id: I9f7edba99481fe4ded0a9d8e45e911b0ee99d269
Reviewed-on: https://gerrit.libreoffice.org/56715
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'emfio')
| -rw-r--r-- | emfio/inc/wmfreader.hxx | 1 | ||||
| -rw-r--r-- | emfio/source/reader/wmfreader.cxx | 3 |
2 files changed, 0 insertions, 4 deletions
diff --git a/emfio/inc/wmfreader.hxx b/emfio/inc/wmfreader.hxx index 4054141f0f0c..9e1059d63760 100644 --- a/emfio/inc/wmfreader.hxx +++ b/emfio/inc/wmfreader.hxx @@ -47,7 +47,6 @@ namespace emfio sal_uInt32 mnEMFSize; sal_uInt32 mnSkipActions; - sal_uInt32 mnCurrentAction; // eventually handed over external header const WmfExternal* mpExternalHeader; diff --git a/emfio/source/reader/wmfreader.cxx b/emfio/source/reader/wmfreader.cxx index 83e7ca6e81d7..f93eb0cbc31a 100644 --- a/emfio/source/reader/wmfreader.cxx +++ b/emfio/source/reader/wmfreader.cxx @@ -1364,7 +1364,6 @@ namespace emfio sal_uInt16 nFunction; mnSkipActions = 0; - mnCurrentAction = 0; mpEMFStream.reset(); mnEMFRecCount = 0; @@ -1388,7 +1387,6 @@ namespace emfio bool bEMFAvailable = false; while( true ) { - mnCurrentAction++; mpInputStream->ReadUInt32(mnRecSize).ReadUInt16( nFunction ); if ( @@ -1854,7 +1852,6 @@ namespace emfio , mnEMFRec(0) , mnEMFSize(0) , mnSkipActions(0) - , mnCurrentAction(0) , mpExternalHeader(pExternalHeader) { } |