[Kde-games-devel] Review Request: Port Kollision to use KGameRenderer
Stefan Majewsky
majewsky at gmx.net
Sun Aug 22 19:59:09 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/5059/#review7155
-----------------------------------------------------------
pictures/theme.desktop is missing from the patch.
/trunk/KDE/kdegames/kollision/mainarea.cpp
<http://reviewboard.kde.org/r/5059/#comment7297>
Shouldn't this be just m_renderer("pictures/theme.desktop")? KGameRenderer/KGameTheme is supposed to do the KStandardDirs stuff.
Other than that, stuff looks good, so I'll commit this patch when the mentioned points have been addressed.
- Stefan
On 2010-08-17 15:20:31, Brian Croom wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5059/
> -----------------------------------------------------------
>
> (Updated 2010-08-17 15:20:31)
>
>
> Review request for KDE Games, Stefan Majewsky and Dmitry Suzdalev.
>
>
> Summary
> -------
>
> This patch ports Kollision to use the KGameRenderer class for rendering balls.
>
>
> Diffs
> -----
>
> /trunk/KDE/kdegames/kollision/CMakeLists.txt 1164677
> /trunk/KDE/kdegames/kollision/ball.h 1164677
> /trunk/KDE/kdegames/kollision/ball.cpp 1164677
> /trunk/KDE/kdegames/kollision/main.cpp 1164677
> /trunk/KDE/kdegames/kollision/mainarea.h 1164677
> /trunk/KDE/kdegames/kollision/mainarea.cpp 1164677
> /trunk/KDE/kdegames/kollision/pictures/CMakeLists.txt 1164677
> /trunk/KDE/kdegames/kollision/renderer.h 1164677
> /trunk/KDE/kdegames/kollision/renderer.cpp 1164677
>
> Diff: http://reviewboard.kde.org/r/5059/diff
>
>
> Testing
> -------
>
> Everything seems to work.
>
>
> Thanks,
>
> Brian
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-games-devel/attachments/20100822/ed097f60/attachment.htm
More information about the kde-games-devel
mailing list