unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juanma Barranquero <lektu@terra.es>
Cc: emacs-devel@gnu.org
Subject: Re: bytecomp warning for CL functions
Date: Fri, 19 Jul 2002 08:09:22 +0200	[thread overview]
Message-ID: <20020719075811.54D9.LEKTU@terra.es> (raw)
In-Reply-To: <rzqfzygx46x.fsf@albion.dl.ac.uk>

On 18 Jul 2002 19:48:38 +0100, Dave Love <d.love@dl.ac.uk> wrote:

> Comments:
> 
> 1. The warning is being issued correctly.

Perhaps, but if defadvice has been patched to stop it, that must mean
that it was useless and annoying nonetheless.

> 2. It's not a good idea to load cl in .emacs -- you can fail to spot
>    problems, apart from the bloat.

Oh, come on. I happen to *like* cl and don't consider it a bloat.
Obviously it is a matter of taste, but I feel vaguely offensive that you
answer to a comment about a newly introduced warning saying what I
should do with my .emacs.

As two recent posters on gnu.emacs.help have said:

"You don't need (require 'cl) though I expect like me you have it in
your .emacs file somewhere so you never quite know whether you need it
or not." (Rob Thorpe)

"The proper place for '(require 'cl)' is in your .emacs file, at the top
level :)" (Marco Antoniotti)

> 3. You can turn off implicit compilation by defadvice.

Yes.

> 4. defadvice should normally be avoided, apart from the bloat.

Again, I don't consider it bloat. And I don't know why should I avoid it.
AFAICS there's a policy of avoiding it in modules, but for my own
customizations? What is advice.el intended for, if not that?

> If a modification you want to make with it is useful, it normally
> indicates the lack of a hook.  I'd just copy the function and
> modify it to avoid sucking in advice in a case like that.

Are you suggesting that would be better to copy the eight functions I'm
currently advising to my .emacs and then manually monitoring if they
ever change in their original sources?

Excuse me, but I think you're taking your likes or dislikes as a measure
of what's good for another user...


                                                           /L/e/k/t/u

  reply	other threads:[~2002-07-19  6:09 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <rzqhejse35l.fsf@albion.dl.ac.uk>
2002-06-25 23:32 ` bytecomp warning for CL functions Richard Stallman
2002-06-26 14:10   ` Stefan Monnier
2002-07-01 18:11     ` Dave Love
2002-07-02 19:45       ` Richard Stallman
2002-07-03  7:21         ` Juanma Barranquero
2002-07-08 18:20           ` Richard Stallman
2002-07-18 18:48           ` Dave Love
2002-07-19  6:09             ` Juanma Barranquero [this message]
2002-07-19 16:54             ` Richard Stallman
2002-07-08 18:19     ` Richard Stallman
2002-07-18 18:53       ` Dave Love
2002-07-19 16:54         ` Richard Stallman
2002-07-19 17:49           ` Stefan Monnier
2002-07-20  0:35             ` Richard Stallman
2002-07-29 22:46           ` Dave Love
2002-07-30 18:46             ` Richard Stallman
2002-07-08 18:20     ` Richard Stallman
2002-07-08 19:30       ` Luc Teirlinck
2002-07-09 18:51         ` Richard Stallman
2002-07-18 18:34         ` Dave Love
2002-07-19 16:54           ` Richard Stallman
2002-07-22 22:15             ` Dave Love

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20020719075811.54D9.LEKTU@terra.es \
    --to=lektu@terra.es \
    --cc=emacs-devel@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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).