@b4n commented on this pull request.


> @@ -4968,6 +4990,7 @@ static ScintillaObject *create_new_sci(GeanyEditor 
> *editor)
                g_signal_connect(sci, "button-press-event", 
G_CALLBACK(on_editor_button_press_event), editor);
                g_signal_connect(sci, "scroll-event", 
G_CALLBACK(on_editor_scroll_event), editor);
                g_signal_connect(sci, "motion-notify-event", 
G_CALLBACK(on_motion_event), NULL);
+               g_signal_connect(sci, "motion-notify-event", 
G_CALLBACK(motion_notify_event), editor);

shouldn't the two `motion-notify-event` handlers be merged?

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

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

Reply via email to