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

Lindsay Mathieson lindsay.mathieson at gmail.com
Sat Sep 25 14:16:31 CEST 2010


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

(Updated 2010-09-25 12:16:31.343488)


Review request for KDE Games.


Changes
-------

Removed spaces
Updated BallItem to use async pixmap loadig (setSpriteKey)
Updated KLinesRender 
  - Fully singleton with static access to exposed members rather than via "self()" function
  - Initiliased/Free'd in KLinesMainWindow constructor and destructor rather than static initialisation


Summary
-------

Updates klines to use KGameRenderer and KGameRenderedItem in place of QSvgRenderer and QGraphicsPixmapItem
Removes KPixmapCache usage


Diffs (updated)
-----

  /trunk/KDE/kdegames/klines/animator.cpp 1177731 
  /trunk/KDE/kdegames/klines/ballitem.h 1177731 
  /trunk/KDE/kdegames/klines/ballitem.cpp 1177731 
  /trunk/KDE/kdegames/klines/klines.cpp 1177731 
  /trunk/KDE/kdegames/klines/previewitem.cpp 1177731 
  /trunk/KDE/kdegames/klines/renderer.h 1177731 
  /trunk/KDE/kdegames/klines/renderer.cpp 1177731 
  /trunk/KDE/kdegames/klines/scene.cpp 1177731 

Diff: http://svn.reviewboard.kde.org/r/5399/diff


Testing
-------

Playing game
Changing themes
Resizing game
Comparison with original


Thanks,

Lindsay

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


More information about the kde-games-devel mailing list