[Github-comments] [geany/geany] Add support for Kotlin tags (PR #3034)

[Jan Dolinár]

That is really surprising :frowning_face:  I'd expect RPi 4 to handle this much better. I don't have any slower hardware readily available right now, but I have tested it with `cpulimit -l 50 -i geany SQLExpressionBuilder.kt`, which on my i5 should mean it should get about 1.2GHz and was still pretty smooth. There might of course also be some difference due to memory speed. 

Well, I started this as a "Christmas project" last year, so I guess I'll just have a another one for this year: to make it faster.

> ... seems to contain many anonymous functions and it seems to be a common pattern in Kotlin files and seeing the symbol tree filled with <lambda> doesn't seem that useful to me so maybe these shouldn't be reported to Geany.

Yes, lambdas can be quite common in Kotlin code and you're right, that they are probably useless in the symbol tree most of the time. I think the only valid reason to keep them is that there might be something else declared in their scope. Perhaps we could filter out all lambdas without any child nodes?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3034#issuecomment-991868661
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20211212/79e02e68/attachment.htm>