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

[Nicolas Noble]

>> As I have argued many times in the past, I don't believe using 
>> std::map will cause any portability issues, as is proven by e.g. 
>> Exult, which runs on almost all the platforms we run on. 
>
>
> Mmh. I don't remember seeing Exult run on a Pocket PC. I know for sure 
> that Embedded Visual C++ 3 will not support built-in STL (it can 
> probably compile stlport, but I wouldn't bet on it since C++ support 
> on EVC 3 with the latest patches is roughly worse than what we get in 
> Visual C 6.0 without patches).
>
> But I've already used it with EVC 4 and it worked fine.
>
> I'd say than switching to STL will increase the WinCE build size and 
> can possibly break the compatibility with older devices (Pocket PC 
> 2000, Handheld PCs). However I think it's a good idea, at least for 
> the other ports that do not rely on a broken compiler :)
>
> Cheers
>
> Nicolas


Hi there,

The actual ps2sdk's environment is quite STL-unfriendly. Raw C++ code 
runs quite fine, but compiling STL code is really painful and dodgy, 
mainly because we are not providing lots of depandancy needed by STL, 
such as locale or multibyte string functions. Those are on the 
TODO-list, but won't prolly be added after a long while, since there's a 
huge lot of other things to work out before.

I know the actual "official" PS2 port of ScummVM is beeing done by 
LavosSpawn (at least, he is trying to, since he told me that some parts 
are still not working yet), and adding more STL dependancy will probably 
produce even more troubles than he is actually experiencing.

If you really want to change this code, I'd suggest a wide testing 
session with all the porters of every platform, and request feedback 
from them before doing any real change.

Just my two cents in.

Cheers,

  -- Nicolas Noble