bug#22009: PATCH: Use `window-total-width' in `window-splittable-p'

[martin rudalics]

> I agree with the asymmetry observation, but I think that the width
> check is the one that gets it right, while the height check is wrong
> and should be corrected.  When a window is split horizontally, the
> part that gets split is the text area, not the margins -- those stay
> at their original size.

Note that ‘window-splittable-p’ is exclusively used by ‘display-buffer’
and the probability is very high that the new window will not be used
for displaying the buffer of the original window.  The new window will
very likely have its own margins and header line.

> So when we decide whether to split
> vertically,

... horizontally ...

> we should consider the text-area size, not the total size.
> Otherwise, we can easily wind up in situations where, due to large
> margins, a window is split horizontally and the width of its text area
> becomes ridiculously small.

This is a problem that bites us much more with ‘split-window’ than with
‘display-buffer’.  As a matter of fact it has been already addressed in
one of the present threads discussing margins.  My opinion is that modes
that manage wide margins have to handle this in the hooks called when
windows are split, deleted or resized.  They also calculate the size of
the margins based on the current window width when they get activated.

> Therefore, I think we should document that split-width-threshold and
> window-min-width refer to the width of the text area, and if there are
> functions in window.el that compare them with window-total-width,
> those places need to be fixed, not this one.  We should also fix the
> height check in window-splittable-p.

‘window-min-width’ (and all related variables and functions) refer to
the total width of windows.  Changing its semantics would constitute
quite some effort.

> The modes that triggered this are special in that they adapt their
> margins to the split, but how would window-splittable-p know that?
> Perhaps such modes should override that function, or maybe we should
> provide a hook for them?

This is not mode-triggered.  ‘window-splittable-p’ is used by
‘display-buffer’ to find the place for displaying *help*, *info*,
*completions*, ...

>> BTW: Bug#5944 is about a related issue.  I never got around to resolve
>> it for a similar reason.
>
> Did you really mean 5944?  That's not an Emacs bug even.

It's bug#17065, which talks about line 5944 of window.el.  And I think
you might agree with the poster ;-)

martin