GNU bug report logs - #22243
eieio-opt: not quite (?) optional functions

Previous Next

To reply to this bug, email your comments to 22243 AT debbugs.gnu.org.

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

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

From: Ivan Shmakov <ivan <at> siamics.net>
To: submit <at> debbugs.gnu.org
Subject: eieio-opt: not quite (?) optional functions 
Date: Sat, 26 Dec 2015 13:00:45 +0000

Package:  emacs
Severity: wishlist

	eieio.el currently (1dcf9a5d2a72; 2015-12-22 17:02:30 UTC)
	reads:

   943	;; Hook ourselves into help system for describing classes and methods.
   944	;; FIXME: This is not actually needed any more since we can click on the
   945	;; hyperlink from the constructor's docstring to see the type definition.
   946	(add-hook 'help-fns-describe-function-functions 'eieio-help-constructor)

	This results that with -Q, 'eieio-opt gets loaded immediately
	once the user tries to use the Emacs help system (say, C-h f),
	making the file’s description (below) somewhat misleading, as
	the library which is hooked into such a basic Emacs facility
	could hardly be called “optional”.

;;; eieio-opt.el -- eieio optional functions (debug, printing, speedbar)

	I don’t think I deal much with EIEIO objects, but I’ve removed
	the function from the hook (below) quite some time ago and seen
	no adverse effects so far.  Hence, I guess the add-hook above
	may be reconsidered (per its own FIXME notice.)

(remove-hook 'help-fns-describe-function-functions
             'eieio-help-constructor)

	As an aside, I’ve spotted this issue because eieio-opt depends
	on 'speedbar, which I’ve become aware of a decade ago, and still
	have found no use for; thus anything dependent on it immediately
	raises my suspicion.

-- 
FSF associate member #7257  http://am-1.org/~ivan/      … 3013 B6A0 230E 334A

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Ivan Shmakov <ivan <at> siamics.net>
Cc: 22243 <at> debbugs.gnu.org
Subject: Re: bug#22243: eieio-opt: not quite (?) optional functions
Date: Thu, 01 Aug 2019 19:09:26 +0200

Ivan Shmakov <ivan <at> siamics.net> writes:

> 	eieio.el currently (1dcf9a5d2a72; 2015-12-22 17:02:30 UTC)
> 	reads:
>
>    943	;; Hook ourselves into help system for describing classes and methods.
>    944	;; FIXME: This is not actually needed any more since we can click on the
>    945	;; hyperlink from the constructor's docstring to see the type definition.
>    946	(add-hook 'help-fns-describe-function-functions 'eieio-help-constructor)
>
> 	This results that with -Q, 'eieio-opt gets loaded immediately
> 	once the user tries to use the Emacs help system (say, C-h f),
> 	making the file’s description (below) somewhat misleading, as
> 	the library which is hooked into such a basic Emacs facility
> 	could hardly be called “optional”.

If comment about this not being needed was added in 2015 by Stefan.  But
it says that it's "actually" not needed, and "actually not" in common
usage means "absolutely is", so that's quite confusing.

But if it's not needed, then it sounds like we should remove it?

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

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

From: Noam Postavsky <npostavs <at> gmail.com>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 22243 <at> debbugs.gnu.org, Ivan Shmakov <ivan <at> siamics.net>
Subject: Re: bug#22243: eieio-opt: not quite (?) optional functions
Date: Thu, 01 Aug 2019 13:33:34 -0400

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> "actually not" in common usage means "absolutely is"

Does it really?  I don't think I've ever seen it used that way.

>    944	;; FIXME: This is not actually needed any more

I interpret this as meaning "this could be removed, but doing so might
also require adjusting some other things".

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Noam Postavsky <npostavs <at> gmail.com>
Cc: 22243 <at> debbugs.gnu.org, Ivan Shmakov <ivan <at> siamics.net>
Subject: Re: bug#22243: eieio-opt: not quite (?) optional functions
Date: Thu, 01 Aug 2019 19:37:59 +0200

Noam Postavsky <npostavs <at> gmail.com> writes:

> Lars Ingebrigtsen <larsi <at> gnus.org> writes:
>
>> "actually not" in common usage means "absolutely is"
>
> Does it really?  I don't think I've ever seen it used that way.

You haven't noticed that when somebody says "This is actually ..." then
it's usually not "..."?

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