diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2015-10-09 12:40:36 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2015-10-09 13:50:37 +0200 |
| commit | 0cab8199f345d01bfcbf2f6a0be63b34b1dbd285 (patch) | |
| tree | 2b7240543bce7c7b1d31a09bb1a9fb832d2a31d0 /vcl | |
| parent | 3f74bbf4f53112db4ba9df014ee9da760898652d (diff) | |
clang-analyzer-deadcode.DeadStores
nSxe, nSye were unused ever since WMFReader::GetPlaceableBound was introduced
(seemingly, involving some copy/paste) in
fa149f402b9222d361e79769933bc6c58101ddef "#98505# if there is no WinExt we try
to calculate the extension instead of using (1/1) default which causes a crash
on unix and linux"
Change-Id: Ica6fb9ec1d25bd5314e73610eccef2a4df0c310f
Diffstat (limited to 'vcl')
| -rw-r--r-- | vcl/source/filter/wmf/winwmf.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/vcl/source/filter/wmf/winwmf.cxx b/vcl/source/filter/wmf/winwmf.cxx index 0566f3b1df0d..e4359eaf3da5 100644 --- a/vcl/source/filter/wmf/winwmf.cxx +++ b/vcl/source/filter/wmf/winwmf.cxx @@ -1697,7 +1697,7 @@ bool WMFReader::GetPlaceableBound( Rectangle& rPlaceableBound, SvStream* pStm ) case W_META_STRETCHDIB: { sal_Int32 nWinROP; - sal_uInt16 nSx, nSy, nSxe, nSye, nUsage; + sal_uInt16 nSx, nSy, nUsage; pStm->ReadInt32( nWinROP ); if( nFunction == W_META_STRETCHDIB ) @@ -1705,9 +1705,10 @@ bool WMFReader::GetPlaceableBound( Rectangle& rPlaceableBound, SvStream* pStm ) // nSye and nSxe is the number of pixels that has to been used if( nFunction == W_META_STRETCHDIB || nFunction == W_META_STRETCHBLT || nFunction == W_META_DIBSTRETCHBLT ) + { + sal_uInt16 nSxe, nSye; pStm->ReadUInt16( nSye ).ReadUInt16( nSxe ); - else - nSye = nSxe = 0; // set this to zero as indicator not to scale the bitmap later + } // nSy and nx is the offset of the first pixel pStm->ReadUInt16( nSy ).ReadUInt16( nSx ); |