all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: trunk r112976: * lisp/subr.el (with-eval-after-load): New
@ 2013-06-15  3:23 Xue Fuqiao
  2013-06-15 14:18 ` Stefan Monnier
  0 siblings, 1 reply; 3+ messages in thread
From: Xue Fuqiao @ 2013-06-15  3:23 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier wrote:

>  * doc/lispref/loading.texi (Hooks for Loading): Document with-eval-after-load
>  instead of eval-after-load.  Don't document after-load-alist.

I think loading.texi needs some further changes.  The last paragraph
("Normally, well-designed Lisp programs should not use
`eval-after-load'...") is superfluous.  My reasons:

1. `eval-after-load' isn't documented any more.  This paragraph is
somewhat suprising to readers who did not know this function before;

2. (info "(elisp) Coding Conventions") has already stated that the use
of `eval-after-load' isn't recommended.

--
Best regards, Xue Fuqiao.
http://www.gnu.org/software/emacs/



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: trunk r112976: * lisp/subr.el (with-eval-after-load): New
  2013-06-15  3:23 trunk r112976: * lisp/subr.el (with-eval-after-load): New Xue Fuqiao
@ 2013-06-15 14:18 ` Stefan Monnier
  2013-06-18 16:49   ` Michael Heerdegen
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Monnier @ 2013-06-15 14:18 UTC (permalink / raw)
  To: Xue Fuqiao; +Cc: emacs-devel

> I think loading.texi needs some further changes.  The last paragraph
> ("Normally, well-designed Lisp programs should not use
> `eval-after-load'...") is superfluous.  My reasons:

> 1. `eval-after-load' isn't documented any more.  This paragraph is
> somewhat suprising to readers who did not know this function before;

The wording should be made more general so as to include eval-after-load
and with-eval-after-load (and eval-next-after-load).

> 2. (info "(elisp) Coding Conventions") has already stated that the use
> of `eval-after-load' isn't recommended.

Repetition is not bad,


        Stefan



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: trunk r112976: * lisp/subr.el (with-eval-after-load): New
  2013-06-15 14:18 ` Stefan Monnier
@ 2013-06-18 16:49   ` Michael Heerdegen
  0 siblings, 0 replies; 3+ messages in thread
From: Michael Heerdegen @ 2013-06-18 16:49 UTC (permalink / raw)
  To: emacs-devel; +Cc: Xue Fuqiao

Hello,

Xue, I guess you should make a bug report, so that this isn't forgotten.

emacs-dev is a good forum for discussion, however, without making a bug
report, things get stalled most of the time, because it's on nobody's
todo list.


Thanks,

Michael.

> > 1. `eval-after-load' isn't documented any more.  This paragraph is
> > somewhat suprising to readers who did not know this function before;
>
> The wording should be made more general so as to include eval-after-load
> and with-eval-after-load (and eval-next-after-load).
>
> > 2. (info "(elisp) Coding Conventions") has already stated that the use
> > of `eval-after-load' isn't recommended.
>
> Repetition is not bad,
>
>
>         Stefan




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-06-18 16:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-15  3:23 trunk r112976: * lisp/subr.el (with-eval-after-load): New Xue Fuqiao
2013-06-15 14:18 ` Stefan Monnier
2013-06-18 16:49   ` Michael Heerdegen

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.