From: Alan Mackenzie <acm@muc.de>
To: Glenn Morris <rgm@gnu.org>
Cc: 9448@debbugs.gnu.org
Subject: bug#9448: 24.0.50; Comments in Awk scripts not fontified with comment face
Date: Thu, 8 Sep 2011 21:13:48 +0000 [thread overview]
Message-ID: <20110908211348.GA2338@acm.acm> (raw)
In-Reply-To: <bc39g6op93.fsf@fencepost.gnu.org>
On Thu, Sep 08, 2011 at 02:36:24PM -0400, Glenn Morris wrote:
> Alan Mackenzie wrote:
> > (defvar awk-mode-syntax-table)
> > . This ought to prevent my solution above from working, but it
> > doesn't. Glenn, I think you put this line in. Why? Would it be OK to
> > take it out again?
> A defvar with no initvalue just suppresses a byte-compiler warning about
> said variable being undefined. It doesn't actually define the variable,
> ...
Well, you learn something new every day. :-) I'd thought that (defvar
foo nil) and (defvar foo) were the same. They're not.
> so I think that line should stay there, to stop the compiler complaining
> about an undefined variable in
> (set-syntax-table awk-mode-syntax-table)
OK.
> (So by the way I think this means cc-bytecomp-defvar is no longer
> needed. Also cc-bytecomp-defun can be replaced by declare-function.)
There's the rub. The more such "tidying up" changes are made in
savannah, the more difficult it becomes for me to synch changes with the
upstream CC Mode. There're already > 80 such differences in cc-mode.el
alone.
> > Glenn, what on earth brought you to try deleting cc-fonts.elc? It seems
> > a stroke of genius, but how did you come to consider it? :-)
> I saw from the ChangeLog that cc-fonts was the most recent part of
> cc-mode to be changed, so I tried reverting that commit. To make it take
> effect, I deleted the old cc-fonts.elc rather than recompiling it.
And the irony is, AWK code doesn't use cc-fonts.elc for its
fontification; it uses a simple regexp based model. ;-)
--
Alan Mackenzie (Nuremberg, Germany).
next prev parent reply other threads:[~2011-09-08 21:13 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-06 4:27 bug#9448: 24.0.50; Comments in Awk scripts not fontified with comment face Eli Zaretskii
2011-09-07 7:51 ` Glenn Morris
2011-09-07 16:30 ` Eli Zaretskii
2011-09-07 17:01 ` bug#9448: asdf Alan Mackenzie
2011-09-07 18:14 ` bug#9448: 24.0.50; Comments in Awk scripts not fontified with comment face Alan Mackenzie
2011-09-08 2:12 ` Stefan Monnier
2011-09-08 6:16 ` Glenn Morris
2011-09-08 8:33 ` Alan Mackenzie
2011-09-08 18:36 ` Glenn Morris
2011-09-08 21:13 ` Alan Mackenzie [this message]
2011-09-09 7:01 ` Eli Zaretskii
2011-09-09 9:38 ` Alan Mackenzie
2011-09-09 10:02 ` Alan Mackenzie
2011-09-07 19:17 ` Glenn Morris
2011-09-08 21:50 ` Glenn Morris
2011-09-08 22:13 ` Alan Mackenzie
2011-09-09 2:27 ` Stefan Monnier
2011-09-09 7:04 ` Eli Zaretskii
2011-09-09 7:12 ` Glenn Morris
2011-09-09 7:19 ` Glenn Morris
2011-09-09 3:44 ` Glenn Morris
2011-09-09 9:56 ` bug#9448: Bug fixed Alan Mackenzie
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=20110908211348.GA2338@acm.acm \
--to=acm@muc.de \
--cc=9448@debbugs.gnu.org \
--cc=rgm@gnu.org \
/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.