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

Reviewed-on: https://gerrit.libreoffice.org/33749
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 2489000d3fd66319a8355fd4e37cfdfda47296d0)

Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625
(cherry picked from commit e540ccfb5eb43bd4a3d6920074dce8436720ba8e)

1 files changed, 2 insertions, 5 deletions

diff --git a/cppcanvas/source/mtfrenderer/emfplus.cxx b/cppcanvas/source/mtfrenderer/emfplus.cxx
index 22bfee8183b2..601146730703 100644
--- a/cppcanvas/source/mtfrenderer/emfplus.cxx
+++ b/cppcanvas/source/mtfrenderer/emfplus.cxx
@@ -2482,8 +2482,8 @@ namespace cppcanvas
                         SAL_INFO("cppcanvas.emf", "EMF+\tglyphs: " << glyphsCount);
 
                         if( ( optionFlags & 1 ) && glyphsCount > 0 ) {
-                            float *charsPosX = new float[glyphsCount];
-                            float *charsPosY = new float[glyphsCount];
+                            std::unique_ptr<float[]> charsPosX( new float[glyphsCount] );
+                            std::unique_ptr<float[]> charsPosY( new float[glyphsCount] );
 
                             OUString text = read_uInt16s_ToOUString(rMF, glyphsCount);
 
@@ -2534,9 +2534,6 @@ namespace cppcanvas
 
                                 rFactoryParms.mrCurrActionIndex += pTextAction->getActionCount()-1;
                             }
-
-                            delete[] charsPosX;
-                            delete[] charsPosY;
                         } else {
                             SAL_INFO("cppcanvas.emf", "EMF+\tTODO: fonts (non-unicode glyphs chars)");
                         }