[Github-comments] Re: [geany/geany] Is there a shortcut to insert a dash? (Discussion #3887)

2024-05-29 Thread elextr via Github-comments
Yeah, so long as its clear that this relates to the operating system and particular hardware keyboards thats fine. Note, I am being totally selfish here, I don't want to be fielding complaints that some binding doesn't work on particular keyboards or systems ;-) Maybe something like "Shortcuts

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-28 Thread elextr via Github-comments
@nyamatongwe Geany doesn't set `SCI_SETLAYOUTCACHE` so that makes it `SC_CACHE_CARET` IIUC so it is likely Scintilla would have used `libpthread`. But then Scintilla doesn't use pthreads directly itself, it uses C++ ``, although I suppose that might consist of macros over pthread calls making

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-28 Thread elextr via Github-comments
One thought, are you _sure_ its not an olde autotools linking stuff in the wrong order? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3888#issuecomment-2136228431 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Is there a shortcut to insert a dash? (Discussion #3887)

2024-05-28 Thread elextr via Github-comments
No, those are not Geany shortcuts, they are system key sequences and are keyboard specific, as I implied [above](https://github.com/geany/geany/discussions/3887#discussioncomment-9564508) they are not available on common keyboards. These are not Geany specific so they do not belong in a table

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-28 Thread elextr via Github-comments
Or you could just not use `-Wl,-z,defs` as part of `--b4n_pedantic` of course, so long as the final library works who cares if the object files have undefined symbols? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3888#issuecomment-2136167172 You are

[Github-comments] Re: [geany/geany] WHY REMOVING GEANY FROM APT ???? (Issue #3889)

2024-05-28 Thread elextr via Github-comments
Yeah, its in new Debians and I guess Ubuntu and Mint get it from there. I could believe some KDE distros might not have an incentive to support a GTK IDE, but since the OP closed the issue without answering the question of which distro, who knows. -- Reply to this email directly or view it

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-28 Thread elextr via Github-comments
Maybe this should be limited to the dodgy versions of compiler/libstdc++ since most compilers seem to work as is. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3888#issuecomment-2134820516 You are receiving this because you are subscribed to this

[Github-comments] Re: [geany/geany] WHY REMOVING GEANY FROM APT ???? (Issue #3889)

2024-05-28 Thread elextr via Github-comments
Which distros? And anyway you need to ask them, they havn't communicated any problems to us. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3889#issuecomment-2134698294 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] WHY REMOVING GEANY FROM APT ???? (Issue #3889)

2024-05-28 Thread elextr via Github-comments
The Geany project does not create any repository entries, the distros do that, so go shout at your distro, not us. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3889#issuecomment-2134688705 You are receiving this because you are subscribed to this

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-28 Thread elextr via Github-comments
So how does GTK/GIO/Glib do threads if pthreads isn't linked properly? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3888#issuecomment-2134636091 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-28 Thread elextr via Github-comments
Hmm, yes, Scintilla in Geany probably doesn't actually use threads, we don't `SCI_SETLAYOUTTHREADS` and we don't use the multithreaded loaders, so its probably safe if it just builds, even if it doesn't work :-) -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Explicitly link to libpthread if available (PR #3888)

2024-05-27 Thread elextr via Github-comments
Possibly the olde libstdcxx does not properly support C++ threads? So even if you force it maybe it will crash and burn? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3888#issuecomment-2134164142 You are receiving this because you are subscribed to

[Github-comments] Re: [geany/geany] Is there a shortcut to insert a dash? (Discussion #3887)

2024-05-27 Thread elextr via Github-comments
Its actually in the [manual](https://www.geany.org/manual/current/index.html#inserting-unicode-characters), although the second paragraph about unspecified "earlier versions" of Geany is probably irrelevant now and could be removed. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-27 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-27 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-27 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany-plugins] Add LSP plugin (PR #1331)

2024-05-26 Thread elextr via Github-comments
Does it include #3865? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1331#issuecomment-2132441626 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Is there a shortcut to insert a dash? (Discussion #3887)

2024-05-26 Thread elextr via Github-comments
Meanwhile, back in the world of US-105 keyboards ... :frowning_face: -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/discussions/3887#discussioncomment-9564508 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] ProjectOrganizer, Workbench: Use GStatBuf for "g_stat()" calls (PR #1353)

2024-05-26 Thread elextr via Github-comments
> For some reason, since the latest MSYS2 update on my Windows system, I got > compiler errors that g_stat() requires a GStatBuf type as second argument. On windows there are many MS `stat` calls and `struct stat` is a structure that changes depending on platform, compiler, target os, wind

[Github-comments] Re: [geany/geany] Plugin extensions (aka LSP API) (PR #3849)

2024-05-26 Thread elextr via Github-comments
First a general comment, the idea of a general idiom that allows items of Geany functionality to be substituted by plugins is a good one. It has the potential to be expanded for many uses. For example you may remember a "discussion" some time ago between me and @b4n about auto indentation and

[Github-comments] Re: [geany/geany-plugins] Add LSP plugin (PR #1331)

2024-05-26 Thread elextr via Github-comments
> I don't think clangd will ever work without a project - it really just > compiles it behind the scenes - so testing it on individual files won't work > and better to use TM for those. Well, although in theory it is possible to not have `compile_commands.json` but really clangd is a pain

[Github-comments] Re: [geany/geany] Is there a shortcut to insert a dash? (Discussion #3887)

2024-05-26 Thread elextr via Github-comments
Making such into a snippet with an appropriate keybinding is left as an exercise for the reader of the manual :grinning: -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/discussions/3887#discussioncomment-9560521 You are receiving this because you are

[Github-comments] Re: [geany/geany] Is there a shortcut to insert a dash? (Discussion #3887)

2024-05-26 Thread elextr via Github-comments
Presuming your "dash" is actually an "em-dash" (the things you put two of are technically "dash"es) then `ctrl+shift+u`2014`enter` will insert one. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/discussions/3887#discussioncomment-9560509 You are receiving

[Github-comments] Re: [geany/geany] #!/bin/busybox ash (Issue #3884)

2024-05-26 Thread elextr via Github-comments
Its not that simple, the process is: 1. read shebang, finds various shells, perl, python, php, ruby, tcl, make, D, js, rust 2. detect HTMLy prefix, eg `Preferences->Various->files.extract_filetype_regex` (the default regex is what detects the `-*- Sh -*-` but users can modify or add to it) 6.

[Github-comments] Re: [geany/geany] #!/bin/busybox ash (Issue #3884)

2024-05-25 Thread elextr via Github-comments
> The user's selection does not last long, because when closing the file and > reopening it, the original language returns. If so, Geany would have to save > a database with the chosen language for each file. It lasts for the _session_ which as you point out is until its closed, note that the

[Github-comments] Re: [geany/geany] #!/bin/busybox ash (Issue #3884)

2024-05-25 Thread elextr via Github-comments
> Another thing I would like is that when I enter # -*- Sh -*- and save the > file, it automatically changes the colors. The filetype is scanned only when a file is first opened in a session, this is because the user can set filetype, and if Geany then changed it that would be disobeying user

[Github-comments] Re: [geany/geany] PowerShell: Update keywords (PR #3886)

2024-05-25 Thread elextr via Github-comments
>From the link above, also missing "clean". What about the workflow keywords? >(He says knowing nothing about what a workflow is). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3886#issuecomment-2131564921 You are receiving this because you are

[Github-comments] Re: [geany/geany] Make Custom Target should have a directory combo box (Issue #3883)

2024-05-25 Thread elextr via Github-comments
We all know you should only run make in the top level (he says subtly referencing "Recursive Make considered harmful" [Miller, 1997, PDF](https://aegis.sourceforge.net/auug97.pdf)) so its not just as simple as parent if your project has more than one level, so something like project top level

[Github-comments] Re: [geany/geany] Feature request: Exclude folders and files inside of project and workspace (Issue #3876)

2024-05-25 Thread elextr via Github-comments
Pull requests are welcome. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3876#issuecomment-2131184708 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] #!/bin/busybox ash (Issue #3884)

2024-05-24 Thread elextr via Github-comments
Sure, but "somebody" has to do it, pull requests are welcome (after discussion about how to do it) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3884#issuecomment-2130506584 You are receiving this because you are subscribed to this thread. Message

[Github-comments] Re: [geany/geany] #!/bin/busybox ash (Issue #3884)

2024-05-24 Thread elextr via Github-comments
Detection from shebang is hard coded [here](https://github.com/geany/geany/blob/7d1b0ca805f1b830a9276ab48ceec2d8acb239de/src/filetypes.c#L595) -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3884#issuecomment-2130491623 You are receiving this because

[Github-comments] Re: [geany/geany] Make Custom Target should have a directory combo box (Issue #3883)

2024-05-24 Thread elextr via Github-comments
Most build systems need to be run in the top level of the tree, so the build command can have a directory specified, if its absolute it won't care about the current file or working dir. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-22 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-22 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-21 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] I have just solved the socket problem in my computer. (Issue #3880)

2024-05-21 Thread elextr via Github-comments
@b4n IIUC using gvfs would add lots of other capability, eg access to networked files. There were a couple of attempts, #963 got a lot of comments, but in the end was not merged, and #1414 got only one comment. Both now have conflicts so I guess "not planned" is a reasonable response. --

[Github-comments] Re: [geany/geany] geany crashes on ubuntu/mate 22.04 when attempting to select text (Issue #3843)

2024-05-21 Thread elextr via Github-comments
¯\\_(ツ)_/¯ -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3843#issuecomment-2122555486 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] geany crashes on ubuntu/mate 22.04 when attempting to select text (Issue #3843)

2024-05-21 Thread elextr via Github-comments
> non-composited Metacity WM under MATE Doesn't that setting relate to composited WMs? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3843#issuecomment-2122536891 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-21 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] geany crashes on ubuntu/mate 22.04 when attempting to select text (Issue #3843)

2024-05-21 Thread elextr via Github-comments
I dunno, thats what came up related to `XLIB_SKIP_ARGB_VISUALS`, anyway not worth learning now, Wayland is the future. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3843#issuecomment-2121988695 You are receiving this because you are subscribed to

[Github-comments] Re: [geany/geany] geany crashes on ubuntu/mate 22.04 when attempting to select text (Issue #3843)

2024-05-20 Thread elextr via Github-comments
> but they all seem to handle the case the visual isn't found Its not that it isn't found, its that apps (and that includes GTK) are not supposed to use them even if they do exist, from my link above "Most X applications ignore this visual since it is not useful to them; however some

[Github-comments] Re: [geany/geany] Remove custom WIN32 compiler macro in favor of _WIN32 (PR #3878)

2024-05-20 Thread elextr via Github-comments
@elextr commented on this pull request. > @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-20 Thread elextr via Github-comments
My basic search of issues does not show any other complaints with this background colour, so simply replacing one default with another is unlikely to be accepted. It would need to be a setting in `filetypes.common` (and possibly overrides from colour schemes) so it can be set by users or

[Github-comments] Re: [geany/geany] geany crashes on ubuntu/mate 22.04 when attempting to select text (Issue #3843)

2024-05-20 Thread elextr via Github-comments
Sounds like its GTK thats misusing the visuals since its not just Geany thats affected. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3843#issuecomment-2120445875 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Please just add a -i in the geany.desktop for not stuck in "other use socket" improperly. (Issue #3871)

2024-05-20 Thread elextr via Github-comments
We really should not be supporting operation with broken systems, and remember two Geanys in the same config do not share settings and sessions nicely anyway, its just generally a "bad thing" to do. Perhaps `-i` should prevent saving settings and sessions. -- Reply to this email directly or

[Github-comments] Re: [geany/geany] Please just add a -i in the geany.desktop for not stuck in "other use socket" improperly. (Issue #3871)

2024-05-20 Thread elextr via Github-comments
And you havn't answered the question of where did you get the copy of Geany that gives a different message to the original, and what else has been changed in that copy? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3871#issuecomment-2120114948 You

[Github-comments] Re: [geany/geany] Please just add a -i in the geany.desktop for not stuck in "other use socket" improperly. (Issue #3871)

2024-05-20 Thread elextr via Github-comments
The OP relates to accessing the socket file that is used for communication to an existing copy of Geany, that stops Geany execution to avoid corruption, it is nothing to do with opening the file. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Please just add a -i in the geany.desktop for not stuck in "other use socket" improperly. (Issue #3871)

2024-05-20 Thread elextr via Github-comments
Ranting is not needed and unhelpful, volunteers using their own time to develop software do not need to be SHOUTED at. What version of Geany is that and where did you get it? The error message in Geany code is

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-20 Thread elextr via Github-comments
The default unfocused selection style is part of [Scintilla](https://www.scintilla.org/), that is another project, its not Geany code. Geany uploads Scintilla code unmodified, we do not have the resources to maintain patches for it. You may try to have Scintilla change that default, but

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-19 Thread elextr via Github-comments
> I click next on the find box and the text is highlighted in grey. The found text is not simply highlighted, it is selected, so what is shown is the unfocused selection style, which as above is hard coded to `#xc0c0c0`. > In notepad, the matched text is highlighted in blue So Notepad either

[Github-comments] Re: [geany/geany] Please just add a -i in the geany.desktop for not stuck in "other use socket" improperly. (Issue #3871)

2024-05-19 Thread elextr via Github-comments
> > The socket file message probably means either Geany crashed > Does it lead to this? I don't think so Crashed ... as another user, eg database_admin ... -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3871#issuecomment-2119456562 You are receiving

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-19 Thread elextr via Github-comments
There is no background that is always right, thats why Scintilla has made it settable I guess. But the grey has sufficient contrast with black on white text and white on black text (on my monitor with my eyes, YMMV). Dimming the selection when not focused is a common idiom on Windows, and

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-19 Thread elextr via Github-comments
@eht16 has pointed out the reason you don't see the selection colour, you still have the focus on the find dialog, so the edit pane does not have focus, and Scintilla makes the selection grey when the edit pane is not focused. There is a fairly new API that allows [changing the unfocused

[Github-comments] Re: [geany/geany] Update ctags to p6.1.20240519.0 (PR #3877)

2024-05-19 Thread elextr via Github-comments
> it doesn't know whether it's inside a function so it shows all assignments > and the result is a little too noisy. Yeah, if they are not scoped then its not very useful. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3877#issuecomment-2119241869

[Github-comments] Re: [geany/geany] Update ctags to p6.1.20240519.0 (PR #3877)

2024-05-19 Thread elextr via Github-comments
Python does this too, every assignment is a new declaration and shows in the sidebar, on the other hand Julia (where assignment is also declaration) has chosen not to show any assignments in the sidebar, YMMV. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Please just add a -i in the geany.desktop for not stuck in "other use socket" improperly. (Issue #3871)

2024-05-19 Thread elextr via Github-comments
The socket file message probably means either Geany crashed (or was crashed by logging out or shutting down with it still running) or Geany was run as root or `sudo` and that still owns the socket link in your config directory. You can look there and the link will be obvious, so you should be

[Github-comments] Re: [geany/geany] Geany takes \n as 'n' in search/replace whole words only. It should treat new lines as '\n' search only (Issue #3875)

2024-05-19 Thread elextr via Github-comments
To add to @eht16 comment, search replace does not understand the context of what it is editing, it does not see a programming language string in which `\n` means newline, it just sees text characters `\` and `n`. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Add `.cjs` extension for JavaScript (PR #3866)

2024-05-18 Thread elextr via Github-comments
Merged #3866 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3866#event-12857263637 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] geanyctags, projectorganizer, vimode: Some README and URL updates (PR #1348)

2024-05-18 Thread elextr via Github-comments
@elextr approved this pull request. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1348#pullrequestreview-2064966967 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Feature request: Exclude folders and files inside of project and workspace (Issue #3876)

2024-05-17 Thread elextr via Github-comments
Perhaps you want one of the project plugins that add functionality to the built-in project capabilities. There is no agreement how the built-in projects should operate hence there are (IIRC) three project plugins. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-17 Thread elextr via Github-comments
This Geany is built from a month or so old Git. But I am on Linux and I see you are on Windows, maybe its something about GTK and Windows? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3874#issuecomment-2117555793 You are receiving this because you

[Github-comments] Re: [geany/geany] Selection color changed in default theme but the changed color doesn't take effect if the "Find" dialog box is open. The changed color takes effect only when the "F

2024-05-17 Thread elextr via Github-comments
Works for me, Geany 2.0, the text found by the find dialog changes colour immediately `filetypes.common` is saved, even with the find dialog open. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3874#issuecomment-2117480957 You are receiving this

[Github-comments] Re: [geany/geany] DOC_VALID() usage correctness (Issue #3872)

2024-05-12 Thread elextr via Github-comments
Yes agree, but any scenario that is possible _can_ occur, and the case of the user closes document while dialog open has been reported already, and see #2599 for a non-KDE example. Since we don't have many occurrences its not something that needs rushing into, just put it on the eternal todo

[Github-comments] Re: [geany/geany] reload dialog doesn't show up anymore (Issue #3839)

2024-05-12 Thread elextr via Github-comments
Closed #3839 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3839#event-12780298369 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] DOC_VALID() usage correctness (Issue #3872)

2024-05-12 Thread elextr via Github-comments
> checking the ID would be better. No, actually checking `id` is essential. I didn't really realise how much until now. Consider for example the sequence: 1. Geany function gets `doc` pointer 2. does something that results in a signal 3. plugin callback is called by signal and it closes the

[Github-comments] Re: [geany/geany] DOC_VALID() usage correctness (Issue #3872)

2024-05-11 Thread elextr via Github-comments
Sigh, welcome to asynchronous programming and state changing beneath you. > if the pointer to the doc was freed in the meantime, DOC_VALID() usage would > lead to memory access over invalid pointer. Document objects are never freed, so you will never get an invalid pointer, so thats one less

[Github-comments] Re: [geany/geany] Windows: allow forward slashes (/) instead of backslashes (\) in "Open file" dialog (Issue #3868)

2024-05-10 Thread elextr via Github-comments
Version of Geany, what version of GTK? AFAICT you are operating totally within the GTK open file dialog, it does the munging of typed input with open directory and Geany can only read the results. So we can't control that behaviour. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] [Question] gtk4 support? (#2713)

2024-05-05 Thread elextr via Github-comments
Closed #2713 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2713#event-12709232704 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] [Question] gtk4 support? (#2713)

2024-05-05 Thread elextr via Github-comments
Until Scintilla is ported to GTK4 Geany cannot even think about it, so no point discussing it. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/2713#issuecomment-2095085963 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] geanyctags, projectorganizer, vimode: Some README and URL updates (PR #1348)

2024-05-05 Thread elextr via Github-comments
@elextr commented on this pull request. Couple of suggestions, otherwise looks good. > @@ -11,7 +11,9 @@ GeanyCtags adds a simple support for generating and > querying ctags files for a G project. It requires that the ctags command is installed in a system path. On unix systems, distributions

[Github-comments] Re: [geany/geany-themes] Flatpak directory location (Issue #47)

2024-05-05 Thread elextr via Github-comments
I am not an expert, but my understanding is that technically there is not much wrong with the flatpack format if a project depends on bleeding edge components but wants to run on an older OS, it can contain/download the newer parts it needs without overwriting the original versions on the OS,

[Github-comments] Re: [geany/geany] Fix crash by protecting tm_ctags_*() functions against TM_PARSER_NONE (PR #3865)

2024-05-05 Thread elextr via Github-comments
> The file tm_ctags.c is our source that hides the ctags "API" behind some thin > layer and it is only this file that includes ctags headers so we don't > pollute the rest of our sources with symbols included from ctags. Too easy then. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Fix crash by protecting tm_ctags_*() functions against TM_PARSER_NONE (PR #3865)

2024-05-05 Thread elextr via Github-comments
> The custom filetypes that don't use tag_parser also set the parser to > TM_PARSER_NONE but don't have any global tags file so the crash doesn't > happen for them. Ok, it didn't make sense that TM_PARSER_NONE worked for custom, but not built-in, but in fact that we are doing something that

[Github-comments] Re: [geany/geany] Fix crash by protecting tm_ctags_*() functions against TM_PARSER_NONE (PR #3865)

2024-05-05 Thread elextr via Github-comments
Sorry, I wasn't clear, many of the custom filetypes have no parser and don't crash. So what do they set the `tag_parser` to? And can `tag_parser=` be set to the same value that successfully works for custom filetypes. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Add `.cjs` extension for JavaScript (PR #3866)

2024-05-05 Thread elextr via Github-comments
LGTM, doesn't clash with any other extensions, if no objections can be merged in about a week. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3866#issuecomment-2094690323 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Fix crash by protecting tm_ctags_*() functions against TM_PARSER_NONE (PR #3865)

2024-05-04 Thread elextr via Github-comments
What does upstream say about the changes? What is the setting set to if the `tag_parser=` is not present? And can `tag_parser=` mean that? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3865#issuecomment-2094556791 You are receiving this because you

[Github-comments] Re: [geany/geany-themes] Flatpak directory location (Issue #47)

2024-05-04 Thread elextr via Github-comments
@AtomicRobotMan0101 since you mentioned LM maybe you should read their [blog](https://blog.linuxmint.com/) on the topic of flatpack verifications, its a ways down but there is no direct link AFAICT. Note that (at least on the website) the "Geany team" on the Geany flatpack is marked "

[Github-comments] Re: [geany/geany] windows下中文字体乱码 (Chinese font garbled under Windows) utf8 unicode (Issue #3862)

2024-05-04 Thread elextr via Github-comments
There are unlikely to be many other editors that use the GTK toolkit on Windows (vscode definitely does not) so the fact that fonts work elsewhere does not help. GTK tends to do its own thing, so how it handles fallback fonts on Windows is unknown. Vscode uses electron which is a browser

[Github-comments] Re: [geany/geany-themes] Flatpak directory location (Issue #47)

2024-05-03 Thread elextr via Github-comments
Can a flatpack access the internet? Anyway Geany itself does not perform any internet access and does not contain the libraries to do that. A couple of the plugins in the Geany plugins collection do, so maybe "somebody" could add a plugin to read colour schemes from the repository if a

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-03 Thread elextr via Github-comments
> The issue you have is that the 4th quote (after the %f) is not an ASCII > quote, but U+201C Left Double Quotation Mark. Just replace the command with: Wow eagle eyed!! :eyes: -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] File Browser doesn't follow path on startup (Issue #3864)

2024-05-03 Thread elextr via Github-comments
> the issue can be moved? Done. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3864#issuecomment-2092603896 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-03 Thread elextr via Github-comments
Your image did not come through. As I said above, you should interact via the github website. Github is not a mailing list, it is a website, and emails are largely intended for notifications. Although you can post via email the variation in the behaviour of mailing tools means many

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-03 Thread elextr via Github-comments
Please post the string from the build entry in github, not via email, the string you have posted [here](https://github.com/geany/geany/issues/3863#issuecomment-2091366390) starts and ends with `*`. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
Note that the problem is nothing to do with Geany (or codeblocks) its the command you are entering. The best approach is to get commands to work in the command line and then paste them into tools. -- Reply to this email directly or view it on GitHub:

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
Closed #3863 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3863#event-12691206525 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
Also I presume its all one line in the build command entry? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3863#issuecomment-2092040960 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
Ok, your problem is that for some reason the website has a `*` at the start and end of the command, those should not be there. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3863#issuecomment-2091968938 You are receiving this because you are

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
Well, the original error message indicates the backticks don't match, maybe post the text from the build command entry (not from the website) on a line by itself with a line of ``` before and another after it so its not recognised as markdown. If we can't see exactly what you are inputting

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
Try a space between the `"%f"` and the text you added. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3863#issuecomment-2089920282 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-02 Thread elextr via Github-comments
> ALL settings are default with the exceptions mentioned in that youtube video. You are asking for help, so requiring people to watch a video to be able to help you means you have reduced the number who will bother, they simply don't have time. As I said, the default "Build" operation for C

[Github-comments] Re: [geany/geany-plugins] File Browser doesn't follow path on startup (Issue #1347)

2024-05-01 Thread elextr via Github-comments
Do you mean filebrowser, which is a core plugin and part of the Geany repository, or treebrowser which is part of this repository? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/issues/1347#issuecomment-2089353041 You are receiving this because you

[Github-comments] Re: [geany/geany] Unable to "Build" using Geany (Issue #3863)

2024-05-01 Thread elextr via Github-comments
The terminal setting has nothing to do with the Build command. The default build command for C does not use "/bin/sh". What have you set the build command to? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3863#issuecomment-2089335228 You are

[Github-comments] Re: [geany/geany-plugins] Enhance cppcheck usage, fix CI and fix reported errors (PR #1346)

2024-05-01 Thread elextr via Github-comments
Ooops, #1197 is the PR that claims to add `library=gtk` but doesn't, #1196 is the issue that shows the error, off by one, /me needs to be cppchecked :grin: -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1346#issuecomment-2088163400 You are

[Github-comments] Re: [geany/geany-plugins] Enhance cppcheck usage, fix CI and fix reported errors (PR #1346)

2024-05-01 Thread elextr via Github-comments
> or it got removed in the meantime Or it got removed in the version of #1197 that was eventually merged. Not sure but I think when @eht16 upped the version of cppcheck in CI the `--library=gtk` got built in or something, so it wasn't included in what got merged. Might mean it won't work for

[Github-comments] Re: [geany/geany] Use GtkFileChooserNative for opening files on Windows and macOS (PR #3861)

2024-04-30 Thread elextr via Github-comments
> Yes, but I think the problem wasn't the dialogs themselves (apart from the > fact they were some "old" open dialogs), but rather the fact they were > implemented using Windows API that nobody understood well. And #3209 crash. But one thing positive they did was that they automounted remote

[Github-comments] Re: [geany/geany-plugins] Enhance cppcheck usage, fix CI and fix reported errors (PR #1346)

2024-04-30 Thread elextr via Github-comments
> not using --library=gtk See #1197 for why its needed > Anyway, what do you guys think? Is it good? Are there too many > false-positives? Should the suppressions be moved back to AM_CPPCHECKFLAGs > not to alter the code, although it makes it harder to maintain? ... should we switch to C++

[Github-comments] Re: [geany/geany] Use GtkFileChooserNative for opening files on Windows and macOS (PR #3861)

2024-04-30 Thread elextr via Github-comments
Since I can't try either of the native dialogs, the reason I (politely) asked what the advantages of the native dialogs are is because as they are hard coded at compile time (AFAICT) users on Windows and Macos are gonna be stuck with them. So its important to know what the advantages are vs

[Github-comments] Re: [geany/geany] Use GtkFileChooserNative for opening files on Windows and macOS (PR #3861)

2024-04-29 Thread elextr via Github-comments
@rdipardo @techee thanks for testing, so apart from the inconsistent theming issue it seems to work. @techee you said in the OP "where the GTK dialogs don't offer the best user experience.". Can you expand on that and why " using native dialogs under Windows and macOS is more important IMO."?

[Github-comments] Re: [geany/geany-plugins] latex: improve readme (PR #1343)

2024-04-28 Thread elextr via Github-comments
Fine, both you and frlan prefer it so no problem, the source is only read by the developers, not the users. IIRC rEsT (never remember the caPitaLization ;-) spaces list items independent of blank lines in the source, so should not matter. The plugin pages in `plugins.geany.org` are made from

[Github-comments] Re: [geany/geany-plugins] latex: improve readme (PR #1343)

2024-04-28 Thread elextr via Github-comments
@xiota exact formatting for READMEs doesn't matter so long as they render ok on `plugins.geany.org` which assumes ReST markup IIRC but I don't know how to see what the new version of the page looks like. -- Reply to this email directly or view it on GitHub:

  1   2   3   4   5   6   7   8   9   10   >