diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-04-17 11:30:06 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-04-17 12:34:52 +0200 |
commit | 3cb8e9e211c30089516f56f465176d3a959631f9 (patch) | |
tree | 187dee1288b0838e3acb63eef02fae552fdd6d56 /vcl/source/filter | |
parent | 434d92a51409b0847ee7d313b50fc547b7615bff (diff) |
loplugin:buriedassign in vcl
Change-Id: I529052c1ba7591d91d3848080af8b06e077b6f14
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92409
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'vcl/source/filter')
-rw-r--r-- | vcl/source/filter/ixbm/xbmread.cxx | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/vcl/source/filter/ixbm/xbmread.cxx b/vcl/source/filter/ixbm/xbmread.cxx index 9b16c901b762..76d41bf22d6d 100644 --- a/vcl/source/filter/ixbm/xbmread.cxx +++ b/vcl/source/filter/ixbm/xbmread.cxx @@ -141,8 +141,8 @@ OString XBMReader::FindTokenLine( SvStream* pInStm, const char* pTok1, { bStatus = false; - if( ( ( nPos2 = aRet.indexOf( pTok2 ) ) != -1 ) && - ( nPos2 > nPos1 ) ) + nPos2 = aRet.indexOf( pTok2 ); + if( ( nPos2 != -1 ) && ( nPos2 > nPos1 ) ) { bStatus = true; } @@ -208,7 +208,9 @@ void XBMReader::ParseData( SvStream* pInStm, const OString& aLastLine, XBMFormat sal_Int32 nPos; // delete opening curly bracket - if( (nPos = ( aLine = aLastLine ).indexOf('{') ) != -1 ) + aLine = aLastLine; + nPos = aLine.indexOf('{'); + if( nPos != -1 ) aLine = aLine.copy(nPos + 1); bFirstLine = false; |