-
Notifications
You must be signed in to change notification settings - Fork 152
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
Multi languages support other than C/C++ #254
base: master
Are you sure you want to change the base?
Conversation
The files with the following file extensions are also parsed by the ctags command with default options for each languages. .go .java .php .pl .py .rb .rs .sh
Hi Hiroaki |
Hi Michael I have prepared a temporal project for your reviewing. http://180.196.25.190:8888/blktrace/blktrace-1.3.0/source/blktrace.c Regards, |
Hi Michael, It took 3 days to be processed and finally the linux kernel is ready for the review. http://180.196.25.190:8888/kernel/latest/source Regards, |
Search "build_error". With this pull request, elixir lists "build_error" defined in *.rs files. |
@@ -28,6 +28,7 @@ | |||
<select name="f" title="Restricts search to specific file families"> | |||
<option value="A" {% if family=="A" %} selected="selected"{% endif %}>All symbols</option> | |||
<option value="C" {% if family=="C" %} selected="selected"{% endif %}>C/CPP/ASM</option> | |||
<option value="E" {% if family=="E" %} selected="selected"{% endif %}>non C/CPP/ASM</option> |
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.
Is it worth giving a separate option for all types that are found?
I think it would be useful to have a separate option at least for rust, since blended C+Rust projects are popping up more (including kernel). Maybe for perl and python too, but those are usually standalone scripts (vs. rust being more interconnected like c)
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.
Thank you for the comments!
Yes, I agree with the suggestions to provide a separate group for each types. The current bootlin/elixir provides searching "All symbols", so after the separations for each types, users can still choose to search within the specific file types or within the whole in the package.
@michaelopdenacker since @miharahiro posted live projects to try on, would you be able to review this? |
Hm actually @miharahiro are the links from #254 (comment) still valid? They do not seem to load for me |
The links started with " http://www.mihalabo.com:8888/ " has been recovered from the short maintenance, now its ready for the review. |
Hi |
The files with the following file extensions are also parsed by the
ctags command with default options for each languages.
.go .java .php .pl .py .rb .rs .sh