[Scummvm-devel] Tinsel on BE archs

[Paul Gilbert]

Hi all,

I think you're right in your summation. Although I haven't actually
reviewed all the individual commits that were done, if they are that hard
to follow, then I agree that it's best we just revert everything to get
back the broken functionality. When/if someone is interested in
re-supporting the Mac version, they can gradually extract and re-apply
changes in a cleaner, more controlled fashion later.

Paul.


On Wed, Dec 4, 2013 at 4:24 PM, Willem Jan Palenstijn <wjp at usecode.org>wrote:

> 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/415that
> 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
>
>
> ------------------------------------------------------------------------------
> Sponsored by Intel(R) XDK
> Develop, test and display web and hybrid apps with a single code base.
> Download it for free now!
>
> http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
> _______________________________________________
> Scummvm-devel mailing list
> Scummvm-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/scummvm-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.scummvm.org/pipermail/scummvm-devel/attachments/20131204/b9e82a4c/attachment.html>