@elextr I don't think this PR is in the state that it should be merged. Some things (like the symbol tree LSP API) are quite ugly and should be replaced by something better. I just needed to create "something" quickly so the LSP plugin could be tested and where it's easy to see for others what Geany code it affects so the proper API can be discussed.

@techee, is there a place where the LSP-plugin is available to be built by itself for use with this?

Not yet but as I'd like to try to modify the plugin so it also builds against unmodified Geany and enables the LSP stuff that typically isn't present in Geany and doesn't interfere with anything in Geany (there will be some ifdefs in the plugin checking whether it builds against modified Geany or unmodified Geany). This plugin would then be completely standalone and I could create a PR against geany-plugins. When the LSP support is merged to Geany, the plugin would just be modified to use the new Geany API.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3571/c1936373755@github.com>