@b4n commented on this pull request.
> {
- /* TODO use the code below post-1.28 as it introduces a
new string
- g_set_error(error, gerror->domain, gerror->code,
- _("Failed to convert working directory into
locale encoding: %s"), gerror->message);
- */
- g_propagate_error(error, gerror);
- success = FALSE;
+ w_working_directory =
g_utf8_to_utf16(utf8_working_directory, -1, NULL, NULL, &gerror);
+ if (! w_working_directory)
+ {
+ /* TODO use the code below post-1.28 as it
introduces a new string
+ g_set_error(error, gerror->domain, gerror->code,
Heh, makes sense I guess đŸ˜…
It'll be slightly harder to handle now I added an additional case, but I'll see
to either do it or remove the comment if it seems too hard/verbose for the
benefit
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4482#discussion_r2600434613
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/4482/review/[email protected]>
