unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Kenichi Handa <handa@m17n.org>
Cc: emacs-devel@gnu.org
Subject: Re: ^M in the info files
Date: Sat, 10 Jan 2009 14:14:18 +0200	[thread overview]
Message-ID: <u7i53jr39.fsf@gnu.org> (raw)
In-Reply-To: <uab9zjtso.fsf@gnu.org>

> Date: Sat, 10 Jan 2009 13:15:51 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: emacs-pretest-bug@gnu.org, bug-gnu-emacs@gnu.org, emacs-devel@gnu.org
> 
> The changes I installed are more thorough than what Juanma posted:
> they change both detect_coding and detect_coding_system, and also bind
> inhibit-null-byte-detection in a couple more places in info.el.

When documenting the new option, I found a strange inconsistency
between detect-coding-region/string and coding detection by
insert-file-contents.  If null bytes are detected, detect_coding sets
up to use no-conversion, but detect-coding-region/string does not call
detect_coding.  Instead, detect-coding-region/string call
detect_coding_system, which does not return no-conversion for a region
or string that include null bytes.  insert-file-contents does use
no-conversion for files that contain null bytes, but it does so
because decode_coding_gap does call detect_coding.

This creates an inconsistency for Lisp programs that do their own
decoding: if they call detect-coding-region/string for a region or
string with null bytes, they will not see no-conversion in the return
value, but insert-file-contents will use no-conversion nonetheless.

I think this inconsistency constitutes a bug.




  reply	other threads:[~2009-01-10 12:14 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-06 11:09 A few bugs not in the bug tracker (I think) Juanma Barranquero
2008-08-06 12:31 ` Kenichi Handa
2008-08-06 13:14   ` Juanma Barranquero
2008-08-06 13:36     ` Juanma Barranquero
2008-08-07  4:25       ` Miles Bader
2008-08-07 11:26         ` Juanma Barranquero
2008-08-07  0:48     ` ^M in the info files Kenichi Handa
2008-08-07  3:20       ` Eli Zaretskii
2008-08-07  3:22       ` Juanma Barranquero
2008-08-07  3:47         ` Kenichi Handa
2008-08-07 13:01           ` Juanma Barranquero
2008-08-08  1:28             ` Kenichi Handa
2008-08-08 11:02               ` Juanma Barranquero
2008-08-13 10:08                 ` Kenichi Handa
2008-08-15 23:09                   ` Juanma Barranquero
2008-08-07 20:18       ` Stefan Monnier
2008-11-28 21:28         ` Drew Adams
2008-11-28 22:39           ` Eli Zaretskii
2008-11-28 22:44             ` Juanma Barranquero
2008-11-29 10:50               ` Eli Zaretskii
2008-11-29 11:56                 ` Juanma Barranquero
2008-11-28 22:49             ` Drew Adams
2008-11-29 10:51               ` Eli Zaretskii
2009-01-10 11:15             ` Eli Zaretskii
2009-01-10 12:14               ` Eli Zaretskii [this message]
2009-01-12 20:56                 ` Stefan Monnier
2009-01-12 22:04                   ` Eli Zaretskii
2009-01-10 14:07               ` Juanma Barranquero
2009-01-10 15:13                 ` Eli Zaretskii
2009-01-10 18:23                   ` Juanma Barranquero
2009-01-10 19:08                     ` Eli Zaretskii
2009-01-10 19:12                     ` Eli Zaretskii
2009-01-10 19:19                     ` Eli Zaretskii
2009-01-10 21:04                       ` Juanma Barranquero
2009-01-10 16:21               ` Drew Adams
2009-01-10 23:16                 ` Lennart Borgman
2009-01-11  5:41                   ` dhruva
2009-01-11  5:51                   ` dhruva
2009-01-12 20:54               ` Stefan Monnier
2009-01-12 22:13                 ` Eli Zaretskii
2009-01-12 22:27                   ` Glenn Morris
2009-01-13  4:01                     ` Eli Zaretskii
2009-01-13  4:24                       ` bug bugs [was Re: ^M in the info files] Glenn Morris
2009-01-13 18:56                         ` Eli Zaretskii
2009-01-13 20:50                           ` bug bugs Glenn Morris
2009-01-14  4:12                             ` Eli Zaretskii
2009-01-14 13:45                               ` Chong Yidong
2009-01-14 19:13                                 ` Eli Zaretskii
2009-01-15  0:30                                   ` Stephen J. Turnbull
2009-01-14 19:40                               ` Glenn Morris
2008-08-06 18:04   ` A few bugs not in the bug tracker (I think) Eli Zaretskii
2008-08-06 13:35 ` Don Armstrong
2008-08-06 13:38   ` Juanma Barranquero
  -- strict thread matches above, loose matches on Subject: below --
2008-06-11  0:43 ^M in the info files Lennart Borgman (gmail)
2008-06-11  4:42 ` dhruva
2008-06-11 15:56   ` Juanma Barranquero
2008-07-09  1:51     ` Juanma Barranquero
2008-07-09  2:44       ` Kenichi Handa
2008-07-09  2:56         ` Juanma Barranquero
2008-07-09  4:33           ` Kenichi Handa
2008-07-09  9:15             ` Jason Rumney
2008-07-09 11:16               ` Kenichi Handa
2008-07-09 16:49                 ` Stefan Monnier
2008-07-09 17:58                   ` James Cloos
2008-07-09 20:19                     ` Juri Linkov
2008-07-14 11:44                       ` Kenichi Handa
2008-07-21 11:18                         ` Juanma Barranquero
2008-07-10 11:17                   ` Kenichi Handa
2008-07-10 16:02                     ` Stefan Monnier
2008-07-10 18:42                       ` Juri Linkov
2008-07-10 20:27                         ` Stefan Monnier
2008-07-10 20:47                           ` Juri Linkov
2008-07-10 22:11                             ` Stefan Monnier
2008-07-19 22:29                           ` Eli Zaretskii
2008-07-21  4:57                             ` Stefan Monnier
2008-07-21 15:08                               ` Eli Zaretskii
2008-07-21 18:20                                 ` Stefan Monnier
2008-07-09 10:02             ` Juanma Barranquero

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=u7i53jr39.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=handa@m17n.org \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).