GNU bug report logs - #16980
eieio-generic-call shouldn't load libraries

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 16980 in the body.
You can then email your comments to 16980 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jonas Bernoulli <jonas <at> bernoul.li>
To: bug-gnu-emacs <at> gnu.org
Subject: eieio-generic-call shouldn't load libraries
Date: Mon, 10 Mar 2014 19:35:28 +0100

eieio-generic-call loads the respective library when the first method
argument is a symbol which also happens to be the name of an autoloaded
function.  This is justifies like this:

    Is the class passed in autoloaded?
    Since class names are also constructors, they can be autoloaded
    via the autoload command.  Check for this, and load them in.
    It is ok if it doesn't turn out to be a class.  Probably want that
    function loaded anyway.

That assumption isn't always true.  E.g. I have written a little tool
that deals with Elisp libraries.  Some of methods are not only
implemented for objects that represent libraries but also for symbols
which stand for such libraries.  I was very surprised when calling these
methods started to load libraries.

  Best regards,
  Jonas

Message #8 received at 16980 <at> debbugs.gnu.org (full text, mbox):

From: Jonas Bernoulli <jonas <at> bernoul.li>
To: 16980 <at> debbugs.gnu.org
Subject: Re: eieio-generic-call shouldn't load libraries
Date: Sun, 16 Aug 2020 21:40:59 +0200

Jonas Bernoulli <jonas <at> bernoul.li> writes:

> eieio-generic-call loads the respective library when the first method
> argument is a symbol which also happens to be the name of an autoloaded
> function.

This function does not exist anymore and this behavior does not appear
to have survived all the major refactorings.  I also tried to reproduce
the old behavior, but that too indicates that the issue is gone:

,----
| (cl-defmethod testibus ((arg (eql battery)))
|   arg)
| 
| (list (featurep 'battery)
|       (testibus 'battery)
|       (featurep 'battery)
|       (require  'battery))
| => (nil battery nil battery)
`----

So this can be close.  Thanks.

Message #11 received at 16980 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Jonas Bernoulli <jonas <at> bernoul.li>
Cc: 16980 <at> debbugs.gnu.org
Subject: Re: bug#16980: eieio-generic-call shouldn't load libraries
Date: Sun, 16 Aug 2020 21:54:01 +0200

Jonas Bernoulli <jonas <at> bernoul.li> writes:

> So this can be close.  Thanks.

OK; closed.

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

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.