[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#25077: 25.1; Incorrect Edebug spec for c-lang-defvar
From: |
Alan Mackenzie |
Subject: |
bug#25077: 25.1; Incorrect Edebug spec for c-lang-defvar |
Date: |
Sat, 4 Feb 2017 16:23:07 +0000 |
User-agent: |
Mutt/1.7.2 (2016-11-26) |
Hello, Eli.
On Sat, Feb 04, 2017 at 13:27:45 +0200, Eli Zaretskii wrote:
> > From: Gemini Lasswell <gazally@runbox.com>
> > Date: Wed, 30 Nov 2016 11:20:28 -0800
> >
> > The Edebug spec for c-lang-defvar doesn't allow for the use of
> > 'dont-doc, which prevents the use of Edebug and Testcover on
> > cc-langs.el.
> >
> > To reproduce, using emacs -Q starting at the root of the Emacs
> > sources:
> >
> > M-: (require 'cc-langs) RET
> > C-x C-f lisp/progmodes/cc-langs.el RET
> > C-s dont-doc RET
> > C-u C-M-x
> >
> > Result: edebug-syntax-error: Invalid read syntax: stringp, "failed"
> >
> > I tried this change to the Edebug spec for c-lang-defvar, and it makes
> > Edebug work on all the c-lang-defvar uses in this file:
> >
> > (def-edebug-spec c-lang-defvar
> > (&define name def-form &optional &or ("quote" "dont-doc") stringp))
> >
> > Alternatively, the 'dont-doc could be removed from the 7 places it is
> > used. The majority of c-lang-defvar calls simply omit the docstring,
> > and the code in c-lang-defvar is not doing any checking for dont-doc,
> > it simply ignores that parameter if it is not a string.
> Alan, could you please look into this?
Yes, I will.
> Thanks.
--
Alan Mackenzie (Nuremberg, Germany).