<br><br><div class="gmail_quote">On Wed, Mar 21, 2012 at 5:27 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Tue, Mar 20, 2012 at 10:52:54PM -0300, Manuel Kaufmann wrote:<br>
> Ability to 'save' (when the user closes the Activity) and 'restore' (when the<br>
> user launch it from the Journal or the Home without holding Alt) the state of<br>
> the game.<br>
><br>
> For this ability I had to upgrade 'olpcgames' to 1.6 because<br>
> 'olpcgames.FILE_READ_REQUEST' and 'olpcgames.FILE_WRITE_REQUEST' events are<br>
> added in that version and those events are needed for this.<br>
><br>
> The data is saved (as JSON, with simplejson module) in the 'event.filename'<br>
> path using the 'sugar.datastore' API.<br>
><br>
> A filename example is:<br>
><br>
>  ~/.sugar/default/datastore/09/0940b7eb-1cb7-4687-aa7a-ac2c174dcfd7/data<br>
><br>
> This commit solves ticket #2393:<br>
>  * <a href="http://bugs.sugarlabs.org/ticket/2393" target="_blank">http://bugs.sugarlabs.org/ticket/2393</a><br>
<br>
</div>Reviewed-by: James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>><br>
<br>
Contrary to Gonzalo, I don't think the olpcgames and game.py changes<br>
should be separated, since they are entirely interdependent.  Applying.<br>
one change or the other does not lead to working code.  Both are<br>
required.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>+1. although its simpler (having two patches), I don't think its necessary as per Jame's comment.</div><div><br></div>

<div><br></div><div>Thanks to humitos for the patches.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font></span></blockquote></div><br>