D6807: Fix segmentation fault caused by certain files
Kevin Funk
noreply at phabricator.kde.org
Fri Jul 21 18:30:42 UTC 2017
kfunk added a comment.
In https://phabricator.kde.org/D6807#127188, @zhigalin wrote:
> In https://phabricator.kde.org/D6807#127164, @apol wrote:
>
> > Maybe you could add a unit test?
>
>
> What I should test?
1. revert your patch
2. add a test case parsing your test case from https://bugs.kde.org/show_bug.cgi?id=381123#c2 -- make sure running the test case inhibits a crash
3. reduce the test case as much as possible
3.1) bonus points for actually figuring out why `internalContext` is nullptr
4. reapply your patch, make sure running the test no longer crashes
REPOSITORY
R52 KDevelop: PHP Support
REVISION DETAIL
https://phabricator.kde.org/D6807
To: zhigalin
Cc: kfunk, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170721/15cde46a/attachment.html>
More information about the KDevelop-devel
mailing list