lars.schnei...@autodesk.com writes:
> + if (print_waiting_for_editor) {
> + fprintf(stderr,
> + _("hint: Waiting for your editor to close the
> file... "));
> + if (is_terminal_dumb())
> + /*
> + * A dumb terminal cannot erase the line later
> on. Add a
> + * newline to separate the hint from subsequent
> output.
> + */
> + fprintf(stderr, "\n");
> + fflush(stderr);
> + }
Was the trailing whitespace at the end of the hint message intended?
If we expect the editor to spit out additional garbage on the line,
it would probably help to have that SP, but if that is why we have it
there, it probably should be done only when !is_terminal_dumb().
If the trailing SP is merely there by accident, then removal without
changing anything else is also OK.
I cannot tell which is the case, hence this comment.
Thanks.
