[Scummvm-devel] Tinsel on BE archs

Willem Jan Palenstijn wjp at usecode.org
Wed Dec 4 22:24:32 CET 2013


Hi all,

Tinsel has been broken on Big Endian architectures since before 1.6.0 by the
series of commits adding support for DW1 Mac.

There appears to be little hope of fixing this, since the code supporting
DW1 Mac mixes endianness in rather complicated ways, and with a commit history
that is not easy to follow. And even if we do fix the reported error at
http://sourceforge.net/p/scummvm/bugs/6345/ , I'm not confident there are no
remaining issues.

Since I think Tinsel BE support is more important than DW1 Mac support given
that we aim to be cross-platform, I think the only remaining option is to
revert it entirely, and return the Tinsel engine to the state from 2012. 

I've opened a pull request at https://github.com/scummvm/scummvm/pull/415 that
is a very quick and dirty revert of most Tinsel commits since then, but we may
want to be a bit more selective.

Opinions?

-Willem Jan




More information about the Scummvm-devel mailing list