@dkondor commented on this pull request.


> @@ -719,6 +729,22 @@ gboolean socket_lock_input_cb(GIOChannel *source, 
> GIOCondition condition, gpoint
                                cl_options.goto_column = atoi(buf);
                        }
                }
+               else if (strncmp(buf, "desktop_startup_id", 18) == 0)
+               {
+                       GdkDisplay *display = gdk_display_get_default();
+                       while (socket_fd_gets(sock, buf, sizeof(buf)) != -1 && 
*buf != '.')

The main reason for the while loop is to make sure we get to the end of the 
message (we should not have a line break in the token, but better be safe); 
I've refactored this to really only use the first line 

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

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

Reply via email to