Review Request 124601: Added look-ahead matching code completion

[Milian Wolff]

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


+1, lgtm - esp. if it's off-by-default.


clangsettings/clangsettingsmanager.cpp (line 67)
<https://git.reviewboard.kde.org/r/124601/#comment57639>

    off-by-default



clangsettings/sessionsettings/sessionconfig.kcfg (line 12)
<https://git.reviewboard.kde.org/r/124601/#comment57638>

    off-by-default



clangsettings/sessionsettings/sessionsettings.ui (line 54)
<https://git.reviewboard.kde.org/r/124601/#comment57637>

    make it off-by-default



codecompletion/context.cpp (line 593)
<https://git.reviewboard.kde.org/r/124601/#comment57640>

    const



codecompletion/context.cpp (line 596)
<https://git.reviewboard.kde.org/r/124601/#comment57641>

    there should be a KDevUtils or similar namespace which contains a helper method for this afaik


- Milian Wolff


On Aug. 3, 2015, 10:02 a.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124601/
> -----------------------------------------------------------
> 
> (Updated Aug. 3, 2015, 10:02 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-clang
> 
> 
> Description
> -------
> 
> Known shortcomings:
> 
> 1)Auto type (and many other types) is not exposed through LibClang. So we assign DelayedType to it instead of IdentifiedType, therefore no declaration attached to the type, hence no look-ahead completion.
> 
> 2)LibClang missing API to determine expected code completion type. To workaround it, now we use types of declarations from the best mathes group. Therefore if no declarations found with a high enough priority, no items will be added to the look-ahead completion group.
> 
> As a result the look-ahead completion works only in the simpliest cases.
> 
> 
> Diffs
> -----
> 
>   clangsettings/clangsettingsmanager.h 0d6eec1 
>   clangsettings/clangsettingsmanager.cpp db125f3 
>   clangsettings/sessionsettings/sessionconfig.kcfg 63fb6c5 
>   clangsettings/sessionsettings/sessionsettings.ui 74a157f 
>   codecompletion/context.cpp 1d52021 
>   tests/test_codecompletion.cpp fca922d 
> 
> Diff: https://git.reviewboard.kde.org/r/124601/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

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