Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Michaël Cadilhac]

[-- Attachment #1.1: Type: text/plain, Size: 1487 bytes --]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> Index: emacs/lisp/ChangeLog
> diff -u emacs/lisp/ChangeLog:1.9582 emacs/lisp/ChangeLog:1.9583
> --- emacs/lisp/ChangeLog:1.9582	Mon May 22 13:19:02 2006
> +++ emacs/lisp/ChangeLog	Mon May 22 18:09:40 2006
> @@ -1,7 +1,14 @@
> +2006-05-22  Stefan Monnier  <monnier@iro.umontreal.ca>
> +
> +	* complete.el (PC-do-completion): Undo the addition of implicit
> +	wildcards if they did not lead to finding any match.
> +	(read-file-name-internal): Don't add the final > if the completion is
> +	not finished.
> +

  When I do C-x C-f /a/b/c, with shell being /bin/sh (zsh emuled),
  I still have a weird behavior:

<TAB>
read-file-name-internal: Opening directory: no such file or directory, /a*/b*/
<TAB>
read-file-name-internal: Opening directory: no such file or directory, /a**/b**/
<TAB>
read-file-name-internal: Opening directory: no such file or directory, /a***/b***/
<TAB>
read-file-name-internal: Opening directory: no such file or directory, /a****/b****/

  AFAIU, this patch fixes the /c* bug (i.e. the last part of the filename
  being completed) but not the whole problem.

-- 
 |      Michaël `Micha' Cadilhac   |   La culture c'est comme la confiture  |
 |         Epita/LRDE Promo 2007   |       c'est meilleur avec du pain.     |
 | http://www.lrde.org/~cadilh_m   |            -- MOI59                    |
 `--  -   JID: micha@amessage.be --'                                   -  --'

[-- Attachment #1.2: Type: application/pgp-signature, Size: 188 bytes --]

[-- Attachment #2: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Stefan Monnier]

>   When I do C-x C-f /a/b/c, with shell being /bin/sh (zsh emuled),
>   I still have a weird behavior:

> <TAB>
> read-file-name-internal: Opening directory: no such file or directory, /a*/b*/
> <TAB>
> read-file-name-internal: Opening directory: no such file or directory, /a**/b**/
> <TAB>
> read-file-name-internal: Opening directory: no such file or directory, /a***/b***/
> <TAB>
> read-file-name-internal: Opening directory: no such file or directory, /a****/b****/

>   AFAIU, this patch fixes the /c* bug (i.e. the last part of the filename
>   being completed) but not the whole problem.

Hmm... I remember working on such a bug report a few months ago.
Can't remember whether I had fixed it at the time and if so how.  But in any
case, I can't reproduce the problem right now.


        Stefan

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Michaël Cadilhac]

[-- Attachment #1.1.1: Type: text/plain, Size: 1142 bytes --]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>>   When I do C-x C-f /a/b/c, with shell being /bin/sh (zsh emuled),
>>   I still have a weird behavior:
>
>> <TAB>
>> read-file-name-internal: Opening directory: no such file or directory, /a*/b*/
>> <TAB>
>> read-file-name-internal: Opening directory: no such file or directory, /a**/b**/
>> <TAB>
>> read-file-name-internal: Opening directory: no such file or directory, /a***/b***/
>> <TAB>
>> read-file-name-internal: Opening directory: no such file or directory, /a****/b****/
>
>>   AFAIU, this patch fixes the /c* bug (i.e. the last part of the filename
>>   being completed) but not the whole problem.
>
> Hmm... I remember working on such a bug report a few months ago.
> Can't remember whether I had fixed it at the time and if so how.  But in any
> case, I can't reproduce the problem right now.

  My bad, I recompiled a freshly checkouted Emacs and it works almost
  right.

  When I do C-x C-f /a/b/c TAB, no star are added, but an _error_ is
  triggered instead of a [no match].

  IMHO, it's due  to file_name_completion, and I'd be  very tempted to
  make the following:


[-- Attachment #1.1.2: direr.patch --]
[-- Type: text/x-patch, Size: 1486 bytes --]

Index: src/ChangeLog
===================================================================
RCS file: /sources/emacs/emacs/src/ChangeLog,v
retrieving revision 1.5083
diff -c -r1.5083 ChangeLog
*** src/ChangeLog	23 May 2006 08:12:13 -0000	1.5083
--- src/ChangeLog	23 May 2006 17:13:37 -0000
***************
*** 1,3 ****
--- 1,8 ----
+ 2006-05-23  Michaël Cadilhac  <michael.cadilhac@lrde.org>
+ 
+ 	* dired.c (file_name_completion): Don't barf if the base directory
+ 	can't be opened.
+ 
  2006-05-23  YAMAMOTO Mitsuharu  <mituharu@math.s.chiba-u.ac.jp>
  
  	* macterm.c (fn_keycode_to_xkeysym_table, convert_fn_keycode): Remove.
Index: src/dired.c
===================================================================
RCS file: /sources/emacs/emacs/src/dired.c,v
retrieving revision 1.122
diff -c -r1.122 dired.c
*** src/dired.c	6 Feb 2006 15:23:20 -0000	1.122
--- src/dired.c	23 May 2006 17:13:37 -0000
***************
*** 503,509 ****
  
        d = opendir (SDATA (Fdirectory_file_name (encoded_dir)));
        if (!d)
! 	report_file_error ("Opening directory", Fcons (dirname, Qnil));
  
        record_unwind_protect (directory_files_internal_unwind,
                               make_save_value (d, 0));
--- 503,509 ----
  
        d = opendir (SDATA (Fdirectory_file_name (encoded_dir)));
        if (!d)
! 	break;
  
        record_unwind_protect (directory_files_internal_unwind,
                               make_save_value (d, 0));

[-- Attachment #1.1.3: Type: text/plain, Size: 422 bytes --]


  It works if I do so, but I don't know if I'm not breaking another
  functionality.

  So ?

-- 
 |      Michaël `Micha' Cadilhac   |   Would someone please DTRT with this  |
 |         Epita/LRDE Promo 2007   |         then ACK?                      |
 | http://www.lrde.org/~cadilh_m   |           -- Richard Stallman          |
 `--  -   JID: micha@amessage.be --'                                   -  --'

[-- Attachment #1.2: Type: application/pgp-signature, Size: 188 bytes --]

[-- Attachment #2: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Stefan Monnier]

>   When I do C-x C-f /a/b/c TAB, no star are added, but an _error_ is
>   triggered instead of a [no match].

I'm not sure it's bad, actually.


        Stefan

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Michaël Cadilhac]

[-- Attachment #1.1: Type: text/plain, Size: 1042 bytes --]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>>   When I do C-x C-f /a/b/c TAB, no star are added, but an _error_ is
>>   triggered instead of a [no match].
>
> I'm not sure it's bad, actually.

  Well,  I think  it is.  The behavior  is not  consistent  with other
  failed completions,  and debugger should  not be called on  it (with
  debug-on-error).

  If a message such  as this one has to be printed,  I think it should
  be, IMHO,  in bracket after the  text entered, like  [no match]. But
  sticking  to short  messages should  be  better, and  [no match]  is
  explicit enough.

  What  makes you  think an  error is  not so  bad, although  no other
  completion system triggers one?

-- 
 |      Michaël `Micha' Cadilhac   |   Pour les 35-40 ans, l'humour         |
 |         Epita/LRDE Promo 2007   |        c'est une plus-value.           |
 | http://www.lrde.org/~cadilh_m   |           -- Guillaume L.              |
 `--  -   JID: micha@amessage.be --'                                   -  --'

[-- Attachment #1.2: Type: application/pgp-signature, Size: 188 bytes --]

[-- Attachment #2: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

PC-do-completion triggers an error (was: [Emacs-diffs] Changes to emacs/lisp/ChangeLog)

[Michaël Cadilhac]

[-- Attachment #1.1: Type: text/plain, Size: 1209 bytes --]

michael.cadilhac@lrde.org (Michaël Cadilhac) writes:

> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>>>   When I do C-x C-f /a/b/c TAB, no star are added, but an _error_ is
>>>   triggered instead of a [no match].
>>
>> I'm not sure it's bad, actually.
>
>   Well,  I think  it is.  The behavior  is not  consistent  with other
>   failed completions,  and debugger should  not be called on  it (with
>   debug-on-error).
>
>   If a message such  as this one has to be printed,  I think it should
>   be, IMHO,  in bracket after the  text entered, like  [no match]. But
>   sticking  to short  messages should  be  better, and  [no match]  is
>   explicit enough.
>
>   What  makes you  think an  error is  not so  bad, although  no other
>   completion system triggers one?

Anyone finding that an error should not be triggered and can review my
patch ?

-- 
 |      Michaël `Micha' Cadilhac   |   La culture c'est comme la confiture  |
 |         Epita/LRDE Promo 2007   |       c'est meilleur avec du pain.     |
 | http://www.lrde.org/~cadilh_m   |            -- MOI59                    |
 `--  -   JID: micha@amessage.be --'                                   -  --'

[-- Attachment #1.2: Type: application/pgp-signature, Size: 188 bytes --]

[-- Attachment #2: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Re: PC-do-completion triggers an error

[Michaël Cadilhac]

[-- Attachment #1.1: Type: text/plain, Size: 1310 bytes --]

michael.cadilhac@lrde.org (Michaël Cadilhac) writes:

> michael.cadilhac@lrde.org (Michaël Cadilhac) writes:
>
>> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>
>>>>   When I do C-x C-f /a/b/c TAB, no star are added, but an _error_ is
>>>>   triggered instead of a [no match].
>>>
>>> I'm not sure it's bad, actually.
>>
>>   Well,  I think  it is.  The behavior  is not  consistent  with other
>>   failed completions,  and debugger should  not be called on  it (with
>>   debug-on-error).
>>
>>   If a message such  as this one has to be printed,  I think it should
>>   be, IMHO,  in bracket after the  text entered, like  [no match]. But
>>   sticking  to short  messages should  be  better, and  [no match]  is
>>   explicit enough.
>>
>>   What  makes you  think an  error is  not so  bad, although  no other
>>   completion system triggers one?
>
> Anyone finding that an error should not be triggered and can review my
> patch ?

It's a no? Bah.

-- 
 |      Michaël `Micha' Cadilhac   |  Would someone please DTRT with this,  |
 |         Epita/LRDE Promo 2007   |        then ACK?                       |
 | http://www.lrde.org/~cadilh_m   |          -- Richard Stallman           |
 `--  -   JID: micha@amessage.be --'                                   -  --'

[-- Attachment #1.2: Type: application/pgp-signature, Size: 188 bytes --]

[-- Attachment #2: Type: text/plain, Size: 142 bytes --]

_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Pavel Janík]

[ Redirected to emacs-devel. ]

   From: Richard Stallman <rms@gnu.org>
   Date: Sun, 31 Mar 2002 09:40:15 -0700 (MST)

   >     Richard, play-sound is not guaranteed to exist.
   > 
   > Yes, I saw that.
   > 
   >     So it is not safe to make play-sound-file unconditional (we can remove
   >     XEmacs comment though).
   > 
   > It seems safe to me.  If play-sound is not defined, play-sound-file
   > will get an error trying to call it.  I think that is cleaner behavior
   > than what happens if play-sound-file is not defined at all.

I think it would be much cleaner to:

- Fprovide (intern ("sound"), Qnil) in sound.c when `play-sound' is defined

- use (featurep 'sound) to check if `play-sound' is defined and if it is
  not, provide meaningful error message to the user telling him that his
  Emacs was compiled without sound support

I can do that if you agree with this. I will also add --with[out]-sound to
configure.
-- 
Pavel Janík

panic("Detected a card I can't drive - whoops\n");
                  -- 2.2.16 drivers/net/daynaport.c

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Eli Zaretskii]

On Mon, 1 Apr 2002, Pavel =?iso-8859-2?q?Jan=EDk?= wrote:

> I think it would be much cleaner to:
> 
> - Fprovide (intern ("sound"), Qnil) in sound.c when `play-sound' is defined
> 
> - use (featurep 'sound) to check if `play-sound' is defined and if it is
>   not, provide meaningful error message to the user telling him that his
>   Emacs was compiled without sound support

Alternatively, rename Fplay_sound to Fplay_sound_internal, and provide a 
Lisp function play-sound, defined on all platforms, that displays an 
error message (or maybe silently does nothing) if play-sound-internal is 
not fboundp.

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Stefan Monnier]

> [ Redirected to emacs-devel. ]
> 
>    From: Richard Stallman <rms@gnu.org>
>    Date: Sun, 31 Mar 2002 09:40:15 -0700 (MST)
> 
>    >     Richard, play-sound is not guaranteed to exist.
>    > 
>    > Yes, I saw that.
>    > 
>    >     So it is not safe to make play-sound-file unconditional (we can remove
>    >     XEmacs comment though).
>    > 
>    > It seems safe to me.  If play-sound is not defined, play-sound-file
>    > will get an error trying to call it.  I think that is cleaner behavior
>    > than what happens if play-sound-file is not defined at all.
> 
> I think it would be much cleaner to:
> 
> - Fprovide (intern ("sound"), Qnil) in sound.c when `play-sound' is defined

Why would

	(featurep 'sound)

be any better than

	(fboundp 'play-sound)

??


	Stefan

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Pavel Janík]

   From: "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu>
   Date: Mon, 01 Apr 2002 10:05:53 -0500

   > > I think it would be much cleaner to:
   > > 
   > > - Fprovide (intern ("sound"), Qnil) in sound.c when `play-sound' is defined
   > 
   > Why would
   > 
   > 	(featurep 'sound)
   > 
   > be any better than
   > 
   > 	(fboundp 'play-sound)
   > 
   > ??

I did not said that :-) Yes, it is better. But both ways are better than
the current behavior.
-- 
Pavel Janík

Choose variable names that won't be confused.
                  --  The Elements of Programming Style (Kernighan & Plaugher)

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Richard Stallman]

It might be a good idea to define play-sound so that it always fails
when sound support is not included.  That could be done in C or Lisp.
It is easy either way.  Want to do it?

    I can do that if you agree with this. I will also add --with[out]-sound to
    configure.

Is there a real practical benefit in having those options?  I don't
see it.  Why bother?

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Pavel Janík]

   From: Richard Stallman <rms@gnu.org>
   Date: Mon, 1 Apr 2002 16:21:21 -0700 (MST)

   > It might be a good idea to define play-sound so that it always fails
   > when sound support is not included.  That could be done in C or Lisp.
   > It is easy either way.  Want to do it?

I think that Eli's idea of renaming play-sound (now in C) to
play-sound-internal and just adding

  (unless (fboundp 'play-sound-internal)
     (error "This Emacs binary lacks sound support."))

at the beginning of real play-sound (which can be in Lisp after that) is
also good solution. What do you think?

   >     I can do that if you agree with this. I will also add --with[out]-sound to
   >     configure.
   > 
   > Is there a real practical benefit in having those options?  I don't
   > see it.  Why bother?

To test the above implementation? ;-) No, it does not have practical
benefit except for maniacs who are trying to make Emacs smaller. How do
you compile Emacs without sound support on the computer which has
soundcard.h and similar files? Yes, it is detail, but it will take
approx. 5 minutes of my time.
-- 
Pavel Janík

Use uniform input formats.
                  --  The Elements of Programming Style (Kernighan & Plaugher)

Re: [Emacs-diffs] Changes to emacs/lisp/ChangeLog

[Richard Stallman]

    I think that Eli's idea of renaming play-sound (now in C) to
    play-sound-internal and just adding

      (unless (fboundp 'play-sound-internal)
	 (error "This Emacs binary lacks sound support."))

    at the beginning of real play-sound (which can be in Lisp after that) is
    also good solution. What do you think?

Either way.

       >     I can do that if you agree with this. I will also add --with[out]-sound to
       >     configure.

I won't argue strenuously against it.