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 <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 2 insertions, 5 deletions

diff --git a/cppcanvas/source/mtfrenderer/emfplus.cxx b/cppcanvas/source/mtfrenderer/emfplus.cxx
index 11dbf62f8872..bb8f0a84b8fb 100644
--- a/cppcanvas/source/mtfrenderer/emfplus.cxx
+++ b/cppcanvas/source/mtfrenderer/emfplus.cxx
@@ -1383,8 +1383,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);
 
@@ -1435,9 +1435,6 @@ namespace cppcanvas
 
                                 rFactoryParms.mrCurrActionIndex += pTextAction->getActionCount()-1;
                             }
-
-                            delete[] charsPosX;
-                            delete[] charsPosY;
                         } else {
                             SAL_INFO("cppcanvas.emf", "EMF+\tTODO: fonts (non-unicode glyphs chars)");
                         }