unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
@ 2010-12-03 10:15 Sven Joachim
  2010-12-03 18:55 ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Sven Joachim @ 2010-12-03 10:15 UTC (permalink / raw)
  To: 7539; +Cc: Sven Joachim


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.





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

* bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
  2010-12-03 10:15 bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23 Sven Joachim
@ 2010-12-03 18:55 ` Glenn Morris
  2010-12-03 19:18   ` Sven Joachim
  0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2010-12-03 18:55 UTC (permalink / raw)
  To: Sven Joachim; +Cc: 7539

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.






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

* bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
  2010-12-03 18:55 ` Glenn Morris
@ 2010-12-03 19:18   ` Sven Joachim
  2010-12-03 19:28     ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Sven Joachim @ 2010-12-03 19:18 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 7539

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





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

* bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23
  2010-12-03 19:18   ` Sven Joachim
@ 2010-12-03 19:28     ` Glenn Morris
  0 siblings, 0 replies; 4+ messages in thread
From: Glenn Morris @ 2010-12-03 19:28 UTC (permalink / raw)
  To: 7539-done

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.





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

end of thread, other threads:[~2010-12-03 19:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-03 10:15 bug#7539: 23.2.90; smie.el uses stuff not available in emacs-23 Sven Joachim
2010-12-03 18:55 ` Glenn Morris
2010-12-03 19:18   ` Sven Joachim
2010-12-03 19:28     ` Glenn Morris

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).