Re: warnings in methods with enabled lexical-binding

On Fri, 24 May 2013 05:46:40 +0500, Eric M. Ludlam <[email protected]>  
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.

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may