[Kde-games-devel] Review Request: Demo mode in Kiriki

[Luiz Romário Santana Rios]

> On 2010-04-12 20:00:54, Parker Coates wrote:
> > /trunk/KDE/kdegames/kiriki/src/row.cpp, lines 12-17
> > <http://reviewboard.kde.org/r/3573/diff/2/?file=23236#file23236line12>
> >
> >     You are correct that using an initialiser list is a good idea here, but this change is unrelated to the original goal of this patch.
> >     
> >     Please try to keep each patch down to a single bug fix, feature, or improvement as it makes reviewing much easier and keeps the VCS history cleaner. Six ten line patches are generally much preferred to one thirty line patch.

Okay, but what should I do? Send a patch only for this?


- Luiz Romário


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3573/#review4987
-----------------------------------------------------------


On 2010-04-12 17:53:41, Luiz Romário Santana Rios wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3573/
> -----------------------------------------------------------
> 
> (Updated 2010-04-12 17:53:41)
> 
> 
> Review request for KDE Games.
> 
> 
> Summary
> -------
> 
> This patch implements demonstration game in Kiriki. Almost complete.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdegames/kiriki/src/configPage.ui 1113356 
>   /trunk/KDE/kdegames/kiriki/src/kiriki.h 1113356 
>   /trunk/KDE/kdegames/kiriki/src/kiriki.cpp 1113356 
>   /trunk/KDE/kdegames/kiriki/src/kiriki.kcfg 1113356 
>   /trunk/KDE/kdegames/kiriki/src/kirikiui.rc 1113356 
>   /trunk/KDE/kdegames/kiriki/src/row.cpp 1113356 
>   /trunk/KDE/kdegames/kiriki/src/scores.cpp 1113356 
> 
> Diff: http://reviewboard.kde.org/r/3573/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Luiz Romário
> 
>