all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: Kenichi Handa <handa@gnu.org>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: request to revert the chnage of revno 112925
Date: Wed, 19 Jun 2013 09:11:42 -0700	[thread overview]
Message-ID: <51C1D83E.7020501@cs.ucla.edu> (raw)
In-Reply-To: <yw7a9mm865j.fsf@fencepost.gnu.org>

On 06/19/2013 08:35 AM, Kenichi Handa wrote:
> I'd like to find a better solution, but at first please
> clarify the requirements.

I assume these requirements should be the same for Elisp
files as for other files that Emacs is asked to read;
if not, could this be clarified?

> * What to do with an ASCII file?  Previously find-file for
>   such a file results in undecided-xxx
>   buffer-file-coding-system.  Now it's utf-8-xxx.

To help think this through, could you please explain
the practical consequences of this change?
If I edit a file that's undecided-xxx, and insert
a character that can be encoded either as UTF-8 or
as ISO-2022-JP say, the buffer becomes utf-8-xxx,
right?  So in that scenario there is not much practical
difference.  What scenarios entail significant differences
depending on whether the file is undecided-xxx or utf-8-xxx?

> * What to do with an invalid UTF-8 file.  Previously,
>   find-file detects a proper coding-system for such a file.
>   Now utf-8 is forced and any invalid UTF-8 byte sequences
>   are decoded as raw bytes.

Surely this should be fixed: the file should be decoded
properly, as before.

> * What to do with null byte detection.  Previously, if a
>   *.el file contains a null byte and
>   inhibit-null-byte-detection is nil (the default), it's
>   detected as a binary file.  Now utf-8 is forced regardless
>   of inhibit-null-byte-detection.

I suggest going back to the old behavior (that's the normal
behavior for random files that Emacs edits, right?).
Elisp files normally don't contain null bytes; such files are not
considered to be text files in the POSIXish world.



  reply	other threads:[~2013-06-19 16:11 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-19 11:54 request to revert the chnage of revno 112925 Kenichi Handa
2013-06-19 12:59 ` Stefan Monnier
2013-06-19 15:35   ` Kenichi Handa
2013-06-19 16:11     ` Paul Eggert [this message]
2013-06-19 20:49       ` Stefan Monnier
2013-06-19 21:15         ` Paul Eggert
2013-06-20  2:09           ` Stefan Monnier
2013-06-21  5:25           ` Achim Gratz
2013-06-21  6:48             ` Stephen J. Turnbull
2013-06-19 16:54     ` Stefan Monnier
2013-06-22 12:36       ` Kenichi Handa
2013-06-22 12:46         ` Eli Zaretskii
2013-06-22 12:54           ` Juanma Barranquero
2013-06-22 15:26         ` Stefan Monnier
2013-06-29  3:50           ` request to revert the change " Kenichi Handa

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=51C1D83E.7020501@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=emacs-devel@gnu.org \
    --cc=handa@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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.