all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Lennart Borgman <lennart.borgman.073@student.lu.se>
Subject: Re: Some q:s about find-file-not-found-set-buffer-file-coding-system
Date: Fri, 16 Jun 2006 21:51:30 +0200	[thread overview]
Message-ID: <44930BC2.3040704@student.lu.se> (raw)
In-Reply-To: <4492D326.4070208@student.lu.se>

Lennart Borgman wrote:
> I believe this function is w32 specific?
>
> 1) In the beginning of this function there is a statement
>
>    (set-buffer (current-buffer))
>
> What does this do?
>
> 2) This function is only used in the (obsolete) hook 
> `find-file-not-found-hooks'. The functions added to this non-normal 
> hook is run until one of the functions return non-nil. I believe this 
> function does not reflect this. Should it not do that?
>
> 3) This function sets buffer-file-coding-system eol type based on 
> untranslated-file-p only (avoiding this if coding-system-for-read or 
> inhibit-eol-conversion is set). Is there a way to set this based on 
> the file name? Otherwise, could this be added?
>
> 4) In addition to setting buffer-file-coding-system it sets 
> buffer-file-type. That actually breaks the return value in 2 above 
> since it is done last. It is also a bit surprising.
>
> 5) buffer-file-type is in my opinion a not very good name. Maybe 
> buffer-file-binary would be easier to understand? The variable is used 
> in very few places in Emacs.

Since I have use for the functionality I suggested above I have 
implemented it separately at the moment in a file named emacsw32-eol.el 
which is here:

    http://ourcomments.org/Emacs/DL/elisp/w32/

I think this should go into dos-w32.el however.

      reply	other threads:[~2006-06-16 19:51 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-16 15:49 Some q:s about find-file-not-found-set-buffer-file-coding-system Lennart Borgman
2006-06-16 19:51 ` Lennart Borgman [this message]

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=44930BC2.3040704@student.lu.se \
    --to=lennart.borgman.073@student.lu.se \
    /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.