unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el
@ 2011-06-13 22:31 Drew Adams
  2011-07-01 14:17 ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Drew Adams @ 2011-06-13 22:31 UTC (permalink / raw)
  To: 8858

I know nothing about eieio.  I have no idea whether this is actually a
bug, but it seemed odd.  There are duplicate defaliases in these two
files.  Should there be?  HTH.
 

In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600)
 of 2011-06-13 on 3249CTO
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (4.5) --no-opt --cflags
-Ic:/build/include'
 






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

* bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el
  2011-06-13 22:31 bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el Drew Adams
@ 2011-07-01 14:17 ` Lars Magne Ingebrigtsen
  2011-07-01 16:50   ` Andreas Schwab
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-01 14:17 UTC (permalink / raw)
  To: Drew Adams; +Cc: 8858, Eric M. Ludlam

"Drew Adams" <drew.adams@oracle.com> writes:

> I know nothing about eieio.  I have no idea whether this is actually a
> bug, but it seemed odd.  There are duplicate defaliases in these two
> files.  Should there be?  HTH.

The autoloads are apparently automatically extracted from the other
files, and put into eieio.el. This seems unnecessary, since Emacs itself
will extract the autoloads.

Example:

;;;### (autoloads (eieio-help-mode-augmentation-maybee eieio-describe-generic
;;;;;;  eieio-describe-constructor eieio-describe-class eieio-browse)
;;;;;;  "eieio-opt" "eieio-opt.el" "1bed0a56310f402683419139ebc18d7f")
;;; Generated autoloads from eieio-opt.el

(autoload 'eieio-browse "eieio-opt" "\
Create an object browser window to show all objects.
If optional ROOT-CLASS, then start with that, otherwise start with
variable `eieio-default-superclass'.

\(fn &optional ROOT-CLASS)" t nil)


-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/





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

* bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el
  2011-07-01 14:17 ` Lars Magne Ingebrigtsen
@ 2011-07-01 16:50   ` Andreas Schwab
  2011-07-01 16:59     ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Andreas Schwab @ 2011-07-01 16:50 UTC (permalink / raw)
  To: Lars Magne Ingebrigtsen; +Cc: 8858, Eric M. Ludlam

Lars Magne Ingebrigtsen <larsi@gnus.org> writes:

> The autoloads are apparently automatically extracted from the other
> files, and put into eieio.el. This seems unnecessary, since Emacs itself
> will extract the autoloads.

Yes, it does, and puts them into eieio.el.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."





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

* bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el
  2011-07-01 16:50   ` Andreas Schwab
@ 2011-07-01 16:59     ` Lars Magne Ingebrigtsen
  2011-07-01 21:10       ` Glenn Morris
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-01 16:59 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: 8858, Eric M. Ludlam

Andreas Schwab <schwab@linux-m68k.org> writes:

> Yes, it does, and puts them into eieio.el.

Oh, I though it put them into auto-autoloads.el or something.

Then I guess this isn't a bug?

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/





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

* bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el
  2011-07-01 16:59     ` Lars Magne Ingebrigtsen
@ 2011-07-01 21:10       ` Glenn Morris
  2011-07-02 12:30         ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Glenn Morris @ 2011-07-01 21:10 UTC (permalink / raw)
  To: Lars Magne Ingebrigtsen; +Cc: 8858

Lars Magne Ingebrigtsen wrote:

> Andreas Schwab <schwab@linux-m68k.org> writes:
>
>> Yes, it does, and puts them into eieio.el.
>
> Oh, I though it put them into auto-autoloads.el or something.

See file-local setting of generated-autoload-file (it's a nice feature
if you have main.el and helper*.el and the way to use the package is to
load main.el).

> Then I guess this isn't a bug?

No. But since eieio-opt requires eieio, the eieio-opt autoloads could be
changed to:

;;;###autoload(defalias 'describe-class 'eieio-describe-class)





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

* bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el
  2011-07-01 21:10       ` Glenn Morris
@ 2011-07-02 12:30         ` Lars Magne Ingebrigtsen
  0 siblings, 0 replies; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 2011-07-02 12:30 UTC (permalink / raw)
  To: Glenn Morris; +Cc: 8858

Glenn Morris <rgm@gnu.org> writes:

> See file-local setting of generated-autoload-file (it's a nice feature
> if you have main.el and helper*.el and the way to use the package is to
> load main.el).

Right; I see.

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/





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

end of thread, other threads:[~2011-07-02 12:30 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-13 22:31 bug#8858: 24.0.50; duplicate defaliases in eieio.el and eieio-opt.el Drew Adams
2011-07-01 14:17 ` Lars Magne Ingebrigtsen
2011-07-01 16:50   ` Andreas Schwab
2011-07-01 16:59     ` Lars Magne Ingebrigtsen
2011-07-01 21:10       ` Glenn Morris
2011-07-02 12:30         ` Lars Magne Ingebrigtsen

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).