<div class="gmail_quote">On Thu, Apr 22, 2010 at 15:36, Angus Lees <span dir="ltr"><<a href="mailto:alees@google.com">alees@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="gmail_quote"><div class="im">On Wed, Apr 21, 2010 at 23:44, Max Horn <span dir="ltr"><<a href="mailto:fingolfin@scummvm.org" target="_blank">fingolfin@scummvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Hi Gus, hi team,<br>
<br>
thanks for your updated Android patch. Did you already see my last comments on your patch? (For reference, I am talking about patch #2603856 <<a href="https://sourceforge.net/tracker/?func=detail&aid=2603856&group_id=37116&atid=418822" target="_blank">https://sourceforge.net/tracker/?func=detail&aid=2603856&group_id=37116&atid=418822</a>>.)<br>


</blockquote><div><br></div></div><div>Just read your reply now.</div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>From my point of view, with just very little bit of work (as detailed in my comment on that tracker item) we could merge your port now -- at least in a preliminary form, i.e., with the changes in gui/ not committed directly, but only in the form of a patch file in backends/platforms/android. We could then together work on resolving that last bit from inside the directory. This is not 100% perfect, but 98%, as it would still avoid most of the burden of porting your patches to each new version.<br>


</blockquote><div><br></div></div><div>This works for me.  How do you want me to submit such a patch?</div></div></blockquote><div><br></div><div>I think this reply may have been lost in a race with me actually subscribing to the -devel list.</div>

<div>fingolfin:  how would you like me to submit a patch with an embedded .zip?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote">

<div>I'll start another thread to discuss the theme .zip changes.</div><div class="im"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

All in all, the port is IMO already in a better shape than some of our existing ports *g*.<br></blockquote><div><br></div></div><div>The big problem with the port is that it uses a bunch of Android-internal functions - and the resulting ABI changes cause it to break across Android major releases.  I'm slowly removing these as the Android guys expose more things through legitimate APIs, but there will be a fair amount of code churn until then.  Unfortunately, many of these changes will probably involve switching from private C++ to public Java functions, which are typically 10x slower.</div>


<div><br></div><div> - Gus</div></div>
</blockquote></div><br>