[Kde-games-devel] Review Request 5760: fix 199844 bug

[Christian Krippendorf]

> On Jan. 23, 2013, 10:59 p.m., Christian Krippendorf wrote:
> > Fixed in branch krippendorf from kmahjongg. Will be solved when the branch gets merged to master. So no longer needed!

Correction: branch "krippendorf" to branch "qgraphics"


- Christian


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


On Nov. 2, 2010, 10:02 a.m., Antonis Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5760/
> -----------------------------------------------------------
> 
> (Updated Nov. 2, 2010, 10:02 a.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> Hello
> 
> This patch is fixing the 199844 bug, which when someone wins then the time is reset and it is start counting again from the beginning. With this patch when someone wins then the time is reset but it doesn't counting again from the beginning. It remains to 00:00:00. I have made the following change into the displayTime because that method it doesn't only display the time but also it calculates it... So i thought that since the game is over and it doesn't need to display/calculate any time so i could fix the bug with this way... 
> 
> 
> This addresses bug 199844.
>     https://bugs.kde.org/show_bug.cgi?id=199844
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdegames/kmahjongg/kmahjongg.cpp 1192151 
> 
> Diff: http://svn.reviewboard.kde.org/r/5760/diff/
> 
> 
> Testing
> -------
> 
> I have try the patch and it works. And there is no problem in the compile. Everything works perfect.
> 
> 
> Thanks,
> 
> Antonis Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20130123/55246f30/attachment.html>