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.
prev parent 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
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=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 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).