[Scummvm-cvs-logs] SF.net SVN: scummvm: [23881] scummvm/trunk/engines/kyra

lordhoto at users.sourceforge.net lordhoto at users.sourceforge.net
Sat Sep 16 14:12:08 CEST 2006


Revision: 23881
          http://svn.sourceforge.net/scummvm/?rev=23881&view=rev
Author:   lordhoto
Date:     2006-09-16 05:12:02 -0700 (Sat, 16 Sep 2006)

Log Message:
-----------
Replaces malloc with new in most cases.

Modified Paths:
--------------
    scummvm/trunk/engines/kyra/screen.cpp
    scummvm/trunk/engines/kyra/vqa.cpp
    scummvm/trunk/engines/kyra/vqa.h

Modified: scummvm/trunk/engines/kyra/screen.cpp
===================================================================
--- scummvm/trunk/engines/kyra/screen.cpp	2006-09-16 12:04:54 UTC (rev 23880)
+++ scummvm/trunk/engines/kyra/screen.cpp	2006-09-16 12:12:02 UTC (rev 23881)
@@ -38,19 +38,19 @@
 
 Screen::~Screen() {
 	for (int pageNum = 0; pageNum < SCREEN_PAGE_NUM; pageNum += 2) {
-		free(_pagePtrs[pageNum]);
+		delete [] _pagePtrs[pageNum];
 		_pagePtrs[pageNum] = _pagePtrs[pageNum + 1] = 0;
 	}
 	for (int f = 0; f < ARRAYSIZE(_fonts); ++f) {
 		delete[] _fonts[f].fontData;
 		_fonts[f].fontData = NULL;
 	}
-	free(_currentPalette);
-	free(_screenPalette);
-	free(_decodeShapeBuffer);
-	free(_animBlockPtr);
+	delete [] _currentPalette;
+	delete [] _screenPalette;
+	delete [] _decodeShapeBuffer;
+	delete [] _animBlockPtr;
 	for (int i = 0; i < ARRAYSIZE(_palettes); ++i) {
-		free(_palettes[i]);
+		delete [] _palettes[i];
 	}
 	delete [] _bitBlitRects;
 	for (int i = 0; i < ARRAYSIZE(_saveLoadPage); ++i) {
@@ -58,8 +58,8 @@
 		_saveLoadPage[i] = 0;
 	}
 
-	free(_unkPtr1);
-	free(_unkPtr2);
+	delete [] _unkPtr1;
+	delete [] _unkPtr2;
 	
 	delete [] _dirtyRects;
 }
@@ -79,26 +79,23 @@
 
 	_curPage = 0;
 	for (int pageNum = 0; pageNum < SCREEN_PAGE_NUM; pageNum += 2) {
-		uint8 *pagePtr = (uint8 *)malloc(SCREEN_PAGE_SIZE);
+		uint8 *pagePtr = new uint8[SCREEN_PAGE_SIZE];
 		if (pagePtr) {
 			memset(pagePtr, 0, SCREEN_PAGE_SIZE);
 			_pagePtrs[pageNum] = _pagePtrs[pageNum + 1] = pagePtr;
 		}
 	}
 	memset(_shapePages, 0, sizeof(_shapePages));
-	_currentPalette = (uint8 *)malloc(768);
-	if (_currentPalette) {
-		memset(_currentPalette, 0, 768);
-	}
-	_screenPalette = (uint8 *)malloc(768);
-	if (_screenPalette) {
-		memset(_screenPalette, 0, 768);
-	}
+	_currentPalette = new uint8[768];
+	assert(_currentPalette);
+	memset(_currentPalette, 0, 768);
+	_screenPalette = new uint8[768];
+	assert(_screenPalette);
+	memset(_screenPalette, 0, 768);
 	for (int i = 0; i < ARRAYSIZE(_palettes); ++i) {
-		_palettes[i] = (uint8 *)malloc(768);
-		if (_palettes[i]) {
-			memset(_palettes[i], 0, 768);
-		}
+		_palettes[i] = new uint8[768];
+		assert(_palettes[i]);
+		memset(_palettes[i], 0, 768);
 	}
 	setScreenPalette(_currentPalette);
 	_curDim = &_screenDimTable[0];
@@ -121,9 +118,11 @@
 	_bitBlitNum = 0;
 	memset(_saveLoadPage, 0, sizeof(_saveLoadPage));
 
