[Kde-games-devel] Review Request: Don't crash if changing KillPercentage/Production with no planet selected.
Albert Astals Cid
aacid at kde.org
Thu Jul 12 17:46:52 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/7006/#review10876
-----------------------------------------------------------
Ship it!
Thanks!
I've commited it (with a sligh modification so that further calls to slotNewMap did not suffer of the same problem)
Can you please mark the request as submitted?
- Albert Astals Cid
On July 12, 2012, 2:31 a.m., Michael Reeves wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/7006/
> -----------------------------------------------------------
>
> (Updated July 12, 2012, 2:31 a.m.)
>
>
> Review request for KDE Games.
>
>
> Description
> -------
>
> Checks m_w->map->hasSelection() before determining the selected sector. Avoids calling selected->hasPlanet() on a null pointer. The bug report is for the KillPercentage half of this.
>
>
> This addresses bug 297748.
> https://bugs.kde.org/show_bug.cgi?id=297748
>
>
> Diffs
> -----
>
> /branches/KDE/4.8/kdegames/konquest/dialogs/newgamedlg.cc 1304945
>
> Diff: http://svn.reviewboard.kde.org/r/7006/diff/
>
>
> Testing
> -------
>
> Verfied that changing KillPercentage/Production works as expected after the patch. Confirmed that crash does not happen with patch.
>
>
> Thanks,
>
> Michael Reeves
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20120712/dabcb4bc/attachment.html>
More information about the kde-games-devel
mailing list