diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2015-03-10 09:47:02 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2015-03-10 10:48:04 +0000 |
| commit | 95af5f51fa6c7a0129360bb9df687fb8660fd3c9 (patch) | |
| tree | 4ba9e086012cf97113250f74429c407cb36725db /reportdesign | |
| parent | 1bb614066738f2672da8923c40b5694eed383e86 (diff) | |
V668 no sense in testing the result of new against null
Change-Id: I85993df91bd14966f9175c356966fbd8eb1a927d
Diffstat (limited to 'reportdesign')
| -rw-r--r-- | reportdesign/source/ui/report/ViewsWindow.cxx | 17 |
1 files changed, 7 insertions, 10 deletions
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; |