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

René J.V. Bertin rjvbertin at gmail.com
Fri Nov 27 18:26:24 UTC 2015



> On Nov. 27, 2015, 7:07 p.m., Aleix Pol Gonzalez wrote:
> > Please next time provide a patch against the stable branch (5.0) so we can test properly.

No idea how long it is still going to take before I can test patches against that branch myself!


- René J.V.


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


On Nov. 27, 2015, 6: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, 6: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/e1076541/attachment.html>


More information about the KDevelop-devel mailing list