RE: [Proposed Patch] button2 on ISO8:T in modeline toggles eol ty pe
- First Post
- Replies
- Stats
-
Go to
- ----- 2025 -----
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2024 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2007 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2006 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2005 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2004 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2003 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2002 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2001 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2000 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 1999 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 1998 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
Stephen J. Turnbull wrote:
>>>> "Christoph" == Christoph Wedler
<Wedler> writes:
Christoph> Any patch that will not allow me to toggle the eol type
Christoph> of a new buffer to the dos ending (like a) is
Christoph> completely useless for me, unless undecided is
Christoph> automatically viewed as dos eol type under Windows-NT.
What's wrong with
(setq file-coding-system-alist
(append file-coding-system-alist '(("." . undecided-dos)))
as a workaround?
It doesn't work:
1. Use the above appended list
2. Open a new file foo.txt with two line of "ab".
3. You see the modeline is still "ISO8" (and
`buffer-file-coding-system' is iso-2208-8), but this would be
acceptable, although not nice (since I also see ISO8 for Unix
endings and ISO8:T for dos)
4. Save the buffer, the length is 6 bytes (i.e., Unix endings)
5. Kill the buffer, re-read it.
6. Modeline ISO8:T, and `buffer-file-coding-system' is #<coding_system
iso-2022-8-dos> -- huh?
7. delete your entry from file-coding-system-alist
8. Kill the buffer, re-read it.
9. Modeline ISO8, and `buffer-file-coding-system' is #<coding_system
iso-2022-8-unix>
BTW, I haven't seen one reason by anyone, why toggling from eol type nil
to some other (whether unix, dos or mac) is bad.
- Christoph
Tuesday, 11 December
Tue, 11 Dec
11:31 a.m.
New subject: [Proposed Patch] button2 on ISO8:T in modeline toggles eol ty pe
>>>> "Christoph" == Christoph Wedler
<Wedler> writes:
Christoph> BTW, I haven't seen one reason by anyone, why toggling
Christoph> from eol type nil to some other (whether unix, dos or
Christoph> mac) is bad.
It's a change. 21.4 is frozen, all changes are bad by definition,
unless they fix major problems. I accept a lot of bad by definition
patches because otherwise my head would lase, what can I say? It's
still my job to say "no", not "yes".
Submit your patch against 21.5 and resistance will most likely
evaporate, or at least become a lot more constructive. Not only that,
it will move the release date of 21.5 up by a few microseconds. If
everybody would do that, pretty soon we'd find out that 21.5 was
released yesterday! :-)
--
Institute of Policy and Planning Sciences http://turnbull.sk.tsukuba.ac.jp
University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN
Don't ask how you can "do" free software business;
ask what your business can "do for" free software.
Yoshiki Hayashi
Friday, 14 December
Fri, 14 Dec
10:16 a.m.
New subject: [Proposed Patch] button2 on ISO8:T in modeline toggles eol ty pe
"Wedler, Christoph" <christoph.wedler(a)sap.com> writes:
> What's wrong with
> (setq file-coding-system-alist
> (append file-coding-system-alist '(("." . undecided-dos)))
> as a workaround?
It doesn't work:
1. Use the above appended list
2. Open a new file foo.txt with two line of "ab".
3. You see the modeline is still "ISO8" (and
`buffer-file-coding-system' is iso-2208-8), but this would be
acceptable, although not nice (since I also see ISO8 for Unix
endings and ISO8:T for dos)
4. Save the buffer, the length is 6 bytes (i.e., Unix endings)
5. Kill the buffer, re-read it.
6. Modeline ISO8:T, and `buffer-file-coding-system' is #<coding_system
iso-2022-8-dos> -- huh?
7. delete your entry from file-coding-system-alist
8. Kill the buffer, re-read it.
9. Modeline ISO8, and `buffer-file-coding-system' is #<coding_system
iso-2022-8-unix>
Did you try this?
(setq-default default-buffer-file-coding-system 'iso-8859-1-dos)
I didn't test it with non-MULE file-coding XEmacs but it
works with MULE and it should work in your case, too.
BTW, I haven't seen one reason by anyone, why toggling from eol
type nil
to some other (whether unix, dos or mac) is bad.
OK, if you insist...
First, it simply doesn't work.
You have LF and you want it to conver to CR. You need to
do M-x toggle-buffer-coding-system twice to get CR from
LF. Then the default becomes CRLF even if the last one
choosed by user is LF.
You may say, 'OK, let's change default always'. But I don't
think it is a good change because it surprises user
Suppose you are living in Unix world. Normally, default
EOL type is nil because nil means LF. Then you wanted to
copy a file to Windows so you changed it to CRLF.
Suddenly, all new files become CRLF while you only
wanted one file to be CRLF.
If the functionality to change default EOL type is desired,
it should be done by different function. The above are
reason why I think overloading toggle-buffer-coding-system
is not a good idea.
--
Yoshiki Hayashi
8722
days inactive
8725
days old
2 comments
3 participants
tags (0)
participants (3)
-
Stephen J. Turnbull -
Wedler, Christoph -
Yoshiki Hayashi