[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#24762: 26.0.50; Spurious &optional in lisp/progmodes/python.el cause
From: |
Mark Oteiza |
Subject: |
bug#24762: 26.0.50; Spurious &optional in lisp/progmodes/python.el causes Edebug error |
Date: |
Wed, 16 Nov 2016 14:41:19 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
Gemini Lasswell <gazally@runbox.com> writes:
> There's an &optional followed by &rest instead of a parameter name in
> python-define-auxiliary-skeleton in lisp/progmodes/python.el. This
> doesn't work with defmacro's Edebug spec and prevents use of Edebug on
> the macro and testcover-start on the file.
>
> To repro:
> 1. start Emacs at the top of its source tree, with -Q if you like
> 2. C-x C-f lisp/progmodes/python.el RET
> 3. C-s &optional &r RET
> 4. C-u C-M-x
>
> Result: Invalid read syntax: "Bad argument:", &rest
Thanks for reporting--it looks to me like the following fixes:
diff --git a/lisp/progmodes/python.el b/lisp/progmodes/python.el
index f9b28c3..9cd9cc8 100644
--- a/lisp/progmodes/python.el
+++ b/lisp/progmodes/python.el
@@ -4040,7 +4040,7 @@ 'python-mode-abbrev-table
"Abbrev table for Python mode."
:parents (list python-mode-skeleton-abbrev-table))
-(defmacro python-define-auxiliary-skeleton (name doc &optional &rest skel)
+(defmacro python-define-auxiliary-skeleton (name &optional doc &rest skel)
"Define a `python-mode' auxiliary skeleton using NAME DOC and SKEL.
The skeleton will be bound to python-skeleton-NAME."
(declare (indent 2))
@@ -4060,11 +4060,11 @@ python-define-auxiliary-skeleton
(signal 'quit t))
,@skel)))
-(python-define-auxiliary-skeleton else nil)
+(python-define-auxiliary-skeleton else)
-(python-define-auxiliary-skeleton except nil)
+(python-define-auxiliary-skeleton except)
-(python-define-auxiliary-skeleton finally nil)
+(python-define-auxiliary-skeleton finally)
(python-skeleton-define if nil
"Condition: "
- bug#24762: 26.0.50; Spurious &optional in lisp/progmodes/python.el causes Edebug error,
Mark Oteiza <=