diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-11-24 11:24:50 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-11-24 15:54:48 +0100 |
commit | 832584eae514ea9cd4b3b42f1a2e9d69a17c5160 (patch) | |
tree | 98d56079c09df88b143fa3054b4797fb38fed051 /sw/source/filter/html/htmltab.cxx | |
parent | 4bae4ebd527e9deed9e1bd1be0e4bc4506d05e6f (diff) |
Clean up loplugin:unnecessaryparen blacklist
* rsc/source/parser/rscyacc.cxx no longer exists
* writerfilter/source/rtftok/rtftokenizer.cxx appears to be just fine nowadays?
* sw/source/filter/html/htmltab.cxx used redundant parentheses around a comma
operator in a while condition, and I see no reason not to remove them (the
result requires a---reasonable---tweak to loplugin:commaoperator, though)
Change-Id: I451132c700b0ae5a43b03d704156484df897ad5c
Reviewed-on: https://gerrit.libreoffice.org/45213
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'sw/source/filter/html/htmltab.cxx')
-rw-r--r-- | sw/source/filter/html/htmltab.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sw/source/filter/html/htmltab.cxx b/sw/source/filter/html/htmltab.cxx index 2a5f87f16d67..b6e1d5db18bf 100644 --- a/sw/source/filter/html/htmltab.cxx +++ b/sw/source/filter/html/htmltab.cxx @@ -1142,7 +1142,7 @@ void HTMLTable::FixRowSpan( sal_uInt16 nRow, sal_uInt16 nCol, { sal_uInt16 nRowSpan=1; HTMLTableCell *pCell; - while( ( pCell=GetCell(nRow,nCol), pCell->GetContents()==pCnts ) ) + while( pCell=GetCell(nRow,nCol), pCell->GetContents()==pCnts ) { pCell->SetRowSpan( nRowSpan ); if( m_pLayoutInfo ) @@ -1404,7 +1404,7 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox, : (nInnerFrameWidth / 2) ); // We only set the item if there's a border or a border distance // If the latter is missing, there's gonna be a border and we'll have to set the distance - aBoxItem.SetAllDistances((nBDist) ? nBDist : MIN_BORDER_DIST); + aBoxItem.SetAllDistances(nBDist ? nBDist : MIN_BORDER_DIST); pFrameFormat->SetFormatAttr( aBoxItem ); } else |