Codechange: remove SimpleSpriteAllocator

src/spritecache.cpp

@@ -887,14 +887,6 @@ void *CacheSpriteAllocator::AllocatePtr(size_t mem_req)
 	}
 }
 
-/**
- * Sprite allocator simply using malloc.
- */
-void *SimpleSpriteAllocator::AllocatePtr(size_t size)
-{
-	return MallocT<uint8_t>(size);
-}
-
 void *UniquePtrSpriteAllocator::AllocatePtr(size_t size)
 {
 	this->data = std::make_unique<uint8_t[]>(size);

src/spritecache.h

@@ -31,12 +31,6 @@ enum SpriteCacheCtrlFlags : uint8_t {
 
 extern uint _sprite_cache_size;
 
-/** SpriteAllocate that uses malloc to allocate memory. */
-class SimpleSpriteAllocator : public SpriteAllocator {
-protected:
-	void *AllocatePtr(size_t size) override;
-};
-
 /** SpriteAllocator that allocates memory via a unique_ptr array. */
 class UniquePtrSpriteAllocator : public SpriteAllocator {
 public:

src/tests/mock_spritecache.cpp

@@ -17,7 +17,7 @@
 
 static bool MockLoadNextSprite(SpriteID load_index)
 {
-	SimpleSpriteAllocator allocator;
+	static UniquePtrSpriteAllocator allocator;
 	static Sprite *sprite = allocator.Allocate<Sprite>(sizeof(*sprite));
 
 	bool is_mapgen = IsMapgenSpriteID(load_index);