From 00e8bc1945ee0727bb77a8e48f8a20edb0619068 Mon Sep 17 00:00:00 2001
From: whitertandrek
Date: Wed, 14 Mar 2018 08:43:06 +0200
Subject: PLUMBERS: Optimized heap allocation

No sense in deleting and then allocating an object again because loadStream method destroy previous data.
---
 engines/plumbers/plumbers.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'engines/plumbers/plumbers.cpp')

diff --git a/engines/plumbers/plumbers.cpp b/engines/plumbers/plumbers.cpp
index 3821170651..5491f81c65 100644
--- a/engines/plumbers/plumbers.cpp
+++ b/engines/plumbers/plumbers.cpp
@@ -90,6 +90,7 @@ static const byte cursorPalette[] = {
 Common::Error PlumbersGame::run() {
 	initGraphics(640, 480);
 	_console = new Console();
+	_image = new Image::BitmapDecoder();
 
 	CursorMan.replaceCursor(MOUSECURSOR_SCI, 11, 16, 0, 0, 0);
 	CursorMan.replaceCursorPalette(cursorPalette, 0, 3);
@@ -182,9 +183,6 @@ void PlumbersGame::loadImage(const Common::String &dirname, const Common::String
 	if (!file.open(name))
 		error("unable to load image %s", name.c_str());
 
-	delete _image;
-
-	_image = new Image::BitmapDecoder();
 	_image->loadStream(file);
 }
 
-- 
cgit v1.2.3