emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: locate-with-filter


From: Peter Breton
Subject: Re: locate-with-filter
Date: Mon, 13 Mar 2006 03:30:08 -0800 (PST)

I agree with Eli on this one.

I did like all Luc's other changes tho.

Peter

> > Date: Sun, 12 Mar 2006 18:18:05 -0600 (CST)
> > From: Luc Teirlinck <address@hidden>
> > Cc: address@hidden, address@hidden
> > 
> >   ;;;###autoload
> >   (defun locate (search-string &optional filter)
> > !   "Run the program `locate', putting results in
> `*Locate*' buffer.
> >   With prefix arg, prompt for the locate command
> to run."
> >     (interactive
> >         (list
> > --- 190,196 ----
> >   
> >   ;;;###autoload
> >   (defun locate (search-string &optional filter)
> > !   "Run the locate program, putting results in
> `*Locate*' buffer.
> >   With prefix arg, prompt for the locate command
> to run."
> 
> This change is for the worse, IMHO.  The original
> doc string makes it
> clear that `locate' is a program, whereas the new
> strings could be
> misunderstood to talk about some mysterious ``locate
> program'' (as in
> ``relocation program'').
> 
> There are more instances of using ``the locate
> program'' in your
> changes; I think they all need to be modified in the
> same way to make
> sure the meaning is clear.
> 





reply via email to

[Prev in Thread] Current Thread [Next in Thread]