RE: [PATCH v3 1/2] launch_editor: waiting for editor message

From: rsbecker@nexbridge.com <rsbecker@nexbridge.com>
Date: Apr 12, 2024, 17:24 UTC
Message-ID: <02b701da8cfe$59be3370$0d3a9a50$@nexbridge.com>
In-Reply-To: <83b34572-498b-438c-8437-dfbb837e60ba@gmail.com>

On Friday, April 12, 2024 1:15 PM, Rubén Justo wrote:

Show 10 quoted lines

>Subject: [PATCH v3 1/2] launch_editor: waiting for editor message
>
>We have a hint shown when we are waiting for user's editor since
>abfb04d0c7 (launch_editor(): indicate that Git waits for user input, 2017-12-07).
>
>After showing the hint, we call start_command() which can return with an error.
>Then we'll show "unable to start editor...", after having said "Waiting for your
>editor...", which may be confusing.
>
>Move the code to show the hint below the start_command().

My thought on this move is for esoteric (but commonly used) terminal emulators. If one is on a t6530, tn3270, or w3270/9 emulator, for example, the emulator switches modes from text on the POSIX side to block/full screen mode when the editor is launched. Printing a message after the editor has launched has the potential to dump the message into the terminal emulation buffer and get caught in the commit text comment. This is not desirable. This change could have seriously undesirable side-effects.

On the other side, if the message is not displayed in the emulation buffer, it is deferred until after the editor closes, which makes the message a bit pointless. --Randall