> + if (s->opts->has_show_cursor &&
s->opts->show_cursor) {
> + s->null_cursor = NULL; /* default pointer */
> + } else {
> + s->null_cursor = gdk_cursor_new_for_display(window_display,
> + GDK_BLANK_CURSOR);
> + }
I think it would make more sense to have all the logic related to what
cursor is used in the same location, instead of split up between here and
gd_update_cursor().
Note that it isn't that simple. There are a few more places in
gd_grab_*() which use s->null_cursor and would need adaption.
cheers,
Gerd