bug#5376: 23.1; dirtrack.el unnecessary errors [PATCH]

[Nathaniel Flath <flat0103@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1005 bytes --]

That may be better, yes - the only isue with that is when large chunks of
input are messaged due to a improper match, which could still be annoying.
However, that is probably fine, and would let the user know if they were
just starting with dirtrack.el that their regex was incorrect.

On Thu, Jan 14, 2010 at 10:04 AM, Stefan Monnier
<monnier@iro.umontreal.ca>wrote:

> > RCS file: /sources/emacs/emacs/lisp/dirtrack.el,v
> > retrieving revision 1.28
> > diff -u -r1.28 dirtrack.el
> > @@ -270,7 +270,7 @@
> >                         (run-hooks 'dirtrack-directory-change-hook)
> >                         (dirtrack-debug-message
> >                          (format "Changing directory to %s"
> prompt-path)))
> > -                (error "Directory %s does not exist" prompt-path)))
> > +                input))
> >              )))))
> >    input)
>
> I agree it shouldn't signal an error.  But we turn the `error' into
> a `message', so the user is kept informed of the problem.
>
>
>        Stefan
>

[-- Attachment #2: Type: text/html, Size: 1422 bytes --]