diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-05-03 09:29:37 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-05-03 14:03:54 +0200 |
| commit | 7cc1d3437a48140382773bf41401a46a3ced4706 (patch) | |
| tree | 5e038178bb0b2699bb9314bd02fb7e197597ecb2 /sc | |
| parent | b912eafa33227a5622c5a4310948cfa07c984726 (diff) | |
loplugin:unnecessaryparen small improvement
when calling a function, and passing only one arg, but the
function has defaulted args, we were ignoring this case.
Change-Id: I86517f18e30531127664088ddc09ef96dbd8bdf5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115033
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/filter/excel/excrecds.cxx | 4 | ||||
| -rw-r--r-- | sc/source/filter/xcl97/xcl97rec.cxx | 2 | ||||
| -rw-r--r-- | sc/source/ui/Accessibility/AccessiblePreviewHeaderCell.cxx | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/sc/source/filter/excel/excrecds.cxx b/sc/source/filter/excel/excrecds.cxx index 7a215c8d1f41..b73c695391bd 100644 --- a/sc/source/filter/excel/excrecds.cxx +++ b/sc/source/filter/excel/excrecds.cxx @@ -590,7 +590,7 @@ static OString lcl_GetValue( sal_uInt8 nType, double fVal, const XclExpString* p { case EXC_AFTYPE_STRING: return XclXmlUtils::ToOString( *pStr ); case EXC_AFTYPE_DOUBLE: return OString::number( fVal ); - case EXC_AFTYPE_BOOLERR: return OString::number( ( fVal != 0 ? 1 : 0 ) ); + case EXC_AFTYPE_BOOLERR: return OString::number( fVal != 0 ? 1 : 0 ); default: return OString(); } } @@ -825,7 +825,7 @@ void XclExpAutofilter::SaveXml( XclExpXmlStream& rStrm ) rWorksheet->singleElement( XML_top10, XML_top, ToPsz( get_flag( nFlags, EXC_AFFLAG_TOP10TOP ) ), XML_percent, ToPsz( get_flag( nFlags, EXC_AFFLAG_TOP10PERC ) ), - XML_val, OString::number((nFlags >> 7)) + XML_val, OString::number(nFlags >> 7) // OOXTODO: XML_filterVal ); } diff --git a/sc/source/filter/xcl97/xcl97rec.cxx b/sc/source/filter/xcl97/xcl97rec.cxx index 485bdc38b784..de7a3762ac8b 100644 --- a/sc/source/filter/xcl97/xcl97rec.cxx +++ b/sc/source/filter/xcl97/xcl97rec.cxx @@ -1384,7 +1384,7 @@ void ExcBundlesheet8::SaveXml( XclExpXmlStream& rStrm ) rStrm.GetCurrentStream()->singleElement( XML_sheet, XML_name, sUnicodeName.toUtf8(), - XML_sheetId, OString::number( ( nTab+1 ) ), + XML_sheetId, OString::number( nTab+1 ), XML_state, nGrbit == 0x0000 ? "visible" : "hidden", FSNS( XML_r, XML_id ), sId.toUtf8() ); } diff --git a/sc/source/ui/Accessibility/AccessiblePreviewHeaderCell.cxx b/sc/source/ui/Accessibility/AccessiblePreviewHeaderCell.cxx index 4e57e1c2a42a..8a009fada4db 100644 --- a/sc/source/ui/Accessibility/AccessiblePreviewHeaderCell.cxx +++ b/sc/source/ui/Accessibility/AccessiblePreviewHeaderCell.cxx @@ -357,7 +357,7 @@ OUString ScAccessiblePreviewHeaderCell::createAccessibleName() else { // name of row header - sName += OUString::number( ( maCellPos.Row() + 1 ) ); + sName += OUString::number( maCellPos.Row() + 1 ); } return sName; |