Re: [David Ponce <david.ponce@wanadoo.fr>] Re: CC Mode 5.30

Re: [David Ponce <david.ponce@wanadoo.fr>] Re: CC Mode 5.30

[Martin Stjernholm]

> From: David Ponce <david.ponce@wanadoo.fr>
> Subject: Re: CC Mode 5.30
> To: mast@lysator.liu.se
> Cc: emacs-devel@gnu.org
> Date: Sat Jul  5 12:18:42 2003 +0200
>
> Hi,
>
>  > > /.../ One possibility is the problem is specific to my platform.  I
>  > > am building emacs with MSVC++ 6.0 and running on windoze 2K.  This
>  > > system may fail to distinguish ".c" from ".C".  /.../
/.../
> That is definitively the case, when system-type is windows-nt (or
> cygwin), Emacs can't distinguish ".c" from ".C".  Here is the relevant
> code in function `set-auto-mode' in files.el, that set
> `case-fold-search' before trying to `string-match' regexps in
> `auto-mode-alist':
>
>              (let ((alist auto-mode-alist)
> 		  (mode nil))
> 	      ;; Find first matching alist entry.
> 	      (let ((case-fold-search
> 		     (memq system-type '(vax-vms windows-nt cygwin))))
> 		(while (and (not mode) alist)
> 		  (if (string-match (car (car alist)) name)
> 		      ....

Wouldn't it be better if it downcased the name and matched it case
sensitively instead? The "normal" case in case sensitive systems is
afaik lowercase, so patterns that are written without special
attention to it use lowercase characters and would match if the name
was simply downcased.

Also, NT file systems can distinguish between upper and lower case
chars visually, even if it doesn't matter when finding files. Thus it
could be an idea to try to match the name case sensitively first, and
if that fails do another attempt with a downcased name (or with
case-fold-search set).

Re: [David Ponce <david.ponce@wanadoo.fr>] Re: CC Mode 5.30

[Martin Stjernholm]

Martin Stjernholm <mast@lysator.liu.se> wrote:

>> From: David Ponce <david.ponce@wanadoo.fr>
/.../
>> That is definitively the case, when system-type is windows-nt (or
>> cygwin), Emacs can't distinguish ".c" from ".C".

I've now changed the autoload directives in cc-mode.el so that ".c"
comes before ".C" in auto-mode-alist. Still, I think the match method
used in `set-auto-mode' should be changed:

>> Here is the relevant code in function `set-auto-mode' in files.el,
>> that set `case-fold-search' before trying to `string-match' regexps
>> in `auto-mode-alist':
>>
>>              (let ((alist auto-mode-alist)
>> 		  (mode nil))
>> 	      ;; Find first matching alist entry.
>> 	      (let ((case-fold-search
>> 		     (memq system-type '(vax-vms windows-nt cygwin))))
>> 		(while (and (not mode) alist)
>> 		  (if (string-match (car (car alist)) name)
>> 		      ....
>
> Wouldn't it be better if it downcased the name and matched it case
> sensitively instead? The "normal" case in case sensitive systems is
> afaik lowercase, so patterns that are written without special
> attention to it use lowercase characters and would match if the name
> was simply downcased.
>
> Also, NT file systems can distinguish between upper and lower case
> chars visually, even if it doesn't matter when finding files. Thus it
> could be an idea to try to match the name case sensitively first, and
> if that fails do another attempt with a downcased name (or with
> case-fold-search set).

Re: [David Ponce <david.ponce@wanadoo.fr>] Re: CC Mode 5.30

[Jason Rumney]

Martin Stjernholm <mast@lysator.liu.se> writes:

> Martin Stjernholm <mast@lysator.liu.se> wrote:
> 
> >> From: David Ponce <david.ponce@wanadoo.fr>
> /.../
> >> That is definitively the case, when system-type is windows-nt (or
> >> cygwin), Emacs can't distinguish ".c" from ".C".
> 
> I've now changed the autoload directives in cc-mode.el so that ".c"
> comes before ".C" in auto-mode-alist. Still, I think the match method
> used in `set-auto-mode' should be changed:

To what?  What we have now works, provided the entries are in the
right order.




-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01

set-auto-mode on case insensitive OS:es [Was: Re: CC Mode 5.30]

[Martin Stjernholm]

Jason Rumney <jasonr@gnu.org> wrote:

