[Scummvm-devel] Broken Sword 2.5 engine

[John Willis]

Speaking as a platform guy,

On Wed, Oct 20, 2010 at 1:27 AM, Joost Peters <joostp at 7fc1.org> wrote:
>In addition to this, it'd be nice if the PNG writing code from
>Sword25::Screenshot::SaveToFile() would use the existing thumbnail code
>instead.
>
>This sounds reasonable. BS2.5 savegames have a 'version' flag, so original
interpreter savegames wouldn't be compatible with ScummVM anyway. Plus I
plan to add a new field to the savegame >format to add the date and time
when the savegame was added anyway, since ScummVM doesn't have a function to
tell you the age of the file. Given that, there's no reason why the entire
PNG code >couldn't be removed.. it only seems to be used for thumbnails and
saving screenshots in any case, and both of those can use existing ScumMVM
functionality.
>
>Paul.

I have to put in a vote for dropping the PNG stuff from the engine if it is
that easy. By all means leave the checks in configure so it's there if
another engine wants it, but it seems daft to drag in additional
dependencies when there are core functions that can do the same job. The
less new libs that get pulled in, the better it is for the smaller platforms
;-).

Regards,

John