loplugin:useuniqueptr in SwSdrUndo
Change-Id: I848f889f34dd7d3ffd83ff0cf209de8e69bbfeae Reviewed-on: https://gerrit.libreoffice.org/59447 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
@@ -40,7 +40,7 @@ class SwSdrUndo : public SwUndo
|
||||
std::unique_ptr<SdrMarkList> pMarkList; // MarkList for all selected SdrObjects
|
||||
|
||||
public:
|
||||
SwSdrUndo( SdrUndoAction* , const SdrMarkList* pMarkList, const SwDoc* pDoc );
|
||||
SwSdrUndo( std::unique_ptr<SdrUndoAction> , const SdrMarkList* pMarkList, const SwDoc* pDoc );
|
||||
|
||||
virtual ~SwSdrUndo() override;
|
||||
|
||||
|
||||
@@ -65,14 +65,14 @@ IMPL_LINK( SwDoc, AddDrawUndo, SdrUndoAction *, pUndo, void )
|
||||
if( pSh && pSh->HasDrawView() )
|
||||
pMarkList = &pSh->GetDrawView()->GetMarkedObjectList();
|
||||
|
||||
GetIDocumentUndoRedo().AppendUndo( new SwSdrUndo(pUndo, pMarkList, this) );
|
||||
GetIDocumentUndoRedo().AppendUndo( new SwSdrUndo(std::unique_ptr<SdrUndoAction>(pUndo), pMarkList, this) );
|
||||
}
|
||||
else
|
||||
delete pUndo;
|
||||
}
|
||||
|
||||
SwSdrUndo::SwSdrUndo( SdrUndoAction* pUndo, const SdrMarkList* pMrkLst, const SwDoc* pDoc )
|
||||
: SwUndo( SwUndoId::DRAWUNDO, pDoc ), pSdrUndo( pUndo )
|
||||
SwSdrUndo::SwSdrUndo( std::unique_ptr<SdrUndoAction> pUndo, const SdrMarkList* pMrkLst, const SwDoc* pDoc )
|
||||
: SwUndo( SwUndoId::DRAWUNDO, pDoc ), pSdrUndo( std::move(pUndo) )
|
||||
{
|
||||
if( pMrkLst && pMrkLst->GetMarkCount() )
|
||||
pMarkList.reset( new SdrMarkList( *pMrkLst ) );
|
||||
|
||||
Reference in New Issue
Block a user