[Sugar-devel] [PATCH Maze 4/4] Save and restore state of the game

Rafael Ortiz rafael at activitycentral.com
Mon Mar 26 23:39:54 EDT 2012


On Mon, Mar 26, 2012 at 1:16 PM, Manuel Kaufmann <humitos at gmail.com> wrote:

> On Mon, Mar 26, 2012 at 14:59, Sascha Silbe <silbe at activitycentral.com>
> wrote:
> > First of all, thanks for the patch! Your help is very welcome.
> >
> > Disclaimer: I'm neither the maintainer of Maze nor a regular
> > contributor to it. My opinions and advice may differ from what you
> > need to do to get your patch accepted by the Maze maintainer.
>
> Wow! That review was awesome!
>
>
+1. kudos to Sasha for his insightful review, I agree with his points.



> I'll take a look at your review very carefully. I'm new on OLPC
> development and I'm trying to learn as much as possible. All my code
> was written taking a look at other Activities, I mean, I don't invent
> anything :P
>
> I took as example "Productive" Activity and some guides from the OLPC
> wiki. So, I will re-read them and try to discuss with you about your
> review with more data.
>
>
Many versions of the same patch can be discussed and
when a consensus is reach I can review, apply, and release.


> Thanks!
>
> --
> Kaufmann Manuel
> Blog: http://humitos.wordpress.com/
> Porfolio: http://fotos.mkaufmann.com.ar/
> PyAr: http://www.python.com.ar/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20120326/6e2b6dd3/attachment.html>


More information about the Sugar-devel mailing list