Review Request 126186: more sensible workaround for code review crash on OS X

Kevin Funk kfunk at kde.org
Fri Nov 27 17:55:18 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126186/#review88899
-----------------------------------------------------------


Code looks good to me, but please let Aleix have another look. Didn't test the patch either.

- Kevin Funk


On Nov. 27, 2015, 5:18 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126186/
> -----------------------------------------------------------
> 
> (Updated Nov. 27, 2015, 5:18 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> This patch moves the workaround from RR #120081 (commits 1f111414 and 4d2a009c) by calling `setPatch(0)` in the `PatchReviewPlugin` dtor, and using `m_patch->deleteLater()` in `setPatch()` (a change which should probably have been committed).
> As a result, the `notifyChange` slot needs a nullptr check, which can then also be used to stop the update timer.
> 
> 
> Diffs
> -----
> 
>   plugins/patchreview/patchreview.cpp ad576c2 
> 
> Diff: https://git.reviewboard.kde.org/r/126186/diff/
> 
> 
> Testing
> -------
> 
> On OS X 10.9.5 with KDELibs 4.14.14 and KDevelop 4.7.2 (git/head).
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20151127/dd9263c2/attachment-0001.html>


More information about the KDevelop-devel mailing list