summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Kaganski <mike.kaganski@collabora.com>2018-10-21 11:58:57 +0200
committerMike Kaganski <mike.kaganski@collabora.com>2018-10-21 15:24:43 +0200
commit4c5ad24111addbf7d171b833da812fe5ba6e36b1 (patch)
treec964c05906e33d635bebabe4f806eac4b6ec49b9
parent33a8afeda6e1b6325f42790a0e1cc3c6cca8409f (diff)
tdf#120703 (PVS): redundant nullptr check
V668 There is no sense in testing the 'm_spNumberFormatterWrapper' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error. Change-Id: Id9e89bfcb412c6034ac3c06f67236a77d335c9fc Reviewed-on: https://gerrit.libreoffice.org/62127 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
-rw-r--r--chart2/source/controller/dialogs/DataBrowser.cxx3
1 files changed, 1 insertions, 2 deletions
diff --git a/chart2/source/controller/dialogs/DataBrowser.cxx b/chart2/source/controller/dialogs/DataBrowser.cxx
index b828828a546f..5e31483aa7c9 100644
--- a/chart2/source/controller/dialogs/DataBrowser.cxx
+++ b/chart2/source/controller/dialogs/DataBrowser.cxx
@@ -819,8 +819,7 @@ void DataBrowser::SetDataFromModel(
new NumberFormatterWrapper(
Reference< util::XNumberFormatsSupplier >( m_xChartDoc, uno::UNO_QUERY )));
- if( m_spNumberFormatterWrapper.get() )
- m_aNumberEditField->SetFormatter( m_spNumberFormatterWrapper->getSvNumberFormatter() );
+ m_aNumberEditField->SetFormatter( m_spNumberFormatterWrapper->getSvNumberFormatter() );
RenewTable();