[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Cleaning up the EIEIO namespace
From: |
Stefan Monnier |
Subject: |
Re: Cleaning up the EIEIO namespace |
Date: |
Sun, 17 Feb 2013 12:08:41 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) |
>>>>> In toplevel form:
>>>>> eieio.el:168:1:Error: Symbol's function definition is void:
>>>>> eieio--class-parent
>>>> Can you (setq byte-compile-error t debug-on-error t) so as to get
>>>> a backtrace?
>>> Did you mean `byte-compile-error-on-warn'? Anyway, I'm afraid I just
>>> don't get a backtrace with debug-on-error.
>> No, I meant byte-compile-debug sorry.
> Ah OK, I didn't know that one. Now I get a trace, but it's huge, so I
> gzipped it.
eieio's extensive abuse of eval-and-compile is a real pain in the rear!
Does the additional patch below make it work for you?
Stefan
--- lisp/emacs-lisp/eieio.el.orig 2013-02-17 12:06:09.252331190 -0500
+++ lisp/emacs-lisp/eieio.el 2013-02-17 12:06:20.336456193 -0500
@@ -121,10 +121,9 @@
(list 'aref x ,index))
defs)
(setq index (1+ index))))
- `(progn
+ `(eval-and-compile
,@(nreverse defs)
- (eval-and-compile
- (defconst ,(intern (format "eieio--%s-num-slots" prefix)) ,index)))))
+ (defconst ,(intern (format "eieio--%s-num-slots" prefix)) ,index))))
(eieio--define-field-accessors class
(-unused-0 ;;FIXME: not sure, but at least there was no accessor!
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, (continued)
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, Stefan Monnier, 2013/02/14
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, David Engster, 2013/02/18
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, Stefan Monnier, 2013/02/18
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, David Engster, 2013/02/19
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, Stefan Monnier, 2013/02/19
- Re: [cedet-eieio] Cleaning up the EIEIO namespace, Eric M. Ludlam, 2013/02/20
- Re: Cleaning up the EIEIO namespace, Stefan Monnier, 2013/02/14
- Re: Cleaning up the EIEIO namespace, David Engster, 2013/02/14
- Re: Cleaning up the EIEIO namespace, Stefan Monnier, 2013/02/14
- Re: Cleaning up the EIEIO namespace, David Engster, 2013/02/14
- Re: Cleaning up the EIEIO namespace,
Stefan Monnier <=
- Re: Cleaning up the EIEIO namespace, David Engster, 2013/02/18
- Re: Cleaning up the EIEIO namespace, Stefan Monnier, 2013/02/18
Re: [cedet-eieio] Cleaning up the EIEIO namespace, Eric M. Ludlam, 2013/02/12