diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-21 11:58:57 +0200 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-21 15:24:43 +0200 |
| commit | 4c5ad24111addbf7d171b833da812fe5ba6e36b1 (patch) | |
| tree | c964c05906e33d635bebabe4f806eac4b6ec49b9 /chart2 | |
| parent | 33a8afeda6e1b6325f42790a0e1cc3c6cca8409f (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>
Diffstat (limited to 'chart2')
| -rw-r--r-- | chart2/source/controller/dialogs/DataBrowser.cxx | 3 |
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(); |