From: Andreas Schwab <schwab@suse.de>
Cc: "Kim F.Storm" <storm@cua.dk>
Subject: Re: Lisp parser broken
Date: Tue, 18 Feb 2003 16:56:29 +0100 [thread overview]
Message-ID: <jevfzhh9aq.fsf@sykes.suse.de> (raw)
In-Reply-To: <20030218164829.32A7.LEKTU@terra.es> (Juanma Barranquero's message of "Tue, 18 Feb 2003 16:50:06 +0100")
Juanma Barranquero <lektu@terra.es> writes:
|> On Tue, 18 Feb 2003 16:01:45 +0100, Andreas Schwab <schwab@suse.de> wrote:
|>
|> > The latest changes in the lisp parser cause cc-vars.el to be misparsed, in
|> > particular the body of the defmacro defcustom-c-stylevar. Previously the
|> > expression `(a `(` a)) was parsed as (\` (a (\` (\` a)))), now it is
|> > parsed as (\` (a (\` ( (\` a))))). Note that there is a symbol with an
|> > empty name after the fourth open paren.
|>
|> I've checked in patches, including one for cc-vars.el, that allow Emacs
|> to bootstrap. Not sure mine's the right fix for `defcustom-c-stylevar',
|> though.
I still think the bug is in the parser. The symbol with the empty name
must not be there. I'll check in the following change shortly if nobody
comes up with a better fix:
2003-02-18 Andreas Schwab <schwab@suse.de>
* lread.c (read1): Don't forbid "`" and "," in a symbol.
--- src/lread.c.~1.307.~ 2003-02-18 13:07:17.000000000 +0100
+++ src/lread.c 2003-02-18 16:08:19.000000000 +0100
@@ -2458,8 +2458,7 @@ read1 (readcharfun, pch, first_in_list)
char *end = read_buffer + read_buffer_size;
while (c > 040
- && !index ("\"'`;()[]#", c)
- && !(new_backquote_flag && c == ','))
+ && !index ("\"';()[]#", c))
{
if (end - p < MAX_MULTIBYTE_LENGTH)
{
Andreas.
--
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
next prev parent reply other threads:[~2003-02-18 15:56 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-02-18 15:01 Lisp parser broken Andreas Schwab
2003-02-18 15:50 ` Juanma Barranquero
2003-02-18 15:56 ` Andreas Schwab [this message]
2003-02-18 16:22 ` Juanma Barranquero
2003-02-18 19:36 ` Stefan Monnier
2003-02-18 23:49 ` Kim F. Storm
2003-02-18 17:11 ` Kim F. Storm
2003-02-19 7:17 ` Richard Stallman
2003-02-19 9:30 ` Kim F. Storm
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=jevfzhh9aq.fsf@sykes.suse.de \
--to=schwab@suse.de \
--cc=storm@cua.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.