all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Robert Cochran <robert+Emacs@cochranmail.com>
To: Vasilij Schneidermann <v.schneidermann@gmail.com>
Cc: 27122@debbugs.gnu.org
Subject: bug#27122: 26.0.50; [PATCH] Add customizable to display unprintables as hex
Date: Mon, 29 May 2017 16:55:37 -0700	[thread overview]
Message-ID: <87o9ubi5fq.fsf@SoraLaptop> (raw)
In-Reply-To: <20170529202021.ax4reeivfvlijkqv@odonien.localdomain> (Vasilij Schneidermann's message of "Mon, 29 May 2017 22:20:21 +0200")

Vasilij Schneidermann <v.schneidermann@gmail.com> writes:

> While this is unfortunate, I find it more instructive to look at it in
> terms of how much complexity is added (an extra conditional and a
> customizable), not how much lurks in the existing code.  If you
> constantly went by that argument, you would have to refrain from any
> kind of Emacs hacking given its impenetrable code base.

Agree with the sentiment, which is probably a better stated version of
what I tried to say earlier. The way I wrote it eariler glossed over the
details rather heavily. Apologies for that.

> The easiest way to solve this without adding any extra complexity would
> be simply replacing the format string by the hex one.  I wouldn't want
> to be involved in the ensuing discussion though...

IMO, this is the best solution. But the size that the ensuing discussion
would inenvitably grow to puts me off of this path as well. I was
initially thinking that the current behavior was more long-standing (10+
years), but it was actually introduced in its current form in 2010, in
d419e1d94 "Fix handling of 8-bit characters in a display table.". This
makes me more convinced that it should have been hex all along, given
the relative youth of the behavior, but no better time than the present
to change it, right?

--
~Robert Cochran

GPG Fingerprint - E778 2DD4 FEA6 6A68 6F26  AD2D E5C3 EB36 4886 8871





  reply	other threads:[~2017-05-29 23:55 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-28 20:42 bug#27122: 26.0.50; [PATCH] Add customizable to display unprintables as hex Vasilij Schneidermann
2017-05-29  2:33 ` Eli Zaretskii
2017-05-29  6:28   ` Vasilij Schneidermann
2017-05-29 18:46     ` Robert Cochran
2017-05-29 19:52       ` Eli Zaretskii
2017-05-29 20:20         ` Vasilij Schneidermann
2017-05-29 23:55           ` Robert Cochran [this message]
2017-05-30  0:11             ` npostavs
2017-05-30  0:47               ` Robert Cochran
2017-05-29 19:39     ` Eli Zaretskii
2017-05-29 20:15       ` Vasilij Schneidermann
2017-05-30  6:24         ` Eli Zaretskii
2017-05-30  8:16           ` Vasilij Schneidermann
2017-05-30  8:38             ` Eli Zaretskii
2017-06-01 17:05               ` Vasilij Schneidermann
2017-06-01 18:19                 ` Eli Zaretskii
2017-05-30 12:06           ` npostavs

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=87o9ubi5fq.fsf@SoraLaptop \
    --to=robert+emacs@cochranmail.com \
    --cc=27122@debbugs.gnu.org \
    --cc=v.schneidermann@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.