Thread: [cedet-eieio] warnings in methods with enabled lexical-binding
Brought to you by:
zappo
From: Sergey P. <qwe...@ya...> - 2013-04-14 15:41:46
|
Hello, I receive following warning during any (e.g. from quick start section in manual) method evaluation: Warning: Unused lexical variable `scoped-class' Does eieio works with lexical binding correctly? |
From: Eric M. L. <er...@si...> - 2013-05-07 00:28:05
|
On 04/14/2013 11:41 AM, Sergey Petrov wrote: > Hello, I receive following warning during any (e.g. from quick start > section in manual) method evaluation: > Warning: Unused lexical variable `scoped-class' > Does eieio works with lexical binding correctly? Hi Sergey, Sorry for the slow response. The scoped-class feature in EIEIO would indeed break lexical binding. I'm not that familiar with lexical binding, or how to 'fix' EIEIO to work with it. I'm open to suggestions to how to fix it from the list. Eric |
From: Sergey <qwe...@ya...> - 2013-05-21 10:30:57
|
On Tue, 07 May 2013 05:27:56 +0500, Eric M. Ludlam wrote: > On 04/14/2013 11:41 AM, Sergey Petrov wrote: >> Hello, I receive following warning during any (e.g. from quick start >> section in manual) method evaluation: >> Warning: Unused lexical variable `scoped-class' >> Does eieio works with lexical binding correctly? > > Hi Sergey, > > Sorry for the slow response. > > The scoped-class feature in EIEIO would indeed break lexical binding. > I'm not that familiar with lexical binding, or how to 'fix' EIEIO to > work with it. > > I'm open to suggestions to how to fix it from the list. > > Eric Thank you. I just wanted a peaceful co-existence for my lexical closures with eieio classes and methods in the same .el file. Probably, the following patch will solve this 'problem'. *** eieio.el 2013-05-21 11:20:04.133759895 +0500 *** 1235,1241 **** ;; is faster to execute this for not byte-compiled. ie, install this, ;; then measure calls going through here. I wonder why. (require 'bytecomp) ! (let ((byte-compile-warnings nil)) (byte-compile `(lambda (&rest local-args) ,doc-string --- 1235,1242 ---- ;; is faster to execute this for not byte-compiled. ie, install this, ;; then measure calls going through here. I wonder why. (require 'bytecomp) ! (let ((byte-compile-warnings nil) ! (lexical-binding nil)) (byte-compile `(lambda (&rest local-args) ,doc-string |
From: Eric M. L. <er...@si...> - 2013-05-24 00:46:50
|
On 05/21/2013 06:30 AM, Sergey wrote: > On Tue, 07 May 2013 05:27:56 +0500, Eric M. Ludlam wrote: > >> On 04/14/2013 11:41 AM, Sergey Petrov wrote: >>> Hello, I receive following warning during any (e.g. from quick start >>> section in manual) method evaluation: >>> Warning: Unused lexical variable `scoped-class' >>> Does eieio works with lexical binding correctly? >> >> Hi Sergey, >> >> Sorry for the slow response. >> >> The scoped-class feature in EIEIO would indeed break lexical binding. >> I'm not that familiar with lexical binding, or how to 'fix' EIEIO to >> work with it. >> >> I'm open to suggestions to how to fix it from the list. >> >> Eric > > > Thank you. I just wanted a peaceful co-existence for my lexical closures > with eieio classes and methods in the same .el file. > Probably, the following patch will solve this 'problem'. > > *** eieio.el 2013-05-21 11:20:04.133759895 +0500 > *** 1235,1241 **** Hi Sergey, Based on this line number, I think you have an old version of eieio. To aid with byte-compilation in Emacs, eieio.el was split into both eieio.el, and eieio-core.el in the CEDET bzr repository. You will also find in that same fcn I recently updated it to fix the scoped-class problem so it shouldn't have lexical binding issues anymore. Perhaps you can try out the latest from CEDET/bzr, and it will fix your issue? Eric |
From: Sergey P. <qwe...@ya...> - 2013-05-24 06:23:03
|
On Fri, 24 May 2013 05:46:40 +0500, Eric M. Ludlam <er...@si...> wrote: > On 05/21/2013 06:30 AM, Sergey wrote: >> On Tue, 07 May 2013 05:27:56 +0500, Eric M. Ludlam wrote: >> >>> On 04/14/2013 11:41 AM, Sergey Petrov wrote: >>>> Hello, I receive following warning during any (e.g. from quick start >>>> section in manual) method evaluation: >>>> Warning: Unused lexical variable `scoped-class' >>>> Does eieio works with lexical binding correctly? >>> >>> Hi Sergey, >>> >>> Sorry for the slow response. >>> >>> The scoped-class feature in EIEIO would indeed break lexical binding. >>> I'm not that familiar with lexical binding, or how to 'fix' EIEIO to >>> work with it. >>> >>> I'm open to suggestions to how to fix it from the list. >>> >>> Eric >> >> >> Thank you. I just wanted a peaceful co-existence for my lexical closures >> with eieio classes and methods in the same .el file. >> Probably, the following patch will solve this 'problem'. >> >> *** eieio.el 2013-05-21 11:20:04.133759895 +0500 >> *** 1235,1241 **** > > Hi Sergey, > > Based on this line number, I think you have an old version of eieio. To > aid with byte-compilation in Emacs, eieio.el was split into both > eieio.el, and eieio-core.el in the CEDET bzr repository. > > You will also find in that same fcn I recently updated it to fix the > scoped-class problem so it shouldn't have lexical binding issues anymore. > > Perhaps you can try out the latest from CEDET/bzr, and it will fix your > issue? > > Eric Yes, in bzr version it is ok. Thank you Eric, and sorry for troubling you. |