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

Some variables declared as enum are not detected #250

Open
michaelopdenacker opened this issue May 15, 2023 · 4 comments · Fixed by #344
Open

Some variables declared as enum are not detected #250

michaelopdenacker opened this issue May 15, 2023 · 4 comments · Fixed by #344
Labels
indexing Related to the index content — missing definitions/references, lexer bugs, new ctags features...

Comments

@michaelopdenacker
Copy link

For example pin_config_param https://elixir.bootlin.com/linux/v6.4-rc2/C/ident/pin_config_param.
Here it's not an "enum" symbol, just a variable declared as enum.

@fstachura
Copy link
Collaborator

I'm not sure what this issue refers to, in current index pin_config_param is an enum, not a variable. Maybe that wasn't the case with the corrupted index mentioned in #246?

From #246

v6.4 database is definitely corrupted. On top of that (dunno if it's a separate bug) the enum fields are not indexed. Ex.: https://elixir.bootlin.com/linux/v6.4-rc2/C/ident/pin_config_param.

This is probably #221, fixed in #344 (not merged yet). Notice that all enumerators contain CONFIG_.

@fstachura fstachura added the indexing Related to the index content — missing definitions/references, lexer bugs, new ctags features... label Feb 21, 2025
@andy-shev
Copy link

andy-shev commented Feb 24, 2025

@fstachura
Look at https://elixir.bootlin.com/linux/v6.13.4/A/ident/PIN_CONFIG_BIAS_DISABLE, for example. It's empty, while it must not.

@fstachura
Copy link
Collaborator

Yeah, but I think that's #221. Elixir skips all identifiers that match *CONFIG_*. It was supposed to be ^CONFIG_, probably, to skip defines exposed by Kconifg. See this. The fix is ready (#344).

@tleb
Copy link
Member

tleb commented Feb 25, 2025

A fresh indexing will be done once #372 is merged!

@tleb tleb closed this as completed in #344 Feb 25, 2025
@tleb tleb reopened this Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
indexing Related to the index content — missing definitions/references, lexer bugs, new ctags features...
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants