[Scummvm-devel] Replacing Common::Map by std::map

[Max Horn]

Am 15.01.2005 um 23:29 schrieb Marcus Comstedt:

>
> Max Horn <max at quendi.de> writes:
>
>> If nobody objects, I want to replace Common::Map by std::map and
>> std::set. I already did the necessary changes in my local
>> checkout. Worked quite fine; the binary size increased by 10kb or so,
>> but since the whole binary is 3.6 MB for me anyway, I don't think that
>> matters. (with MT32 disabled; with MT32, it's about 10 MB, but that is
>> another story :-/).
>
> The fact that the binary is huge should be taken as an incentive to
> try to make it smaller, rather than one to make it even larger for no
> good reason, no?

Indeed. Fully agreed. My point just was that the big bytes are mostly 
lost in other spots, like in the half dozen engines we now support. 
Furthermore, the good reasons are given in the part of the mail you 
didn't quote.


Bye,

Max