From 95af5f51fa6c7a0129360bb9df687fb8660fd3c9 Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Tue, 10 Mar 2015 09:47:02 +0000 Subject: V668 no sense in testing the result of new against null Change-Id: I85993df91bd14966f9175c356966fbd8eb1a927d --- reportdesign/source/ui/report/ViewsWindow.cxx | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) (limited to 'reportdesign') diff --git a/reportdesign/source/ui/report/ViewsWindow.cxx b/reportdesign/source/ui/report/ViewsWindow.cxx index c849571e2c10..730b37258717 100644 --- a/reportdesign/source/ui/report/ViewsWindow.cxx +++ b/reportdesign/source/ui/report/ViewsWindow.cxx @@ -1011,18 +1011,15 @@ void OViewsWindow::BegDragObj_createInvisibleObjectAtPosition(const Rectangle& _ if ( &rView != &_rSection ) { SdrObject *pNewObj = new SdrUnoObj(OUString("com.sun.star.form.component.FixedText")); - if (pNewObj) - { - pNewObj->SetLogicRect(_aRect); + pNewObj->SetLogicRect(_aRect); - pNewObj->Move(Size(0, aNewPos.Y())); - bool bChanged = rView.GetModel()->IsChanged(); - rReportSection.getPage()->InsertObject(pNewObj); - rView.GetModel()->SetChanged(bChanged); - m_aBegDragTempList.push_back(pNewObj); + pNewObj->Move(Size(0, aNewPos.Y())); + bool bChanged = rView.GetModel()->IsChanged(); + rReportSection.getPage()->InsertObject(pNewObj); + rView.GetModel()->SetChanged(bChanged); + m_aBegDragTempList.push_back(pNewObj); - rView.MarkObj( pNewObj, rView.GetSdrPageView() ); - } + rView.MarkObj( pNewObj, rView.GetSdrPageView() ); } const long nSectionHeight = rReportSection.PixelToLogic(rReportSection.GetOutputSizePixel()).Height(); aNewPos.Y() -= nSectionHeight; -- cgit