Good to hear that it works for you.

However, re-reviewing this, I noticed a potential issue: according to the spec, 
the result of getenv() should not be used after unsetting the same environment 
variable (even though it seems to work in practice). I've added a workaround 
that makes a copy instead. Note that with this change, `desktop_startup_id` is 
not freed if Geany exits from within `parse_command_line_options()` -- if this 
is an issue, I can think of refactoring further to ensure this.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4071#issuecomment-2500083177
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/4071/[email protected]>

Reply via email to