[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#35464: [PATCH] Refactor update_window_begin and update_window_end ho
bug#35464: [PATCH] Refactor update_window_begin and update_window_end hooks
Sun, 28 Apr 2019 12:37:28 -0600
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux)
Eli Zaretskii <address@hidden> writes:
> Don't know, the problem was specific to X. I think we should leave
> the code there, the NS port has enough redisplay problems already.
It seems that it's needed (see commit 96e78d1fb) for both X and W32, so
I wouldn't be surprised if NS needs the change as well, or, at least, if
it doesn't matter for that port.
>> @@ -13145,8 +13069,8 @@ static struct redisplay_interface
>> x_redisplay_interface =
>> - x_update_window_begin,
>> - x_update_window_end,
>> + gui_update_window_begin,
>> + gui_update_window_end,
> This looks like a step in the wrong direction to me: the different
> implementations are all almost completely identical, except that w32
> has a small quirk there. So I'd say make a single function
> window_update_begin, that will be called directly (not via a hook
> pointer), and make the w32 part be an optional hook called only if
> Also, please don't add gui_* functions extracted from the *term.c
> files in xdisp.c, as that file is already too large. Renaming
> existing functions in xdisp.c is OK, as well as adding static utility
> functions. But for new gui_* functions that were originally in
> xterm.c etc., I'd prefer a new file, let's call it gui_term.c.
A gui_term.c might be beneficial in the future, but I'm not sure it
should be the place for RIF procedures like these. I attached an updated
patch that currently puts them in terminal.c (beside update_begin and
update_end), but I think a better place would be to put them beside
update_window in dispnew.c. WDYT?