<p></p>
<blockquote>
<p dir="auto">But some things are slow, like highlighting, which can take about a second or two, that may be a timer or simply thats how long the server takes to do a proper compiler anaylsis of the syntax and semantics and respond.</p>
</blockquote>
<p dir="auto">Depends what we use the language server for. The only thing that really sucks with tag manager right now in my opinion is autocompletion - the rest is quite fine. So initially the usage of LSP could concentrate on that.</p>
<blockquote>
<p dir="auto">Yeah, I was hoping for libraries, not having to adapt a client internals, tagmanager 2.0 anyone??</p>
</blockquote>
<p dir="auto">It took just about 15 years to recover from tag manager integration and we are almost there. We should start thinking about what to do in another 15 years :-).</p>
<p dir="auto">But if LSP usage is limited to some features only, it may turn out that implementing it as a plugin is enough. I think initially though for some working prototype it would be best to hack it directly into Geany to know what API might be missing and decide based on that.</p>
<p dir="auto">No matter whether we take some client code from somewhere or implement our own, there will have to be some JSON parser/serializer (json-glib seems to be a natural choice) in its core that will have to be either bundled or an external dependency - not sure if it's fine or not for Geany, for plugin it would be a easier decision.</p>
<blockquote>
<p dir="auto">But none of this is relevant to using libreadtags which seems a fine idea.</p>
</blockquote>
<p dir="auto">Yeah, back on earth, tag manager is what we have. The rosy world of LSP autocompletion for me is just an excuse why not improve the scary tag-manager-based scope completion much :-).</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/3049#issuecomment-994576821">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ3Y4G2B5SFCKYYQ4BLURBOV3ANCNFSM5JUGWD2A">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<img src="https://github.com/notifications/beacon/AAIOWJ2K2V4BSYJ2UICOWMTURBOV3A5CNFSM5JUGWD2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHNEATNI.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/3049#issuecomment-994576821",
"url": "https://github.com/geany/geany/pull/3049#issuecomment-994576821",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>