Re: Zoom: a window management minor mode -- best practices and questions

[martin rudalics]

> There's one problem with `buffer-list-update-hook` though, it gets
> called (multiple times) even wen the buffer list is not changed, e.g.,
> simply by clicking in the buffer. Is this the expected behaviour?

Yes and no.  Some users accumulate several hundred entries in their
buffer lists.  For them, checking the buffer list every time a buffer
is recorded could create an unnecessary performance burden.  So it's
expected that the function on the hook remembers the old buffer list
and compares it against the new one if this is of any importance (I
suppose that in many cases checking the head of the list is all that's
needed).

> Besides this, if there's no way to get rid of false positives in event
> handling (i.e., a relayout is triggered but no actual change happened)
> I guess I need a way to decide if a relayout is *really* needed. A
> naive way would be saving the current window and buffer but I'm not
> sure if we can do better.

For example 'switch-to-buffer' does

(select-window (selected-window)))

in order to record the buffer it displays to make sure that a "false
positive" is made when the selected window already displays the
buffer.  So saving the selected window and the buffer it displays is
the best thing one can do.

The Elisp manual mentions this as follows:

   However, when its NORECORD argument is `nil', `select-window'
updates the buffer list and thus indirectly runs the normal hook
`buffer-list-update-hook' (*note Buffer List::).  Consequently, that
hook provides a reasonable way to run a function whenever a window gets
selected more "permanently".

   Since `buffer-list-update-hook' is also run by functions that are
not related to window management, it will usually make sense to save the
value of the selected window somewhere and compare it with the value of
`selected-window' while running that hook.  Also, to avoid false
positives when using `buffer-list-update-hook', it is good practice
that every `select-window' call supposed to select a window only
temporarily passes a non-`nil' NORECORD argument.  If possible, the
macro `with-selected-window' (see below) should be used in such cases.

martin