cpp/openttd-patchpack/source Changeset - r11202:ec3713bd39fb

Changeset - r11202:ec3713bd39fb

Parent rev.

Child rev.

[Not reviewed]

master

0 5 0

rubidium - 15 years ago 2009-02-23 10:50:25
rubidium@openttd.org

(svn r15555) -Codechange: remove the mallocs + frees for temporary data from loading sprites.

5 files changed with 39 insertions and 13 deletions:

src/fontcache.cpp

src/spritecache.cpp

src/spriteloader/grf.cpp

src/spriteloader/png.cpp

src/spriteloader/spriteloader.hpp

0 comments (0 inline, 0 general)

src/fontcache.cpp

➞

Show inline comments

@@ @@ -710,7 +710,7 @@ const Sprite *GetGlyph(FontSize size, WC @@
 	height = max(1, slot->bitmap.rows  + (size == FS_NORMAL));
 	/* FreeType has rendered the glyph, now we allocate a sprite and copy the image into it */
-	sprite.data = CallocT<SpriteLoader::CommonPixel>(width * height);
+	sprite.AllocateData(width * height);
 	sprite.width = width;
 	sprite.height = height;
 	sprite.x_offs = slot->bitmap_left;
@@ @@ -740,7 +740,6 @@ const Sprite *GetGlyph(FontSize size, WC @@
+	}
 	new_glyph.sprite = BlitterFactoryBase::GetCurrentBlitter()->Encode(&sprite, AllocateFont);
 	free(sprite.data);
 	new_glyph.width  = (slot->advance.x >> 6) + (size != FS_NORMAL);
 	SetGlyphPtr(size, key, &new_glyph);

src/spritecache.cpp

➞

Show inline comments

@@ @@ -154,7 +154,6 @@ static void *ReadSprite(SpriteCache *sc, @@
 		if (sprite_loader.LoadSprite(&sprite, file_slot, sc->id, sprite_type)) {
 			sc->ptr = BlitterFactoryBase::GetCurrentBlitter()->Encode(&sprite, &AllocSprite);
 			free(sprite.data);
 			return sc->ptr;
+		}
@@ @@ -256,7 +255,6 @@ static void *ReadSprite(SpriteCache *sc, @@
 		return (void*)GetRawSprite(SPR_IMG_QUERY, ST_NORMAL);
+	}
 	sc->ptr = BlitterFactoryBase::GetCurrentBlitter()->Encode(&sprite, &AllocSprite);
 	free(sprite.data);
 	return sc->ptr;
+}
@@ @@ -561,3 +559,18 @@ void GfxInitSpriteMem() @@
 	_compact_cache_counter = 0;
+}
 void SpriteLoader::Sprite::AllocateData(size_t size)
+{
 	if (Sprite::size < size) {
 		Sprite::size = size;
 		Sprite::mem = ReallocT<SpriteLoader::CommonPixel>(Sprite::mem, Sprite::size);
+	}
 	memset(Sprite::mem, 0, sizeof(SpriteLoader::CommonPixel) * size);
 	this->data = Sprite::mem;
+}
 /* static */ SpriteLoader::CommonPixel *SpriteLoader::Sprite::mem = NULL;
 /* static */ size_t SpriteLoader::Sprite::size = 0;

src/spriteloader/grf.cpp

➞

Show inline comments

@@ @@ -86,7 +86,7 @@ bool SpriteLoaderGrf::LoadSprite(SpriteL @@
 	if (num != 0) return WarnCorruptSprite(file_slot, file_pos, __LINE__);
-	sprite->data = CallocT<SpriteLoader::CommonPixel>(sprite->width * sprite->height);
+	sprite->AllocateData(sprite->width * sprite->height);
 	/* When there are transparency pixels, this format has an other trick.. decode it */
 	if (type & 0x08) {

src/spriteloader/png.cpp

➞

Show inline comments

@@ @@ -51,7 +51,6 @@ static bool LoadPNG(SpriteLoader::Sprite @@
 	png_infop info_ptr, end_info;
 	uint bit_depth, colour_type;
 	uint i, pixelsize;
 	png_bytep row_pointer;
 	SpriteLoader::CommonPixel *dst;
 	if (!OpenPNGFile(filename, id, mask)) return mask; // If mask is true, and file not found, continue true anyway, as it isn't a show-stopper
@@ @@ -102,7 +101,7 @@ static bool LoadPNG(SpriteLoader::Sprite @@
 		sprite->height = info_ptr->height;
 		sprite->width  = info_ptr->width;
-		sprite->data = CallocT<SpriteLoader::CommonPixel>(sprite->width * sprite->height);
+		sprite->AllocateData(sprite->width * sprite->height);
+	}
 	bit_depth  = png_get_bit_depth(png_ptr, info_ptr);
@@ @@ -134,11 +133,7 @@ static bool LoadPNG(SpriteLoader::Sprite @@
 		pixelsize = sizeof(uint8);
+	}
 	row_pointer = (png_byte *)MallocT<byte>(info_ptr->width * pixelsize);
 	if (row_pointer == NULL) {
 		png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
 		return false;
+	}
 	png_bytep row_pointer = AllocaM(png_byte, info_ptr->width * pixelsize);
 	for (i = 0; i < info_ptr->height; i++) {
 		png_read_row(png_ptr, row_pointer, NULL);
@@ @@ -164,7 +159,6 @@ static bool LoadPNG(SpriteLoader::Sprite @@
+		}
+	}
 	free(row_pointer);
 	png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
 	return true;

src/spriteloader/spriteloader.hpp

➞

Show inline comments

@@ @@ -15,12 +15,32 @@ public: @@
 		uint8 m;  ///< Remap-channel
 	};
 	/**
 	 * Structure for passing information from the sprite loader to the blitter.
 	 * You can only use this struct once at a time when using AllocateData to
 	 * allocate the memory as that will always return the same memory address.
 	 * This to prevent thousands of malloc + frees just to load a sprite.
 	 */
 	struct Sprite {
 		Sprite() : data(NULL) {}
 		~Sprite() { assert(this->data == NULL || this->data == Sprite::mem); }
 		uint16 height;                   ///< Height of the sprite
 		uint16 width;                    ///< Width of the sprite
 		int16 x_offs;                    ///< The x-offset of where the sprite will be drawn
 		int16 y_offs;                    ///< The y-offset of where the sprite will be drawn
 		SpriteLoader::CommonPixel *data; ///< The sprite itself
 		/**
 		 * Allocate the sprite data of this sprite.
 		 * @param size the minimum size of the data field.
 		 */
 		void AllocateData(size_t size);
 	private:
 		/** Allocated memory to pass sprite data around */
 		static SpriteLoader::CommonPixel *mem;
 		/** Size (in items) of the above memory. */
 		static size_t size;
 	};
 	/**

0 comments (0 inline, 0 general)