From 2489000d3fd66319a8355fd4e37cfdfda47296d0 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 31 Jan 2017 14:46:38 +0200 Subject: loplugin:useuniqueptr extend to check local vars just the simple and obvious case for now, of a local var being allocated and deleted inside a single local block, and the delete happening at the end of the block Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625 Reviewed-on: https://gerrit.libreoffice.org/33749 Tested-by: Jenkins Reviewed-by: Noel Grandin --- editeng/source/outliner/outlvw.cxx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'editeng') diff --git a/editeng/source/outliner/outlvw.cxx b/editeng/source/outliner/outlvw.cxx index 2c176c242a94..18c072bb7095 100644 --- a/editeng/source/outliner/outlvw.cxx +++ b/editeng/source/outliner/outlvw.cxx @@ -1075,11 +1075,10 @@ void OutlinerView::ApplyBulletsNumbering( const SvxNumberFormat* pNewFmt = aNewRule.Get(nLevel); if (pOldFmt && pNewFmt && (pOldFmt->GetFirstLineOffset() != pNewFmt->GetFirstLineOffset() || pOldFmt->GetAbsLSpace() != pNewFmt->GetAbsLSpace())) { - SvxNumberFormat* pNewFmtClone = new SvxNumberFormat(*pNewFmt); + std::unique_ptr pNewFmtClone(new SvxNumberFormat(*pNewFmt)); pNewFmtClone->SetFirstLineOffset(pOldFmt->GetFirstLineOffset()); pNewFmtClone->SetAbsLSpace(pOldFmt->GetAbsLSpace()); - aNewRule.SetLevel(nLevel, pNewFmtClone); - delete pNewFmtClone; + aNewRule.SetLevel(nLevel, pNewFmtClone.get()); } } } -- cgit v1.2.3