GNU bug report logs - #7539
23.2.90; smie.el uses stuff not available in emacs-23

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 7539 in the body.
You can then email your comments to 7539 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Sven Joachim <svenjoac <at> gmx.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.2.90; smie.el uses stuff not available in emacs-23
Date: Fri, 03 Dec 2010 11:15:15 +0100

Byte-compiling lisp/emacs-lisp/smie.el in emacs-23 gives the following
warnings:

,----
| In smie-setup:
| smie.el:1536:14:Warning: assignment to free variable
|     `blink-matching-check-function'
| 
| In end of data:
| smie.el:1559:1:Warning: the function `blink-matching-check-mismatch' is not
|     known to be defined.
`----

Indeed blink-matching-check-function and blink-matching-check-mismatch
only exist in the trunk, not in emacs-23.

Message #8 received at 7539 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Sven Joachim <svenjoac <at> gmx.de>
Cc: 7539 <at> debbugs.gnu.org
Subject: Re: bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
Date: Fri, 03 Dec 2010 13:55:52 -0500

Sven Joachim wrote:

> Indeed blink-matching-check-function and blink-matching-check-mismatch
> only exist in the trunk, not in emacs-23.

If it is just setting up support code for a feature not present in
emacs-23, then this is not a problem, because if the feature isn't
present, the code won't be called.

Message #11 received at 7539 <at> debbugs.gnu.org (full text, mbox):

From: Sven Joachim <svenjoac <at> gmx.de>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 7539 <at> debbugs.gnu.org
Subject: Re: bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
Date: Fri, 03 Dec 2010 20:18:39 +0100

On 2010-12-03 19:55 +0100, Glenn Morris wrote:

> Sven Joachim wrote:
>
>> Indeed blink-matching-check-function and blink-matching-check-mismatch
>> only exist in the trunk, not in emacs-23.
>
> If it is just setting up support code for a feature not present in
> emacs-23, then this is not a problem, because if the feature isn't
> present, the code won't be called.

I presume that's the case here, since blink-matching-check-mismatch is
only used in the smie-blink-matching-check function which in turn is
used as a local variable for blink-matching-check-function.  So there's
probably not a problem.

Then again, I don't quite understand why smie.el was added to emacs-23
at all, it does not have seem to have any users there (as opposed to the
trunk where several programming modes use it already).

Regards,
        Sven

Message #16 received at 7539-done <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 7539-done <at> debbugs.gnu.org
Subject: Re: bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
Date: Fri, 03 Dec 2010 14:28:12 -0500

Sven Joachim wrote:

> Then again, I don't quite understand why smie.el was added to emacs-23
> at all, it does not have seem to have any users there (as opposed to the
> trunk where several programming modes use it already).

Presumably so that people maintaining programming modes outside Emacs
have the option to start using smie from Emacs 23.3 onwards.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.