Review Request 111347: Fix overloaded-virtual of computeCompletions.

Commit Hook null at kde.org
Mon Jul 1 17:48:18 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111347/#review35389
-----------------------------------------------------------


This review has been submitted with commit 3bddbafad8c47d23b37afb1b3268d6f2277dfecd by Kevin Funk on behalf of Ben Wagner to branch 4.5.

- Commit Hook


On July 1, 2013, 5:48 p.m., Ben Wagner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111347/
> -----------------------------------------------------------
> 
> (Updated July 1, 2013, 5:48 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> MissingIncludeCompletionWorker::computeCompletions causes an overloaded-virtual warning when building kdevelop. Upon investigation, it appears that it was intended to override CodeCompletionWorker::computeCompletions, but the third parameter is incorrect.
> 
> KDevelop was crashing left and right and with this change seems a bit more stable.
> 
> 
> Diffs
> -----
> 
>   languages/cpp/codecompletion/missingincludemodel.h 7c5673e 
>   languages/cpp/codecompletion/missingincludemodel.cpp 346e8a1 
> 
> Diff: http://git.reviewboard.kde.org/r/111347/diff/
> 
> 
> Testing
> -------
> 
> Built, ran. Verified that this overloaded-virtual warning no longer occurs.
> 
> 
> File Attachments
> ----------------
> 
> git format-patch
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/07/01/0001-Fix-overloaded-virtual-of-computeCompletions.patch
> 
> 
> Thanks,
> 
> Ben Wagner
> 
>

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


More information about the KDevelop-devel mailing list