all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: master b5393c6 2/3: Silence undefined function warnings in eieio-tests.el
       [not found] ` <20211220151238.2DADD209AC@vcs0.savannah.gnu.org>
@ 2021-12-20 16:21   ` Stefan Monnier
  2021-12-20 17:38     ` Stefan Kangas
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2021-12-20 16:21 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: emacs-devel

> +;; Silence byte-compiler.
> +(defun eitest-subordinate--eieio-childp ())
> +(defun class-alloc-initarg--eieio-childp ())

Hmm... I don't understand this code.
If `declare-function` can't be used, we should have a comment
explaining why.


        Stefan




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

* Re: master b5393c6 2/3: Silence undefined function warnings in eieio-tests.el
  2021-12-20 16:21   ` master b5393c6 2/3: Silence undefined function warnings in eieio-tests.el Stefan Monnier
@ 2021-12-20 17:38     ` Stefan Kangas
  2022-01-09 19:19       ` Jean Louis
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Kangas @ 2021-12-20 17:38 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> +;; Silence byte-compiler.
>> +(defun eitest-subordinate--eieio-childp ())
>> +(defun class-alloc-initarg--eieio-childp ())
>
> Hmm... I don't understand this code.
> If `declare-function` can't be used, we should have a comment
> explaining why.

Duh, I somehow forgot that I can use that also for Lisp functions.

Now fixed, thanks.

BTW, there are only four remaining warnings in the test suite if someone who
knows eieio feels like taking a look.



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

* Re: master b5393c6 2/3: Silence undefined function warnings in eieio-tests.el
  2021-12-20 17:38     ` Stefan Kangas
@ 2022-01-09 19:19       ` Jean Louis
  2022-01-10  3:38         ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Jean Louis @ 2022-01-09 19:19 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: Stefan Monnier, emacs-devel

* Stefan Kangas <stefan@marxist.se> [2021-12-21 13:55]:
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
> 
> >> +;; Silence byte-compiler.
> >> +(defun eitest-subordinate--eieio-childp ())
> >> +(defun class-alloc-initarg--eieio-childp ())
> >
> > Hmm... I don't understand this code.
> > If `declare-function` can't be used, we should have a comment
> > explaining why.
> 
> Duh, I somehow forgot that I can use that also for Lisp functions.
> 
> Now fixed, thanks.
> 
> BTW, there are only four remaining warnings in the test suite if someone who
> knows eieio feels like taking a look.

What I know is that I cannot run helm and other packages because in
latest development version I am getting:

byte-code: Symbol’s function definition is void: eieio--defgeneric-init-form


-- 
Jean

Take action in Free Software Foundation campaigns:
https://www.fsf.org/campaigns

In support of Richard M. Stallman
https://stallmansupport.org/



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

* Re: master b5393c6 2/3: Silence undefined function warnings in eieio-tests.el
  2022-01-09 19:19       ` Jean Louis
@ 2022-01-10  3:38         ` Stefan Monnier
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2022-01-10  3:38 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: emacs-devel

> What I know is that I cannot run helm and other packages because in
> latest development version I am getting:
>
> byte-code: Symbol’s function definition is void: eieio--defgeneric-init-form

The etc/NEWS explains that:

    ** Support for old EIEIO functions is not autoloaded any more.
    You need an explicit (require 'eieio-compat) to use 'defmethod'
    and 'defgeneric' (which have been made obsolete in Emacs-25 with
    'cl-defmethod' and 'cl-defgeneric').
    Similarly you might need to (require 'eieio-compat) before loading
    files that were compiled with an old EIEIO (Emacs<25).

The better fix is probably to recompile those packages with an Emacs≥25.


        Stefan




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

end of thread, other threads:[~2022-01-10  3:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20211220151236.23091.98968@vcs0.savannah.gnu.org>
     [not found] ` <20211220151238.2DADD209AC@vcs0.savannah.gnu.org>
2021-12-20 16:21   ` master b5393c6 2/3: Silence undefined function warnings in eieio-tests.el Stefan Monnier
2021-12-20 17:38     ` Stefan Kangas
2022-01-09 19:19       ` Jean Louis
2022-01-10  3:38         ` Stefan Monnier

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.