On 16 September 2013 17:43, Esteban Lorenzano <estebanlm@gmail.com> wrote:

do not lose your time trying to fix that.
We are going to change it anyway.

but i don't want to wait till your changes will come,
��i want it now, so i will push the slice, and if you don't want it, just don't integrate :)

��

Esteban

On Sep 16, 2013, at 5:39 PM, Igor Stasenko <siguctua@gmail.com> wrote:

> trying to fix completion autopopups when there is nothing to suggest
> to prevent popping up the empty and useless menu,
> i stumbled upon this:
>
> initializeInstVars
> �� (clazz isNil or: [ includeVariables not ]) ifTrue: [ ^ self ].
> �� instVars := clazz allInstVarNames asSortedCollection.
> �� instVars := instVars collect:
> �� [ :each | ��NECInstVarEntry contents: each type: #instVar ].
> �� instVars add: (NECSelfEntry
> �� contents: 'self'
> �� type: #self).
> �� instVars add: (NECSuperEntry
> �� contents: 'super'
> �� type: #super)
>
>
> there's something sick with that..
> first we using
> NECSelfEntry
> but then, if that's not enough,
> we pass 'self' as its contents..
> but that's also not enough!
> we pass 'self' as its type as well..
>
> checking many subclasses of NECEntry
> and how they initialized, it is almost every time the same:
>
> - you use NECXXXXEntry , we always pass separate #XXXX
> as its type like just using NECXXXXEntry not enough to indicate that.
>
> i going to fix that as well
>
> --
> Best regards,
> Igor Stasenko.

--
Best regards,
Igor Stasenko.