[debbugs-tracker] bug#34104: closed (26.1; Inaccurate error report of (sort "cba" #'<))

[GNU bug Tracking System]

Your message dated Sun, 20 Jan 2019 00:05:04 +0100
with message-id <address@hidden>
and subject line Re: bug#34104: 26.1; Inaccurate error report of (sort "cba" 
#'<)
has caused the debbugs.gnu.org bug report #34104,
regarding 26.1; Inaccurate error report of (sort "cba" #'<)
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
34104: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=34104
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 26.1; Inaccurate error report of (sort "cba" #'<)
>
>
>
>
> Date: 
>
> Wed, 16 Jan 2019 22:53:19 +0800
>
>
>
>
> Eval (sort "cba" #'<) gives
>
> Debugger entered--Lisp error: (wrong-type-argument sequencep "cba")
>   sort("cba" <)
>   eval((sort "cba" (function <)) nil)
>   elisp--eval-last-sexp(nil)
>   eval-last-sexp(nil)
>   funcall-interactively(eval-last-sexp nil)
>   call-interactively(eval-last-sexp nil nil)
>   command-execute(eval-last-sexp)
>
> the error is inaccurate since "cba" is indeed a sequence
>
> (sequencep "cba")
> ;; => t
>
> it should report "cba" is not a list or vector instead.
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#34104: 26.1; Inaccurate error report of (sort "cba" #'<)
>
>
>
>
> Date: 
>
> Sun, 20 Jan 2019 00:05:04 +0100
>
>
>
>
> Am Fr., 18. Jan. 2019 um 09:31 Uhr schrieb Eli Zaretskii <address@hidden>:
> >
> > > From: Philipp Stephani <address@hidden>
> > > Date: Thu, 17 Jan 2019 23:43:16 +0100
> > > Cc: address@hidden
> > >
> > > > Debugger entered--Lisp error: (wrong-type-argument sequencep "cba")
> > > >   sort("cba" <)
> > > >   eval((sort "cba" (function <)) nil)
> > > >   elisp--eval-last-sexp(nil)
> > > >   eval-last-sexp(nil)
> > > >   funcall-interactively(eval-last-sexp nil)
> > > >   call-interactively(eval-last-sexp nil nil)
> > > >   command-execute(eval-last-sexp)
> > > >
> > > > the error is inaccurate since "cba" is indeed a sequence
> > > >
> > > > (sequencep "cba")
> > > > ;; => t
> > > >
> > > > it should report "cba" is not a list or vector instead.
> > > >
> > >
> > >
> > > Here's a patch.
> >
> > LGTM, thanks.
>
> Pushed as 551051596f.
>
>
> --- End Message ---