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 /xmloff | |
| 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 'xmloff')
| -rw-r--r-- | xmloff/source/chart/SchXMLExport.cxx | 4 | ||||
| -rw-r--r-- | xmloff/source/style/impastpl.cxx | 14 |
2 files changed, 9 insertions, 9 deletions
diff --git a/xmloff/source/chart/SchXMLExport.cxx b/xmloff/source/chart/SchXMLExport.cxx index 87bfb050164a..af5885d3e354 100644 --- a/xmloff/source/chart/SchXMLExport.cxx +++ b/xmloff/source/chart/SchXMLExport.cxx @@ -3595,7 +3595,7 @@ void SchXMLExportHelper_Impl::exportDataPoints( if( aLastPoint.mnRepeat > 1 ) mrExport.AddAttribute( XML_NAMESPACE_CHART, XML_REPEATED, - OUString::number( ( aLastPoint.mnRepeat ) )); + OUString::number( aLastPoint.mnRepeat )); for (auto& deletedLegendEntry : deletedLegendEntriesSeq) { @@ -3621,7 +3621,7 @@ void SchXMLExportHelper_Impl::exportDataPoints( if( aLastPoint.mnRepeat > 1 ) mrExport.AddAttribute( XML_NAMESPACE_CHART, XML_REPEATED, - OUString::number( ( aLastPoint.mnRepeat ) )); + OUString::number( aLastPoint.mnRepeat )); for (auto& deletedLegendEntry : deletedLegendEntriesSeq) { diff --git a/xmloff/source/style/impastpl.cxx b/xmloff/source/style/impastpl.cxx index 34dc16a32cc0..0a3e2e97595f 100644 --- a/xmloff/source/style/impastpl.cxx +++ b/xmloff/source/style/impastpl.cxx @@ -103,15 +103,15 @@ data2string(void *data, case typelib_TypeClass_BOOLEAN: return *static_cast<const sal_Bool*>(data) ? OUString("true") : OUString("false"); case typelib_TypeClass_BYTE: - return OUString::number((*static_cast<const sal_Int8*>(data))); + return OUString::number(*static_cast<const sal_Int8*>(data)); case typelib_TypeClass_SHORT: - return OUString::number((*static_cast<const sal_Int16*>(data))); + return OUString::number(*static_cast<const sal_Int16*>(data)); case typelib_TypeClass_LONG: - return OUString::number((*static_cast<const sal_Int32*>(data))); + return OUString::number(*static_cast<const sal_Int32*>(data)); case typelib_TypeClass_HYPER: - return OUString::number((*static_cast<const sal_Int64*>(data))); + return OUString::number(*static_cast<const sal_Int64*>(data)); case typelib_TypeClass_UNSIGNED_SHORT: - return OUString::number((*static_cast<const sal_uInt16*>(data))); + return OUString::number(*static_cast<const sal_uInt16*>(data)); case typelib_TypeClass_UNSIGNED_LONG: return OUString::number((*static_cast<const sal_uInt32*>(data)), 16); case typelib_TypeClass_UNSIGNED_HYPER: @@ -121,7 +121,7 @@ data2string(void *data, case typelib_TypeClass_DOUBLE: return OUString::number(*static_cast<const double*>(data)); case typelib_TypeClass_CHAR: - return ("U+" + OUString::number((*static_cast<const sal_uInt16*>(data)))); + return ("U+" + OUString::number(*static_cast<const sal_uInt16*>(data))); case typelib_TypeClass_STRING: return *static_cast<OUString*>(data); case typelib_TypeClass_TYPE: @@ -132,7 +132,7 @@ data2string(void *data, case typelib_TypeClass_STRUCT: return struct2string(data, type->pType); case typelib_TypeClass_ENUM: - return OUString::number((*static_cast<const sal_Int32*>(data))); + return OUString::number(*static_cast<const sal_Int32*>(data)); default: assert(false); // this cannot happen I hope break; |