[Scummvm-devel] ScummVM binary size comparison

[Vicent Marti]

There is no reason to keep the original Theme and the new ThemeEngine.
It was only done that way during SoC to easen the integration. They
can be merged without any side issues, afaik. Do you have a patch
already, Max?

Also, I've modded the Python script to remove some extra whitespace,
should slightly reduce the binary size when including the builtin
theme. The binary format for themes looks like a bit too much of an
overkill... It can be done though, if we are *that* short on memory.
In fact, it sounds like a fun project... What are everyone else's
thoughts on this?

Willem: The thing is that the platforms where we are short on memory
are the platforms where we don't really want to forcedly bundle any
more stuff, I think.

Vicent Martí
----------------
http://www.smartlikearoboc.com



On Sat, Nov 8, 2008 at 12:53 PM, Willem Jan Palenstijn <wjp at usecode.org> wrote:
> On Sat, Nov 08, 2008 at 01:43:05AM +0100, Max Horn wrote:
>> But I am quite confused why ThemeEngine.o is so big (on my OSX system,
>> too). 50k of code -- for what?
>
> I looked into this a bit, and apparently ThemeEngine.cpp includes
> gui/themes/default.inc, which is a 34K string.
>
> Disabling this builtin theme (GUI_ENABLE_BUILTIN_THEME) sounds like an
> easy way to save this 34K at the expense of extra data file loading on
> platforms where this is worthwhile.
>
> -Willem Jan
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Scummvm-devel mailing list
> Scummvm-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/scummvm-devel
>