[Kde-games-devel] Review Request 113671: KMines: krazy fix: Using QPointers when showing modal dialogs via exec()
Commit Hook
null at kde.org
Tue Nov 12 22:53:54 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113671/#review43559
-----------------------------------------------------------
This review has been submitted with commit a6973045804bb0a030b2b83711d548314df1e00b by Albert Astals Cid on behalf of Harsh Kumar to branch master.
- Commit Hook
On Nov. 12, 2013, 7:32 a.m., Harsh Kumar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113671/
> -----------------------------------------------------------
>
> (Updated Nov. 12, 2013, 7:32 a.m.)
>
>
> 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/04e5ee61/attachment.html>
More information about the kde-games-devel
mailing list