diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-11-16 14:06:06 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-11-16 15:05:58 +0100 |
| commit | c8907b19dc04a57b74cfc14dc641ac41177e2142 (patch) | |
| tree | abd7d18e2faa6f995144d40a72e06a04eae034ac /sw/source/filter/ww8/ww8par5.cxx | |
| parent | 11300c48d4350c2f6748a1ca04b919cfae338d41 (diff) | |
loplugin:buriedassign in sw
Change-Id: If2adf22a0ac3e030fca1b4ecd0173cac58f0f21e
Reviewed-on: https://gerrit.libreoffice.org/63470
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw/source/filter/ww8/ww8par5.cxx')
| -rw-r--r-- | sw/source/filter/ww8/ww8par5.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sw/source/filter/ww8/ww8par5.cxx b/sw/source/filter/ww8/ww8par5.cxx index 49c506b3d5d6..c77292333300 100644 --- a/sw/source/filter/ww8/ww8par5.cxx +++ b/sw/source/filter/ww8/ww8par5.cxx @@ -1047,7 +1047,8 @@ void SwWW8ImplReader::MakeTagString( OUString& rStr, const OUString& rOrg ) nI < rStr.getLength() && rStr.getLength() < (MAX_FIELDLEN - 4); ++nI ) { bool bSetAsHex = false; - switch( cChar = rStr[ nI ] ) + cChar = rStr[ nI ]; + switch( cChar ) { case 132: // Exchange typographical quotation marks for normal ones case 148: @@ -1467,7 +1468,7 @@ eF_ResT SwWW8ImplReader::Read_F_Seq( WW8FieldDesc*, OUString& rStr ) aField.SetSubType(aField.GetSubType() | nsSwExtendedSubType::SUB_INVISIBLE); if (!sStart.isEmpty()) - aField.SetFormula( ( aSequenceName += "=" ) += sStart ); + aField.SetFormula( aSequenceName + "=" + sStart ); else if (!bCountOn) aField.SetFormula(aSequenceName); @@ -3513,7 +3514,7 @@ eF_ResT SwWW8ImplReader::Read_F_Hyperlink( WW8FieldDesc* /*pF*/, OUString& rStr OSL_ENSURE(!sURL.isEmpty() || !sMark.isEmpty(), "WW8: Empty URL"); if( !sMark.isEmpty() ) - ( sURL += "#" ) += sMark; + sURL = sURL + "#" + sMark; SwFormatINetFormat aURL(sURL, sTarget); // If on loading TOC field, change the default style into the "index link" |