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

Luiz Romário Santana Rios luizromario at gmail.com
Mon Apr 12 23:21:50 CEST 2010



> 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
> 
>



More information about the kde-games-devel mailing list