Gnus development mailing list
 help / color / mirror / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: emacs-pretest-bug@gnu.org, yamaoka@jpl.org, ding@gnus.org
Subject: Re: Change in bytecomp.el breaks Gnus
Date: Sun, 14 Nov 2004 14:17:08 -0500	[thread overview]
Message-ID: <877joo45bz.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <200411141729.iAEHTSQ24178@raven.dms.auburn.edu> (Luc Teirlinck's message of "Sun, 14 Nov 2004 11:29:28 -0600 (CST)")

> In Emacs, the situation is already hopeless.  I redirected make
> bootstrap's error messages to a file.  That file is 8155 lines long.
> Not all these lines are compiler warnings but a lot of them are.

Elisp compilation warnings are basically useless to anyone else than the
author.  To fix this part is currently hopeless.
To the elisp author, OTOH they can be very handy if she chooses to pay
attention to them.

> `(elisp)Coding Conventions' says to avoid compiler warnings, but many
> authors ignore this.

I think there's a whole range of cases:
1 - the author never byte compiles her files.
2 - she byte-compiles occasionally but just ignores all the warnings.
3 - she'd like to pay attention to the warnings, but there's so many that
    she's discouraged before even starting (anybody who's taken a look at
    Calc knows what I'm talking about).
4 - she'd pays attention to the warnings but several of them can't be fixed
    without ugly workarounds.  For vars the (defvar foo) form is a godsent,
    but for functions, there's nothing equivalent (yet).  The recently
    proposed (defun foo) form would be extremely helpful.
5 - ...

> Most "function from cl package called at runtime" seem to involve
> situations where loading the .el file loads cl, but loading the .elc
> does not.  I believe that this is acceptable, in which case most of
> these warnings are bogus.

Many of those warnings are bogus.  One of the most common case of "bogus"
is when a CL function is used in a macro and this macro is only used inside
the file, so it's only used during byte-compilation and never at runtime.
The warning is "bogus" is the sense that it's harmless but it is real in the
sense that loading the resulting .elc file will define a macro which will
burp if you try to use it without first loading CL.


        Stefan

  reply	other threads:[~2004-11-14 19:17 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <877jouepy8.fsf@telia.com>
     [not found] ` <b9yd5ymkq11.fsf@jpl.org>
2004-11-11  7:40   ` Katsumi Yamaoka
2004-11-11 11:18     ` Katsumi Yamaoka
2004-11-11 17:45       ` Luc Teirlinck
2004-11-11 21:55         ` Stefan
2004-11-11 22:41           ` Luc Teirlinck
2004-11-11 22:54             ` Luc Teirlinck
2004-11-12  0:22           ` Katsumi Yamaoka
2004-11-12  3:57             ` Stefan Monnier
2004-11-12  4:30               ` Katsumi Yamaoka
2004-11-13 23:03                 ` Stefan
2004-11-14  0:14                   ` Katsumi Yamaoka
2004-11-14  5:21                     ` Stefan
2004-11-14  6:10                       ` Katsumi Yamaoka
2004-11-14  6:36                         ` Stefan
2004-11-14 13:44                           ` Katsumi Yamaoka
2004-11-14 17:29                         ` Luc Teirlinck
2004-11-14 19:17                           ` Stefan Monnier [this message]
2004-11-15 14:00                             ` Richard Stallman
2004-11-15 15:36                               ` Stefan Monnier
2004-11-15 23:22                                 ` Miles Bader
2004-11-15 23:28                                   ` Stefan Monnier
2004-11-15 23:38                                     ` Miles Bader
2004-11-15 23:44                                       ` Stefan Monnier
2004-11-15 23:57                               ` Luc Teirlinck
2004-11-16  1:29                                 ` Nick Roberts
2004-11-16  1:49                                   ` Luc Teirlinck
2004-11-16  2:19                                   ` Luc Teirlinck
2004-11-16 16:49                           ` Richard Stallman

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=877joo45bz.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=ding@gnus.org \
    --cc=emacs-pretest-bug@gnu.org \
    --cc=yamaoka@jpl.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).