[Kde-games-devel] Review Request: Port KReversi to use KGameRenderer

[Stefan Majewsky]

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



/trunk/KDE/kdegames/kreversi/kreversichip.cpp
<http://reviewboard.kde.org/r/4948/#comment6905>

    Brian, you should add your copyright for legal reasons, and if you like, you can also add a credit in main.cpp to show up in the about dialog. I think this big contribution justifies this.



/trunk/KDE/kdegames/kreversi/kreversiscene.cpp
<http://reviewboard.kde.org/r/4948/#comment6907>

    The code of this function can be simplified significantly, for example the first line could be
    
    p->drawPixmap(sceneRect().topLeft(), m_renderer.spritePixmap("background", sceneRect().size().toSize()));



/trunk/KDE/kdegames/kreversi/pics/default_theme.desktop
<http://reviewboard.kde.org/r/4948/#comment6908>

    Huh? From where did you get the translations? I can't find them in my working dir.


- Stefan


On 2010-08-09 20:21:40, Brian Croom wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4948/
> -----------------------------------------------------------
> 
> (Updated 2010-08-09 20:21:40)
> 
> 
> Review request for KDE Games, Stefan Majewsky and Dmitry Suzdalev.
> 
> 
> Summary
> -------
> 
> This patch modifies KReversi to use KGameRenderer for loading its theme and generating/receiving its pixmaps. This is my first port attempt (first KDE contribution woohoo!) so please critique.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdegames/kreversi/CMakeLists.txt 1161066 
>   /trunk/KDE/kdegames/kreversi/kreversichip.h 1161066 
>   /trunk/KDE/kdegames/kreversi/kreversichip.cpp 1161066 
>   /trunk/KDE/kdegames/kreversi/kreversirenderer.h 1161066 
>   /trunk/KDE/kdegames/kreversi/kreversirenderer.cpp 1161066 
>   /trunk/KDE/kdegames/kreversi/kreversiscene.h 1161066 
>   /trunk/KDE/kdegames/kreversi/kreversiscene.cpp 1161066 
>   /trunk/KDE/kdegames/kreversi/mainwindow.cpp 1161066 
>   /trunk/KDE/kdegames/kreversi/pics/CMakeLists.txt 1161066 
>   /trunk/KDE/kdegames/kreversi/pics/default_theme.desktop PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/4948/diff
> 
> 
> Testing
> -------
> 
> I have tested the various hinting/marking options, changing chip color set, etc. Everything seems to be working.
> 
> 
> Thanks,
> 
> Brian
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-games-devel/attachments/20100809/ab132162/attachment-0001.htm