From 1f42f440927744a6acb57bf220560606078b3991 Mon Sep 17 00:00:00 2001 From: smatz Date: Tue, 24 Feb 2009 21:32:23 +0000 Subject: (svn r15569) -Codechange: replace MallocT + memset( 0 ) calls by CallocT --- src/oldpool.cpp | 5 +---- src/screenshot.cpp | 11 ++++------- src/vehicle.cpp | 5 +---- 3 files changed, 6 insertions(+), 15 deletions(-) (limited to 'src') diff --git a/src/oldpool.cpp b/src/oldpool.cpp index 50b1635f6..7cec9d597 100644 --- a/src/oldpool.cpp +++ b/src/oldpool.cpp @@ -55,10 +55,7 @@ bool OldMemoryPoolBase::AddBlockToPool() this->blocks = ReallocT(this->blocks, this->current_blocks + 1); /* Allocate memory to the new block item */ - this->blocks[this->current_blocks] = MallocT(this->item_size * (1 << this->block_size_bits)); - - /* Clean the content of the new block */ - memset(this->blocks[this->current_blocks], 0, this->item_size * (1 << this->block_size_bits)); + this->blocks[this->current_blocks] = CallocT(this->item_size * (1 << this->block_size_bits)); /* Call a custom function if defined (e.g. to fill indexes) */ if (this->new_block_proc != NULL) this->new_block_proc(this->current_blocks * (1 << this->block_size_bits)); diff --git a/src/screenshot.cpp b/src/screenshot.cpp index 960c08416..8d9d3b61d 100644 --- a/src/screenshot.cpp +++ b/src/screenshot.cpp @@ -126,8 +126,7 @@ static bool MakeBmpImage(const char *name, ScreenshotCallback *callb, void *user maxlines = Clamp(65536 / padw, 16, 128); /* now generate the bitmap bits */ - void *buff = MallocT(padw * maxlines * bpp); // by default generate 128 lines at a time. - memset(buff, 0, padw * maxlines); // zero the buffer to have the padding bytes set to 0 + uint8 *buff = CallocT(padw * maxlines * bpp); // by default generate 128 lines at a time. /* start at the bottom, since bitmaps are stored bottom up. */ do { @@ -140,7 +139,7 @@ static bool MakeBmpImage(const char *name, ScreenshotCallback *callb, void *user /* write each line */ while (n) - if (fwrite((uint8 *)buff + (--n) * padw * bpp, padw * bpp, 1, f) != 1) { + if (fwrite(buff + (--n) * padw * bpp, padw * bpp, 1, f) != 1) { free(buff); fclose(f); return false; @@ -250,8 +249,7 @@ static bool MakePNGImage(const char *name, ScreenshotCallback *callb, void *user maxlines = Clamp(65536 / w, 16, 128); /* now generate the bitmap bits */ - void *buff = MallocT(w * maxlines * bpp); // by default generate 128 lines at a time. - memset(buff, 0, w * maxlines * bpp); + void *buff = CallocT(w * maxlines * bpp); // by default generate 128 lines at a time. y = 0; do { @@ -345,8 +343,7 @@ static bool MakePCXImage(const char *name, ScreenshotCallback *callb, void *user maxlines = Clamp(65536 / w, 16, 128); /* now generate the bitmap bits */ - uint8 *buff = MallocT(w * maxlines); // by default generate 128 lines at a time. - memset(buff, 0, w * maxlines); // zero the buffer to have the padding bytes set to 0 + uint8 *buff = CallocT(w * maxlines); // by default generate 128 lines at a time. y = 0; do { diff --git a/src/vehicle.cpp b/src/vehicle.cpp index 0ca03f3b6..471f5b8c9 100644 --- a/src/vehicle.cpp +++ b/src/vehicle.cpp @@ -37,7 +37,6 @@ #include "ai/ai.hpp" #include "core/smallmap_type.hpp" #include "vehiclelist.h" -#include "core/mem_func.hpp" #include "depot_func.h" #include "settings_type.h" @@ -1746,9 +1745,7 @@ FreeUnitIDGenerator::FreeUnitIDGenerator(VehicleType type, CompanyID owner) : ca this->maxid++; // so there is space for last item (with v->unitnumber == maxid) this->maxid++; // this one will always be free (well, it will fail when there are 65535 units, so this overflows) - this->cache = MallocT(this->maxid); - - MemSetT(this->cache, 0, this->maxid); + this->cache = CallocT(this->maxid); /* Fill the cache */ FOR_ALL_VEHICLES(v) { -- cgit v1.2.3-54-g00ecf