Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Show brief comments (updated patches) #385

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Show brief comments (updated patches) #385

wants to merge 3 commits into from

Commits on Mar 6, 2014

  1. Import latest cindex.py file

    @tobiasgrosser
    tobiasgrosser committed Mar 6, 2014
    Configuration menu
    Copy the full SHA
    cfca797 View commit details
    Browse the repository at this point in the history

  2. Use complete flags properly 

    Our complete flag handling has been completely broken. In ancient times,
we had to calculate the bits in the complete flags ourselves, but
nowadays cindex.py has a nice interface based on named parameters.  However,
we have still been passing the ancient complete flags to this interface,
which meant we always set complete_macros=True and all other flags to
false independently of the actual complete options set. This patch fixes
this. It ensures that we pass complete flags as named parameters which
makes the individual flags work again.

Based on patches from Dmitri Gribenko <gribozavr@gmail.com>
    @tobiasgrosser
    tobiasgrosser committed Mar 6, 2014
    Configuration menu
    Copy the full SHA
    d7e2cda View commit details
    Browse the repository at this point in the history

  3. Show brief comments in the completion results 

    We add a new option g:clang_include_brief_comments which includes brief
comments in the completion results

This change also introduces routines to translate python to vim data
structures. In an earlier review I saw performance regressions, but I
could not reproduce them before. I also did not include further
optimizing patches as they caused problems in the presence of '\n'.

Based on patches from Dmitri Gribenko <gribozavr@gmail.com>
    @tobiasgrosser
    tobiasgrosser committed Mar 6, 2014
    Configuration menu
    Copy the full SHA
    812abba View commit details
    Browse the repository at this point in the history