From 129629de708b35154a0812eaf19dd19e76e38331 Mon Sep 17 00:00:00 2001 From: Mike Kaganski Date: Tue, 17 Nov 2020 13:19:16 +0100 Subject: Don't use new/delete here when automatic variable is enough Change-Id: Ia119da3b54fd957f3316637ddaa047cfd6a399fb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105994 Reviewed-by: Noel Grandin Reviewed-by: Mike Kaganski Tested-by: Jenkins --- vcl/source/gdi/gdimtf.cxx | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) (limited to 'vcl') diff --git a/vcl/source/gdi/gdimtf.cxx b/vcl/source/gdi/gdimtf.cxx index 8ffa24994ebb..02f47b5c6bba 100644 --- a/vcl/source/gdi/gdimtf.cxx +++ b/vcl/source/gdi/gdimtf.cxx @@ -2745,22 +2745,21 @@ SvStream& GDIMetaFile::Read( SvStream& rIStm ) SvStream& GDIMetaFile::Write( SvStream& rOStm ) { - VersionCompat* pCompat; const SvStreamCompressFlags nStmCompressMode = rOStm.GetCompressMode(); SvStreamEndian nOldFormat = rOStm.GetEndian(); rOStm.SetEndian( SvStreamEndian::LITTLE ); rOStm.WriteBytes( "VCLMTF", 6 ); - pCompat = new VersionCompat( rOStm, StreamMode::WRITE, 1 ); - - rOStm.WriteUInt32( static_cast(nStmCompressMode) ); - WriteMapMode( rOStm, m_aPrefMapMode ); - TypeSerializer aSerializer(rOStm); - aSerializer.writeSize(m_aPrefSize); - rOStm.WriteUInt32( GetActionSize() ); - - delete pCompat; + { + VersionCompat aCompat(rOStm, StreamMode::WRITE, 1); + + rOStm.WriteUInt32(static_cast(nStmCompressMode)); + WriteMapMode(rOStm, m_aPrefMapMode); + TypeSerializer aSerializer(rOStm); + aSerializer.writeSize(m_aPrefSize); + rOStm.WriteUInt32(GetActionSize()); + } // VersionCompat dtor writes stuff into the header ImplMetaWriteData aWriteData; -- cgit