[Kde-games-devel] Review Request: Porting Bomber from KGameCanvas to QGraphicsView

[Mehmet Emre]

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

(Updated March 13, 2012, 9:28 a.m.)


Review request for KDE Games.


Changes
-------

Fixed:
- Whitespace issues
- The memory leak in Building::setHeight(unsigned int height)

Removed the "QGraphicsView *m_view;" in BomberGameWidget


Description
-------

I've ported bomber to QGraphicsView as requested here: http://techbase.kde.org/Projects/Games/Porting. I've made some modifications on the classes below:
- Explodable
- BomberBoard
- BomberGameWidget
- Building


Diffs (updated)
-----

  trunk/KDE/kdegames/bomber/src/board.h 1284898 
  trunk/KDE/kdegames/bomber/src/board.cpp 1284898 
  trunk/KDE/kdegames/bomber/src/bomberwidget.h 1284898 
  trunk/KDE/kdegames/bomber/src/bomberwidget.cpp 1284898 
  trunk/KDE/kdegames/bomber/src/building.h 1284898 
  trunk/KDE/kdegames/bomber/src/building.cpp 1284898 
  trunk/KDE/kdegames/bomber/src/explodable.h 1284898 
  trunk/KDE/kdegames/bomber/src/explodable.cpp 1284898 

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


Testing
-------

I've done these tests and solved the problems that I've bumped into:
- Played bomber's first two levels.
- Paused the game
- Changed settings during the game and while game is paused.


Thanks,

Mehmet Emre

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