>> /.../ Still, I think the match method used in `set-auto-mode'
>> should be changed:
>
> To what?

To what I quoted further down in that mail. I repeat:

    Wouldn't it be better if it downcased the name and matched it case
    sensitively instead? The "normal" case in case sensitive systems
    is afaik lowercase, so patterns that are written without special
    attention to it use lowercase characters and would match if the
    name was simply downcased.

    Also, NT file systems can distinguish between upper and lower case
    chars visually, even if it doesn't matter when finding files. Thus
    it could be an idea to try to match the name case sensitively
    first, and if that fails do another attempt with a downcased name
    (or with case-fold-search set).

> What we have now works, provided the entries are in the right order.

It'd be less dependent on order with my suggestion. That'd be a good
thing since there's no really good way to control the order on
auto-mode-alist from autoload directives in a package, given that one
doesn't want to make assumptions on the entries that are already on
it.

Furthermore, the second paragraph in my suggestion would allow e.g. NT
users to handle .c and .C files differently too.


-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01

Re: set-auto-mode on case insensitive OS:es [Was: Re: CC Mode 5.30]

[Richard Stallman]

	Wouldn't it be better if it downcased the name and matched it case
	sensitively instead? The "normal" case in case sensitive systems
	is afaik lowercase, so patterns that are written without special
	attention to it use lowercase characters and would match if the
	name was simply downcased.

That seems like a good idea to me.  Is there any case where it would give
bad results?

	Also, NT file systems can distinguish between upper and lower case
	chars visually, even if it doesn't matter when finding files. Thus
	it could be an idea to try to match the name case sensitively
	first, and if that fails do another attempt with a downcased name
	(or with case-fold-search set).

Whether this would really give the desired results is a tricky
question.  I think that we had better wait for MS-Windows users to tell
us if that would really work well with their typical patterns of usage.


-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01

Re: set-auto-mode on case insensitive OS:es [Was: Re: CC Mode 5.30]

[Jason Rumney]

Richard Stallman <rms@gnu.org> writes:

> 	Also, NT file systems can distinguish between upper and lower case
> 	chars visually, even if it doesn't matter when finding files. Thus
> 	it could be an idea to try to match the name case sensitively
> 	first, and if that fails do another attempt with a downcased name
> 	(or with case-fold-search set).
> 
> Whether this would really give the desired results is a tricky
> question.  I think that we had better wait for MS-Windows users to tell
> us if that would really work well with their typical patterns of usage.

I don't think so. Other tools have a tendancy to change the case of
the file name at random.

Re: set-auto-mode on case insensitive OS:es [Was: Re: CC Mode 5.30]

[Stefan Monnier]

> 	Also, NT file systems can distinguish between upper and lower case
> 	chars visually, even if it doesn't matter when finding files. Thus
> 	it could be an idea to try to match the name case sensitively
> 	first, and if that fails do another attempt with a downcased name
> 	(or with case-fold-search set).
> 
> Whether this would really give the desired results is a tricky
> question.  I think that we had better wait for MS-Windows users to tell
> us if that would really work well with their typical patterns of usage.

I think it's a good idea, but probably will hit various problems
where the capitalization of the filename gets lost (because it
passed through a program that is not careful to preserve it).
I seem to remember that double-clicking on an file icon to have
Emacs open the file passes the 8+3 allcaps filename to Emacs,
so with the above change, doing it on a C file would open it in
C++ mode since the extension would be .C rather than .c.


	Stefan



-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01

Re: set-auto-mode on case insensitive OS:es [Was: Re: CC Mode 5.30]

[Jason Rumney]

"Stefan Monnier" <monnier+gnu/emacs@cs.yale.edu> writes:

> I seem to remember that double-clicking on an file icon to have
> Emacs open the file passes the 8+3 allcaps filename to Emacs,
> so with the above change, doing it on a C file would open it in
> C++ mode since the extension would be .C rather than .c.

Not any more, but Samba mounts have a problem where if the filename
fits into 8+3, the filename is always reported as allcaps.





-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01

Re: set-auto-mode on case insensitive OS:es [Was: Re: CC Mode 5.30]

[Richard Stallman]

	I seem to remember that double-clicking on an file icon to have
	Emacs open the file passes the 8+3 allcaps filename to Emacs,
	so with the above change, doing it on a C file would open it in
	C++ mode since the extension would be .C rather than .c.

    Not any more, but Samba mounts have a problem where if the filename
    fits into 8+3, the filename is always reported as allcaps.

That sounds like sufficient reason not to try to depend on
the apparent case pattern.


-------------------------------------------------------
This SF.Net email sponsored by: Parasoft
Error proof Web apps, automate testing & more.
Download & eval WebKing and get a free book.
www.parasoft.com/bulletproofapps

Re: [David Ponce <david.ponce@wanadoo.fr>] Re: CC Mode 5.30

[Tak Ota]

Sat, 05 Jul 2003 22:10:36 +0200: Martin Stjernholm <mast@lysator.liu.se> wrote:

> Martin Stjernholm <mast@lysator.liu.se> wrote:
> 
> >> From: David Ponce <david.ponce@wanadoo.fr>
> /.../
> >> That is definitively the case, when system-type is windows-nt (or
> >> cygwin), Emacs can't distinguish ".c" from ".C".

I have confirmed that the problem I experienced is now fixed.  Thanks.

-Tak


-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01