all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 33230@debbugs.gnu.org, bruno@charron.email, npostavs@gmail.com
Subject: bug#33230: 26.1; Soft-wrap issue in term.el with term-suppress-hard-newline
Date: Wed, 14 Nov 2018 09:32:57 +0100	[thread overview]
Message-ID: <5BEBDDB9.20401@gmx.at> (raw)
In-Reply-To: <83muqc99rf.fsf@gnu.org>

 >> FWIW this is Bug#32720 for which I am responsible.  It will be fixed
 >> in Emacs 26.2 by reestablishing the behavior of Emacs 25.
 >
 > And your proposed change runs window-configuration-change-hook in two
 > additional places.

Right.

 > My confusion about this is twofold: (1) the original report for this
 > bug doesn't seem to involve any resizing of a frame,

At the end it says:

   Another issue I found that may need to be addressed to get a behavior
   similar to gnome-terminal above is that the shell is not aware
   (checking $COLUMS) of when the frame is resized, only when its window
   is resized and there is another window on the side.

 > and (2) the ELisp
 > manual explicitly says that "resizing the frame or individual windows
 > do not count as configuration changes", and thus this hook shouldn't
 > be run when the frame is resized.
 >
 > So how does the proposed change fix the problem at hand,

By running 'window-configuration-change-hook' for frame resizes as
with Emacs 25.

 > and why do
 > you want to do exactly what the ELisp manual says we don't?

The Elisp manual doesn't represent the facts because we still run the
hook when resizing single windows.  The idea behind that text was to
avoid that new code runs 'window-configuration-change-hook' to trace
window size changes because that is unreliable (not all size changes
are caught) and costly (it's often run when no sizes changed at all).
Also NEWS warned that

*** Resizing a frame no longer runs 'window-configuration-change-hook'.
'window-size-change-functions' should be used instead.

and I checked known clients of 'window-configuration-change-hook'
whether they should call 'window-size-change-functions' instead.
Little did I expect to find such a client in window.el though, so this
went unnoticed.

We could add a call to 'window-size-change-functions' as Gary proposed
in the report of Bug#32720.  But then 'window--adjust-process-windows'
would be run by both 'window-configuration-change-hook' and
'window-size-change-functions' effectively increasing the number of
calls of that function instead of decreasing it.  If you prefer that
solution we can certainly do it.

martin





  reply	other threads:[~2018-11-14  8:32 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-01 18:11 bug#33230: 26.1; Soft-wrap issue in term.el with term-suppress-hard-newline Bruno CHARRON
2018-11-01 22:53 ` Noam Postavsky
2018-11-02  0:50   ` Bruno CHARRON
2018-11-03  2:15     ` Noam Postavsky
2018-11-04  7:23       ` Bruno CHARRON
2018-11-04  9:07         ` martin rudalics
2018-11-04  9:56           ` martin rudalics
2018-11-04 11:06             ` Bruno CHARRON
2018-11-13  9:07               ` martin rudalics
2018-11-13 17:18                 ` Eli Zaretskii
2018-11-14  8:32                   ` martin rudalics [this message]
2018-11-14 15:36                     ` Eli Zaretskii
2018-11-18  9:22               ` martin rudalics
2018-11-18 16:17                 ` Bruno Charron
2018-11-18 19:40                   ` martin rudalics
2022-03-22 15:42                   ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5BEBDDB9.20401@gmx.at \
    --to=rudalics@gmx.at \
    --cc=33230@debbugs.gnu.org \
    --cc=bruno@charron.email \
    --cc=eliz@gnu.org \
    --cc=npostavs@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.