Gnus development mailing list
 help / color / mirror / Atom feed
From: Katsumi Yamaoka <yamaoka@jpl.org>
To: Glenn Morris <rgm@gnu.org>
Cc: ding@gnus.org, emacs-devel@gnu.org
Subject: Re: Gnus and No Gnus
Date: Tue, 08 Sep 2009 20:13:24 +0900	[thread overview]
Message-ID: <b4mmy55r8ob.fsf@jpl.org> (raw)
In-Reply-To: <1xfxaxlx7n.fsf@fencepost.gnu.org>

>>>>> Glenn Morris wrote:
> Katsumi Yamaoka wrote:

>> Unfortunately `with-no-warnings' is not available in Emacs 21
>> and XEmacs.

> Argh...
> Could you add a compat definition in Gnus? Just an alias to progn
> should be fine.

It will be possible to add a compat definition that replaces
`with-no-warnings' with `progn' when compiling to dgnushack.el
or whatever.  However, it disables people, who use XEmacs or
Emacs 21 to develop Gnus, from loading .el files.  So, it is
better to add it to every file which uses `with-no-warnings'.
Is it ok?

In that case, the definition should be made available only when
compiling (or when loading .el files) because `with-no-warnings'
should never be bound normally in XEmacs and Emacs 21.

(eval-when-compile
  (unless (fboundp 'with-no-warnings)
    (defmacro with-no-warnings (&rest body)
      `(progn ,@body))))

> Otherwise, I'd say it's up to you as to how complicated you are
> willing to make the code just in order to suppress (let's face it,
> harmless) warnings.

Yes, that is for *only* suppressing warnings, in place of
`with-no-warnings'.  Using neither the complicated things nor
`with-no-warnings' makes it simple, if we don't mind warnings
when bootstrapping Emacs.




  reply	other threads:[~2009-09-08 11:13 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-07 12:07 Katsumi Yamaoka
2009-09-07 21:31 ` Glenn Morris
     [not found] ` <bnocpmzbka.fsf@fencepost.gnu.org>
2009-09-08  2:15   ` Stephen J. Turnbull
2009-09-08  4:46     ` Teemu Likonen
2009-09-15  8:10       ` Steinar Bang
2009-09-08  7:11     ` Glenn Morris
2009-09-15  8:13       ` Steinar Bang
2009-09-15 13:10         ` Ted Zlatanov
2009-09-08  5:15   ` Katsumi Yamaoka
2009-09-08  7:19     ` Glenn Morris
2009-09-08 11:13       ` Katsumi Yamaoka [this message]
2009-09-08 16:49         ` Glenn Morris
2009-09-08 19:08           ` Stephen J. Turnbull
2009-09-09  9:31           ` Katsumi Yamaoka
2009-09-15  8:14     ` Steinar Bang
2009-09-15 17:15     ` Reiner Steib

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=b4mmy55r8ob.fsf@jpl.org \
    --to=yamaoka@jpl.org \
    --cc=ding@gnus.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.
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 NNTP newsgroup(s).