<div dir="ltr">Hi all,<div><br></div><div>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.</div>
<div><br></div><div>Paul.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Dec 4, 2013 at 4:24 PM, Willem Jan Palenstijn <span dir="ltr"><<a href="mailto:wjp@usecode.org" target="_blank">wjp@usecode.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
Tinsel has been broken on Big Endian architectures since before 1.6.0 by the<br>
series of commits adding support for DW1 Mac.<br>
<br>
There appears to be little hope of fixing this, since the code supporting<br>
DW1 Mac mixes endianness in rather complicated ways, and with a commit history<br>
that is not easy to follow. And even if we do fix the reported error at<br>
<a href="http://sourceforge.net/p/scummvm/bugs/6345/" target="_blank">http://sourceforge.net/p/scummvm/bugs/6345/</a> , I'm not confident there are no<br>
remaining issues.<br>
<br>
Since I think Tinsel BE support is more important than DW1 Mac support given<br>
that we aim to be cross-platform, I think the only remaining option is to<br>
revert it entirely, and return the Tinsel engine to the state from 2012.<br>
<br>
I've opened a pull request at <a href="https://github.com/scummvm/scummvm/pull/415" target="_blank">https://github.com/scummvm/scummvm/pull/415</a> that<br>
is a very quick and dirty revert of most Tinsel commits since then, but we may<br>
want to be a bit more selective.<br>
<br>
Opinions?<br>
<br>
-Willem Jan<br>
<br>
------------------------------------------------------------------------------<br>
Sponsored by Intel(R) XDK<br>
Develop, test and display web and hybrid apps with a single code base.<br>
Download it for free now!<br>
<a href="http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk" target="_blank">http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk</a><br>
_______________________________________________<br>
Scummvm-devel mailing list<br>
<a href="mailto:Scummvm-devel@lists.sourceforge.net">Scummvm-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br>
</blockquote></div><br></div>