unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#29585: 26.0.90; Elint gives spurious warning with condition-case-unless-debug
@ 2017-12-05 22:26 John Wiegley
  2019-07-14 18:24 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 2+ messages in thread
From: John Wiegley @ 2017-12-05 22:26 UTC (permalink / raw)
  To: 29585

Wherever condition-case-unless-debug is used, elint will complain with:

use-package-core.el:1292:Warning: Not an error symbol in error handler: debug

This is due to the definition of condition-case-unless-debug:

--8<---------------cut here---------------start------------->8---
(defmacro condition-case-unless-debug (var bodyform &rest handlers)
  "Like `condition-case' except that it does not prevent debugging.
More specifically if `debug-on-error' is set then the debugger will be invoked
even if this catches the signal."
  (declare (debug condition-case) (indent 2))
  `(condition-case ,var
       ,bodyform
     ,@(mapcar (lambda (handler)
                 `((debug ,@(if (listp (car handler)) (car handler)
                              (list (car handler))))
                   ,@(cdr handler)))
               handlers)))
--8<---------------cut here---------------end--------------->8---

Maybe elint needs to be taught about this case?

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2





^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#29585: 26.0.90; Elint gives spurious warning with condition-case-unless-debug
  2017-12-05 22:26 bug#29585: 26.0.90; Elint gives spurious warning with condition-case-unless-debug John Wiegley
@ 2019-07-14 18:24 ` Lars Ingebrigtsen
  0 siblings, 0 replies; 2+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-14 18:24 UTC (permalink / raw)
  To: 29585

"John Wiegley" <johnw@gnu.org> writes:

> Wherever condition-case-unless-debug is used, elint will complain with:
>
> use-package-core.el:1292:Warning: Not an error symbol in error handler: debug
>
> This is due to the definition of condition-case-unless-debug:

I've now added the macro to the elint definitions.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-07-14 18:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-05 22:26 bug#29585: 26.0.90; Elint gives spurious warning with condition-case-unless-debug John Wiegley
2019-07-14 18:24 ` Lars Ingebrigtsen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).