[Github-comments] [geany/geany] Enable local variable and function parameter tag reporting for C/C++ (PR #3175)

elextr notifications at github.com
Sun Apr 24 11:29:13 UTC 2022


> _local variables only_

Oh, _local variables only_ :smile:  my bad.  Agree with @kugel-, removing the forward limitation is better for _local variables only_ as well.

> This is normal autocompletion (scope autocompletion only happens when you type . or ->) where the result contains all tags.

`pit` is a local, I thought you had sorted them first (my understanding from (5) in the OP), so I was pointing out `pid_t` showed before it, and its definitely not a local, in fact its from the C/C++ global tagfile IIUC.

> Tooltips right now just look at corresponding function calls which foo isn't. 

Well, its not a C function call, but it is a C++ function call, (have I mentioned not all languages are C?). If the problem is in determining function calls and not in finding the operator function tag then for sure its not part of this PR.

> Constructors are reported by ctags as methods so we see them in the same way as methods. We could probably detect them by checking if the "function" name is the same as the name of the class and filter them out.

Ok, if ctags reports them as normal member functions then its understandable, and yes they are a member function where the name matches the class name so that is indeed how to filter them.  Its just my idiom to Title case class names making the constructor sort first due to ASCII order and making them visible :stuck_out_tongue_winking_eye: .

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3175#issuecomment-1107821608
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3175/c1107821608 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220424/3bb38bb2/attachment.htm>


More information about the Github-comments mailing list