-
Notifications
You must be signed in to change notification settings - Fork 768
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New completer subcommands GetType and GetParent #88
Conversation
b827031
to
d31117e
Compare
return "Unknown type"; | ||
|
||
// | ||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove these two empty commented-out lines.
I added some more minor comments, but this is looking good! |
GetType - returns the type of the word at a specific location. This is used by YouCompleteMe and echo'd in vim. Where the canonical type (the "derived" type, e.g. for a typedef) is different from the declared type, that is also returned, separated with " => ". GetParent - returns the semantic parent of the word at a specific location. This is also echo'd in vim by YouCompleteMe. One other change was required to satisty the cylomatic complexity check in the test suite: the if:elif: pattern in OnUserCommand was changed to a pythonic switch (a dictionary)
d31117e
to
a00fe22
Compare
Updated as a00fe22 |
Thanks again! |
New completer subcommands GetType and GetParent
This appears to require recompiling the support library, but I didn't get a notification when I ran "BundleUpdate"; is this a bug? |
yes i think it might be. @Valloric how do you want to tackle this? |
I am so excited to be at the bleeding edge of YCM functionality. :D |
Ah, seems we forgot to bump the internal API version numbers. I'll get on it. |
OK, updated both ycmd and YCM. The message about recompiling should show up after syncing now. |
GetType - returns the type of the word at a specific location. This is used by
YouCompleteMe and echo'd in vim. Where the canonical type (the "derived" type,
e.g. for a typedef) is different from the declared type, that is also returned,
separated with " => ".
GetParent - returns the semantic parent of the word at a specific location.
This is also echo'd in vim by YouCompleteMe.
One other change was required to satisty the cylomatic complexity check in the
test suite: the if:elif: pattern in OnUserCommand was changed to a pythonic
switch (a dictionary)
see also #87 for details and history.