-	_unkPtr1 = (uint8*)malloc(getRectSize(1, 144));
+	_unkPtr1 = new uint8[getRectSize(1, 144)];
+	assert(_unkPtr1);
 	memset(_unkPtr1, 0, getRectSize(1, 144));
-	_unkPtr2 = (uint8*)malloc(getRectSize(1, 144));
+	_unkPtr2 = new uint8[getRectSize(1, 144)];
+	assert(_unkPtr2);
 	memset(_unkPtr2, 0, getRectSize(1, 144));
 	
 	_forceFullUpdate = false;
@@ -647,8 +646,8 @@
 
 void Screen::setAnimBlockPtr(int size) {
 	debugC(9, kDebugLevelScreen, "Screen::setAnimBlockPtr(%d)", size);
-	free(_animBlockPtr);
-	_animBlockPtr = (uint8 *)malloc(size);
+	delete [] _animBlockPtr;
+	_animBlockPtr = new uint8[size];
 	assert(_animBlockPtr);
 	memset(_animBlockPtr, 0, size);
 	_animBlockSize = size;
@@ -957,8 +956,8 @@
 	
 	int shapeSize = shapeWidth * shapeHeight;
 	if (_decodeShapeBufferSize < shapeSize) {
-		free(_decodeShapeBuffer);
-		_decodeShapeBuffer = (uint8 *)malloc(shapeSize);
+		delete [] _decodeShapeBuffer;
+		_decodeShapeBuffer = new uint8[shapeSize];
 		_decodeShapeBufferSize = shapeSize;
 	}
 	if (!_decodeShapeBuffer) {
@@ -1962,7 +1961,7 @@
 	if ((_vm->game() == GI_KYRA1) && (_vm->features() & GF_TALKIE))
 		shape -= 2;
 
-	uint8 *cursor = (uint8 *)malloc(mouseHeight * mouseWidth);
+	uint8 *cursor = new uint8[mouseHeight * mouseWidth];
 	fillRect(0, 0, mouseWidth, mouseHeight, 0, 8);
 	drawShape(8, shape, 0, 0, 0, 0);
 
@@ -1970,7 +1969,7 @@
 	copyRegionToBuffer(8, 0, 0, mouseWidth, mouseHeight, cursor);
 	CursorMan.replaceCursor(cursor, mouseWidth, mouseHeight, x, y, 0);
 	CursorMan.showMouse(true);
-	free(cursor);
+	delete [] cursor;
 
 	// makes sure that the cursor is drawn
 	// we do not use Screen::updateScreen here

Modified: scummvm/trunk/engines/kyra/vqa.cpp
===================================================================
--- scummvm/trunk/engines/kyra/vqa.cpp	2006-09-16 12:04:54 UTC (rev 23880)
+++ scummvm/trunk/engines/kyra/vqa.cpp	2006-09-16 12:12:02 UTC (rev 23881)
@@ -51,7 +51,7 @@
 
 void VQAMovie::initBuffers() {
 	for (int i = 0; i < ARRAYSIZE(_buffers); i++) {
-		_buffers[i].data = NULL;
+		_buffers[i].data = 0;
 		_buffers[i].size = 0;
 	}
 }
@@ -65,8 +65,8 @@
 		 * We could use realloc() here, but we don't actually need the
 		 * old contents of the buffer.
 		 */
-		free(_buffers[num].data);
-		_buffers[num].data = malloc(size);
+		delete [] _buffers[num].data;
+		_buffers[num].data = new uint8[size];
 		_buffers[num].size = size;
 	}
 
@@ -77,7 +77,7 @@
 
 void VQAMovie::freeBuffers() {
 	for (int i = 0; i < ARRAYSIZE(_buffers); i++) {
-		free(_buffers[i].data);
+		delete [] _buffers[i].data;
 		_buffers[i].data = NULL;
 		_buffers[i].size = 0;
 	}

Modified: scummvm/trunk/engines/kyra/vqa.h
===================================================================
--- scummvm/trunk/engines/kyra/vqa.h	2006-09-16 12:04:54 UTC (rev 23880)
+++ scummvm/trunk/engines/kyra/vqa.h	2006-09-16 12:12:02 UTC (rev 23881)
@@ -81,7 +81,7 @@
 	};
 
 	struct Buffer {
-		void *data;
+		uint8 *data;
 		uint32 size;
 	};
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list