[Kde-games-devel] Review Request 113671: KMines: krazy fix: Using QPointers when showing modal dialogs via exec()

Harsh Kumar harsh1kumar at gmail.com
Tue Nov 12 22:53:58 UTC 2013


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

(Updated Nov. 12, 2013, 10:53 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Games.


Repository: kmines


Description
-------

This is to fix a issue flagged by Krazy Code Checker :
ebn.kde.org/krazy/reports/kde-4.x/kdegames/kmines/index.html

In the fix, QPointers is used to create an object for modal dialogs which are shown via exec(). As mentioned in blogs.kde.org/node/3919, this fix will prevent crashing of the application when it is exited externally, for example by dbus.

Patch Changes the file - kmines/mainwindow.cpp


Diffs
-----

  mainwindow.cpp 078dc6e 

Diff: http://git.reviewboard.kde.org/r/113671/diff/


Testing
-------

Compiles & runs (on Arch Linux)


Thanks,

Harsh Kumar

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


More information about the kde-games-devel mailing list