From: Richard Stallman <rms@gnu.org>
To: Glenn Morris <rgm@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: Problems setting byte-compile-warnings to t
Date: Fri, 26 Oct 2007 22:28:53 -0400 [thread overview]
Message-ID: <E1IlbPx-0008VA-BD@fencepost.gnu.org> (raw)
In-Reply-To: <9p8x5qiwik.fsf@fencepost.gnu.org> (message from Glenn Morris on Thu, 25 Oct 2007 19:50:59 -0400)
Looking at it some more, the value of byte-compile-warnings set by
byte-compile-close-variables gets clobbered when
byte-compile-insert-header is called. It changes the buffer to
" *Compiler Input", where the buffer local value shadows the value in
the let-binding.
It is true that it changes the current buffer, and that is ugly.
Perhaps we should make it use with-current-buffer.
But as soon as it returns, you get to
(with-current-buffer inbuffer
...)
followed by
(and filename (byte-compile-fix-header filename inbuffer outbuffer))))
and that function does with-current-buffer too.
So where is the code that is affected by the change in the current
buffer made by byte-compile-insert-header? Is it in the caller of
byte-compile-from-buffer? I don't see how that could happen, since
the byte-compile-close-variables form has already exited there.
Is it inside byte-compile-insert-header itself?
I don't see where in that function it could be.
next prev parent reply other threads:[~2007-10-27 2:28 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-25 22:42 Problems setting byte-compile-warnings to t Glenn Morris
2007-10-25 23:50 ` Glenn Morris
2007-10-27 2:28 ` Richard Stallman [this message]
2007-10-27 3:36 ` Glenn Morris
2007-10-28 13:51 ` Richard Stallman
2007-10-28 23:58 ` Glenn Morris
2007-10-29 9:22 ` Richard Stallman
2007-10-29 18:39 ` Dan Nicolaescu
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=E1IlbPx-0008VA-BD@fencepost.gnu.org \
--to=rms@gnu.org \
--cc=emacs-devel@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.