[Github-comments] [geany/geany-plugins] lsp: Fix incremental synchronization with multi-byte characters (PR #1451)

2025-06-18 Thread Colomban Wendling via Github-comments
At least clangd discards the file if the range doesn't match what it sees, so this has the effect of breaking the LSP features whenever one type a multi-byte character. --- :warning: beware, this has been barely tested. I am moderately certain the fix is correct as it's the unit other pieces o

[Github-comments] Re: [geany/geany-plugins] French translation update (PR #1449)

2025-06-18 Thread Colomban Wendling via Github-comments
b4n left a comment (geany/geany-plugins#1449) > Frankly, I'm not sure all of them are unique even in the English version. It > was like "I'll use this letter as it seems it hasn't been used anywhere else" > without making really sure ;-) I bet :smile: -- and no, that didn't work. FWIW, issues

[Github-comments] Re: [geany/geany-plugins] lsp, projectorganizer: Fix translator comments (PR #1450)

2025-06-18 Thread Jiří Techet via Github-comments
Merged #1450 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1450#event-18216756945 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] lsp, projectorganizer: Fix translator comments (PR #1450)

2025-06-18 Thread Colomban Wendling via Github-comments
b4n left a comment (geany/geany-plugins#1450) I can be merged now, no worries, it only gives additional context for translators. But it will appear in .po files only after next update -- which we might want to do, @frlan ? Its not critical though. -- Reply to this email directly or view it on

[Github-comments] Re: [geany/geany-plugins] lsp, projectorganizer: Fix translator comments (PR #1450)

2025-06-18 Thread Jiří Techet via Github-comments
@techee approved this pull request. LGTM, thanks. Can this be merged now or should it wait after 2.1? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1450#pullrequestreview-2940225736 You are receiving this because you are subscribed to this t

[Github-comments] Re: [geany/geany-plugins] Crash with lsp+git-changebar (Issue #1447)

2025-06-18 Thread Jiří Techet via Github-comments
techee left a comment (geany/geany-plugins#1447) > Yeah I really don't know what creates such odd-looking stack traces in a > thread, maybe a crash at the wrong time when trying to exec()? Under normal conditions (if the server didn't crash for instance), new server would only be started if you

[Github-comments] Re: [geany/geany-plugins] GeanyLua: geany.fileinfo: Added a field for single-line comments; all 'comment' fields are renamed as in filetype definition files. (PR #1397)

2025-06-18 Thread Skif-off via Github-comments
@Skif-off pushed 1 commit. 71f6692373cf174ee5b1ecf44f8978acf1d62611 GeanyLua: geany.fileinfo: Added a field for single-line comments; all 'comment' fields are renamed as in filetype definition files. Cosmetics (alignment). -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1397

[Github-comments] Re: [geany/geany-plugins] Crash with lsp+git-changebar (Issue #1447)

2025-06-18 Thread Colomban Wendling via Github-comments
b4n left a comment (geany/geany-plugins#1447) I'll try that and see if I get a crash and more info > You could also check `top` output and memory consumption if either the plugin > or the server don't run out of memory because of some leak. Well, I'll keep an eye, but I highly doubt it: it was

[Github-comments] Re: [geany/geany-plugins] Crash with lsp+git-changebar (Issue #1447)

2025-06-18 Thread Jiří Techet via Github-comments
techee left a comment (geany/geany-plugins#1447) Maybe you could try ```ini rpc_log=stdout rpc_log_full=false # or even true, it's just a little too verbose show_server_stderr=true ``` in the server configuration. This will print some info to stderr and could give us some clue what was being ca

[Github-comments] Re: [geany/geany-plugins] French translation update (PR #1449)

2025-06-18 Thread Jiří Techet via Github-comments
techee left a comment (geany/geany-plugins#1449) > @techee The endless LSP menu is a nightmare regarding mnemonics… 😅 Frankly, I'm not sure all of them are unique even in the English version. It was like "I'll use this letter as it seems it hasn't been used anywhere else" without making really

[Github-comments] Re: [geany/geany-plugins] French translation update (PR #1449)

2025-06-18 Thread Colomban Wendling via Github-comments
@b4n pushed 1 commit. ad937475a274f00e0cfec8184386b8ddaed26091 Avoid mnemonic conflicts -- View it on GitHub: https://github.com/geany/geany-plugins/pull/1449/files/8dc59a7f76eea2153b8afe4148e7416e2c875208..ad937475a274f00e0cfec8184386b8ddaed26091 You are receiving this because you are subscrib