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.

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

Allow indentation of options in .ctags files #409

Closed
jfelchner opened this issue Jun 30, 2015 · 2 comments
Closed

Allow indentation of options in .ctags files #409

jfelchner opened this issue Jun 30, 2015 · 2 comments

Comments

@jfelchner
Copy link

This is a continuation of #407

It'd be nice to be able to add indentation to .ctags files for better organization. Currently if an option is indented, it causes a ctags: Warning: Ignoring non-option in ./.ctags warning.

@masatake
Copy link
Member

masatake commented Jul 1, 2015

Thank you for good idea.

Those who have interests in implementing this feature, I try to give hints:

  1. add 'trimHeaderWhitespace' or something boolean parameter to nextFileLine function in args.c:
    if trimHeaderWhitespace is true, call vStringStripLeading before strcpy:

    strcpy (result, vStringValue (vs));
    
  2. From nextFileLineSkippingComments, pass TRUE as the second argument when calling nextFileLine.

  3. Write Tmain test case.

  4. Send a PR. Don't forget "Close Allow indentation of options in .ctags files #409" to you comment.

Not so difficult. Any volunteer? We are looking for not only a person who can review parser code but also who has interests core part of ctags.

masatake added a commit to masatake/ctags that referenced this issue Jul 12, 2015
Suggested by @jfelchner in universal-ctags#409.

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
@jfelchner
Copy link
Author

@masatake amazing! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants