diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-13 14:56:54 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-13 21:14:26 +0200 |
| commit | b6491f09e9dc3e41d571abce3c4f16467cb60c9c (patch) | |
| tree | dfe1b5ad2273be6f5423a2b1994e1855811d3f78 /reportdesign | |
| parent | 5e1719ff1ed5b4350684c3ed67b5375bb49e0223 (diff) | |
loplugin:sequentialassign in oox..reportdesign
Change-Id: I59ef0a6da411b8af8bdf8d8efb1d733db7475d9c
Reviewed-on: https://gerrit.libreoffice.org/70707
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'reportdesign')
| -rw-r--r-- | reportdesign/source/ui/report/ReportController.cxx | 3 | ||||
| -rw-r--r-- | reportdesign/source/ui/report/ReportSection.cxx | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/reportdesign/source/ui/report/ReportController.cxx b/reportdesign/source/ui/report/ReportController.cxx index b78f70d7681c..d4c823561aa4 100644 --- a/reportdesign/source/ui/report/ReportController.cxx +++ b/reportdesign/source/ui/report/ReportController.cxx @@ -1647,8 +1647,7 @@ void OReportController::impl_initialize( ) m_xFormatter->attachNumberFormatsSupplier(Reference< XNumberFormatsSupplier>(m_xReportDefinition,uno::UNO_QUERY)); utl::MediaDescriptor aDescriptor( m_xReportDefinition->getArgs() ); - OUString sHierarchicalDocumentName; - sHierarchicalDocumentName = aDescriptor.getUnpackedValueOrDefault("HierarchicalDocumentName",sHierarchicalDocumentName); + OUString sHierarchicalDocumentName = aDescriptor.getUnpackedValueOrDefault("HierarchicalDocumentName",OUString()); if ( sHierarchicalDocumentName.isEmpty() && getConnection().is() ) { diff --git a/reportdesign/source/ui/report/ReportSection.cxx b/reportdesign/source/ui/report/ReportSection.cxx index 9e49938e5484..43814411a25f 100644 --- a/reportdesign/source/ui/report/ReportSection.cxx +++ b/reportdesign/source/ui/report/ReportSection.cxx @@ -690,8 +690,7 @@ sal_Int8 OReportSection::AcceptDrop( const AcceptDropEvent& _rEvt ) ) { if (!m_pParent) return DND_ACTION_NONE; - sal_uInt16 nCurrentPosition = 0; - nCurrentPosition = m_pParent->getViewsWindow()->getPosition(m_pParent); + sal_uInt16 nCurrentPosition = m_pParent->getViewsWindow()->getPosition(m_pParent); if (_rEvt.mnAction == DND_ACTION_COPY ) { // we must assure, we can't drop in the top section |