unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Reiner Steib'" <Reiner.Steib@gmx.de>, <emacs-devel@gnu.org>
Cc: 'Xavier Maillard' <xma@gnu.org>
Subject: RE: Readings for an emacs package maintainer ?
Date: Mon, 1 Jun 2009 12:11:44 -0700	[thread overview]
Message-ID: <BD64AE52533544D5837B46CB9DAEACC7@us.oracle.com> (raw)
In-Reply-To: <87fxeju6js.fsf@marauder.physik.uni-ulm.de>

> > There are other tips in the Elisp manual, node Coding
> > Conventions. These, for instance:
...
> It's funny that I pointed Xavier to this very node in March 2008 when
> reporting such a bug in color-theme (see https://gna.org/bugs/?9494)
> which overwrites `replace-in-string' in an incompatible way which
> breaks Gnus, Muse, emacs-jabber, etc.  Unfortunately Xavier didn't fix
> this up to now, AFAIK.

Dunno about that particular context, but I like to remember that guidelines are,
well just guidelines. They are meant to help, and they help more if reasons are
provide, so people can understand them.

My feeling is that advice should be freely given, with nothing expected in
return. It is unreasonable to expect someone to follow some advice you give, no
matter how well intentioned you are, and no matter how good the advice might be.
If there is such an expectation, then it is not advice that you give, but
something else.

I myself do redefine some Emacs functions, and I occasionally do use `defadvice'
(though I avoid it). I have my reasons. But I have no problem with someone
giving me advice not to do so. And I agree with the manual's guidelines.

They are only guidelines, however, which means (1) They are likely to be helpful
_in general_. (2) Use your own judgment: use them when you want to; ignore them
when you want to.





  parent reply	other threads:[~2009-06-01 19:11 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-01  7:18 Readings for an emacs package maintainer ? Xavier Maillard
2009-06-01  9:13 ` Lennart Borgman
2009-06-01 16:14   ` Drew Adams
2009-06-01 18:56     ` Reiner Steib
2009-06-01 19:04       ` Lennart Borgman
2009-06-01 19:11       ` Drew Adams [this message]
2009-06-01 12:43 ` Xavier MAILLARD
2009-06-01 14:15   ` Miles Bader
2009-06-02  2:17     ` Stephen J. Turnbull
2009-06-02  2:36       ` Miles Bader
2009-06-02 12:02       ` Deniz Dogan
2009-06-02 14:07         ` Stephen J. Turnbull
2009-06-01 16:15   ` Drew Adams
2009-06-01 16:21 ` Bastien
2009-06-01 17:05   ` Drew Adams
2009-06-01 20:35 ` Stefan Monnier
2009-06-01 22:01   ` Leo

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=BD64AE52533544D5837B46CB9DAEACC7@us.oracle.com \
    --to=drew.adams@oracle.com \
    --cc=Reiner.Steib@gmx.de \
    --cc=emacs-devel@gnu.org \
    --cc=xma@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).