Review Request 124007: Don't add functions for code-completion based on parameters.

[Sergey Kalinichev]

> On June 6, 2015, 5:09 p.m., Milian Wolff wrote:
> > codecompletion/context.cpp, line 687
> > <https://git.reviewboard.kde.org/r/124007/diff/2/?file=378961#file378961line687>
> >
> >     else it gets lost - is that correct?
> 
> Sergey Kalinichev wrote:
>     Yes. This is intentional. This code path for the special completion identifier - "operator=" for which we don't have a declaration. E.g. "class A{}; a.|".  At | we get "operator=(const A&)" as completion item which is completely useless and just pollutes the test case.
>     
>     I've added a comment for it.
> 
> Milian Wolff wrote:
>     is it only for implicit declarations, i.e. will we still get operator== and the like? There are sometimes cases where one wants to write that, and having code completion for it can be useful. But I agree that it is very rare so we can keep it as-is for now. thanks for the clarification!

Yes, this is only for the implicitly generated copy assigment operator and we still have code-completion for operator== and the like.


- Sergey


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


On June 7, 2015, 11:12 a.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124007/
> -----------------------------------------------------------
> 
> (Updated June 7, 2015, 11:12 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-clang
> 
> 
> Description
> -------
> 
> E.g.: class Abc; int f(Abc){ Ab|
> Completion invoked at the "|", that gives us "Abc" as a completion result. Before this we had "Abc" and "f(Abc)", which is obviously wrong.
>   
> Also this makes the "Detailed completion" option work.
> 
> 
> Diffs
> -----
> 
>   codecompletion/context.cpp 2bf8c81 
>   tests/test_codecompletion.cpp 470dcc0 
> 
> Diff: https://git.reviewboard.kde.org/r/124007/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

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