Hi janko

I checked and
        - pharo does not complain for what seems good. So no bugs from our  
side.

        - SwazooBenchmarks seems to have an instance class variable with  
uppercase
                Singleton

can you check and confirm. I tried to load beta2.2
Thanks

Stef

        

On Dec 29, 2008, at 5:16 PM, Janko Mivšek wrote:

> Hi Ramon,
>
> Ramon Leon wrote:
>
>>>> Yes I contacted Janko if he can fix that
>
>>> I can fix of course but I'm a bit surprised that in Squeak the class
>>> variables are supposed to be lower case. I have in mind back
>>> from 1996
>>> that they must be uppercase.
>
>> Janko, I'm pretty sure Lukas was talking about a class instance  
>> variable,
>> not a class variable.  Class instance variables should be lowercase  
>> while
>> class vars are uppercase.  Swazoo apparently contains a class- 
>> instance
>> variable that is uppercased, which regardless of the outcome of this
>> discussion, should probably be fixed.
>
> Here is the exact definition of a Swazoo class in question:
>
> Object subclass: #SwazooServer
>     instanceVariableNames: 'sites servers watchdog'
>     classVariableNames: 'Singleton'
>     poolDictionaries: ''
>     category: 'Swazoo-Core'
>
> And here the same definition of Swazoo SiteRegistry in VW 3.0 back  
> from
> 2000:
>
> Object subclass: #SiteRegistry
>     instanceVariableNames: 'sites servers '
>     classVariableNames: 'Singleton '
>     poolDictionaries: ''
>     category: 'Swazoo'
>
> So, this is obviously a class variable definition and not an instance
> one. Where is the problem then? I don't find any class instance
> variables in other classes and in any way the Singleton should be only
> class variable in use in Swazoo.
>
> Best regards
> Janko
>
>
>
>
> -- 
> Janko Mivšek
> Svetovalec za informatiko
> Eranova d.o.o.
> Ljubljana, Slovenija
> www.eranova.si
> tel:  01 514 22 55
> faks: 01 514 22 56
> gsm: 031 674 565
>
> _______________________________________________
> Pharo-project mailing list
> [email protected]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>


_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Reply via email to