[Github-comments] [geany/geany] Enable local variables for C/C++ and improve autocompletion (PR #3185)

Jiří Techet notifications at github.com
Thu May 5 12:17:27 UTC 2022


@kugel- I'd also like to discuss the stuff from here:

https://github.com/geany/geany/pull/3185#pullrequestreview-958518326

I just don't know whether it means you are against the patch completely or whether it would get approved by you under some conditions. Some things are unfixable by nature - it will be a C/C++ only thing and it will make some assumptions. Potential performance problems are fixable as I hinted in my previous posts. As @elextr said, it's a heuristic only to improve sorting so it doesn't have to be perfect and the results seem better with it than without it.

I'd also like to know your opinion on how it should be implemented so I don't spend time preparing a PR that doesn't get approved.

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

Message ID: <geany/geany/pull/3185/c1118479723 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220505/0ff7b6c1/attachment.htm>


More information about the Github-comments mailing list