loplugin:simplifyconstruct in reportdesign..sax

Change-Id: I7d2a754cdc5576b5a5b35db2fbffd19ea17c16ff
Reviewed-on: https://gerrit.libreoffice.org/60224
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
Noel Grandin
2018-09-09 19:41:49 +02:00
parent 926e4e469d
commit 4b3430f27a
10 changed files with 3 additions and 14 deletions

View File

@@ -153,8 +153,6 @@ OAddFieldWindow::OAddFieldWindow(vcl::Window* pParent ,const uno::Reference< bea
, m_pListBox(VclPtr<OAddFieldWindowListBox>::Create(get<vcl::Window>("box"), this)) , m_pListBox(VclPtr<OAddFieldWindowListBox>::Create(get<vcl::Window>("box"), this))
, m_nCommandType(0) , m_nCommandType(0)
, m_bEscapeProcessing(false) , m_bEscapeProcessing(false)
, m_pChangeListener(nullptr)
, m_pContainerListener(nullptr)
{ {
get(m_aActions, "toolbox"); get(m_aActions, "toolbox");
m_nSortUpId = m_aActions->GetItemId(0); m_nSortUpId = m_aActions->GetItemId(0);

View File

@@ -108,8 +108,6 @@ Condition::Condition( vcl::Window* _pParent, IConditionalFormatAction& _rAction,
, m_aColorWrapper(this) , m_aColorWrapper(this)
, m_rController(_rController) , m_rController(_rController)
, m_rAction(_rAction) , m_rAction(_rAction)
, m_pBtnUpdaterFontColor(nullptr)
, m_pBtnUpdaterBackgroundColor(nullptr)
, m_nCondIndex(0) , m_nCondIndex(0)
, m_bInDestruction(false) , m_bInDestruction(false)
{ {

View File

@@ -803,7 +803,6 @@ OGroupsSortingDialog::OGroupsSortingDialog(vcl::Window* _pParent, bool _bReadOnl
: FloatingWindow(_pParent, "FloatingSort", "modules/dbreport/ui/floatingsort.ui") : FloatingWindow(_pParent, "FloatingSort", "modules/dbreport/ui/floatingsort.ui")
, OPropertyChangeListener(m_aMutex) , OPropertyChangeListener(m_aMutex)
, m_pController(_pController) , m_pController(_pController)
, m_pCurrentGroupListener(nullptr)
, m_xGroups(m_pController->getReportDefinition()->getGroups()) , m_xGroups(m_pController->getReportDefinition()->getGroups())
, m_bReadOnly(_bReadOnly) , m_bReadOnly(_bReadOnly)
{ {

View File

@@ -176,7 +176,7 @@ void adjustSectionName(const uno::Reference< report::XGroup >& _xGroup,sal_Int32
::rtl::Reference< comphelper::OPropertyChangeMultiplexer> addStyleListener(const uno::Reference< report::XReportDefinition >& _xReportDefinition,::comphelper::OPropertyChangeListener* _pListener) ::rtl::Reference< comphelper::OPropertyChangeMultiplexer> addStyleListener(const uno::Reference< report::XReportDefinition >& _xReportDefinition,::comphelper::OPropertyChangeListener* _pListener)
{ {
::rtl::Reference< comphelper::OPropertyChangeMultiplexer> pRet = nullptr; ::rtl::Reference< comphelper::OPropertyChangeMultiplexer> pRet;
if ( _xReportDefinition.is() ) if ( _xReportDefinition.is() )
{ {
uno::Reference<beans::XPropertySet> xPageStyle(getUsedStyle(_xReportDefinition),uno::UNO_QUERY); uno::Reference<beans::XPropertySet> xPageStyle(getUsedStyle(_xReportDefinition),uno::UNO_QUERY);

View File

@@ -82,8 +82,6 @@ OReportSection::OReportSection(OSectionWindow* _pParent,const uno::Reference< re
, m_pPage(nullptr) , m_pPage(nullptr)
, m_pView(nullptr) , m_pView(nullptr)
, m_pParent(_pParent) , m_pParent(_pParent)
, m_pMulti(nullptr)
, m_pReportListener(nullptr)
, m_xSection(_xSection) , m_xSection(_xSection)
, m_nPaintEntranceCount(0) , m_nPaintEntranceCount(0)
, m_eMode(DlgEdMode::Select) , m_eMode(DlgEdMode::Select)

View File

@@ -50,7 +50,6 @@ OScrollWindowHelper::OScrollWindowHelper( ODesignView* _pDesignView)
,m_aCornerWin( VclPtr<ScrollBarBox>::Create(this) ) ,m_aCornerWin( VclPtr<ScrollBarBox>::Create(this) )
,m_pParent(_pDesignView) ,m_pParent(_pDesignView)
,m_aReportWindow(VclPtr<rptui::OReportWindow>::Create(this,m_pParent)) ,m_aReportWindow(VclPtr<rptui::OReportWindow>::Create(this,m_pParent))
,m_pReportDefinitionMultiPlexer(nullptr)
{ {
SetMapMode( MapMode( MapUnit::Map100thMM ) ); SetMapMode( MapMode( MapUnit::Map100thMM ) );

View File

@@ -113,7 +113,6 @@ void DlgEdFunc::ForceScroll( const Point& rPos )
DlgEdFunc::DlgEdFunc( OReportSection* _pParent ) DlgEdFunc::DlgEdFunc( OReportSection* _pParent )
: m_pParent(_pParent) : m_pParent(_pParent)
, m_rView(_pParent->getSectionView()) , m_rView(_pParent->getSectionView())
, m_xOverlappingObj(nullptr)
, m_pOverlappingObj(nullptr) , m_pOverlappingObj(nullptr)
, m_nOverlappedControlColor(0) , m_nOverlappedControlColor(0)
, m_nOldColor(0) , m_nOldColor(0)

View File

@@ -84,7 +84,7 @@ class TestReferenceRefCounting : public CppUnit::TestFixture
static_cast<long>(2), test2->use_count()); static_cast<long>(2), test2->use_count());
// use count should decrement // use count should decrement
test2 = rtl::Reference< MoveTestClass >(nullptr); test2 = rtl::Reference< MoveTestClass >();
CPPUNIT_ASSERT_EQUAL_MESSAGE("test1.use_count() == 1", CPPUNIT_ASSERT_EQUAL_MESSAGE("test1.use_count() == 1",
static_cast<long>(1), test1->use_count()); static_cast<long>(1), test1->use_count());

View File

@@ -939,8 +939,7 @@ class SAXWriter :
{ {
public: public:
SAXWriter() SAXWriter()
: m_pSaxWriterHelper(nullptr) : m_bDocStarted(false)
, m_bDocStarted(false)
, m_bIsCDATA(false) , m_bIsCDATA(false)
, m_bForceLineBreak(false) , m_bForceLineBreak(false)
, m_bAllowLineBreak(false) , m_bAllowLineBreak(false)

View File

@@ -47,7 +47,6 @@ public:
, mpCache(mnMaximumSize) , mpCache(mnMaximumSize)
, pSeq(mpCache.get()) , pSeq(mpCache.get())
, mbWriteToOutStream(true) , mbWriteToOutStream(true)
, mpForMerge(nullptr)
{} {}
const css::uno::Reference< css::io::XOutputStream >& getOutputStream() const const css::uno::Reference< css::io::XOutputStream >& getOutputStream() const