- Pharo-dev - lists.pharo.org

Re: [Pharo-project] class API question

by Jorge Ressia

Yep, It would be much better to sell out the names. On Fri, Aug 27, 2010 at 11:53 AM, Lukas Renggli <renggli(a)gmail.com> wrote: > I think it would be beneficial to adapt the protocol of the refactoring browser. > > It especially spells out all the names: > > Â Â #addInstanceVariable: > Â Â #allInstanceVariableNames > Â Â #instanceVariableNames > > Lukas > > On 27 August 2010 11:50, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: >> done :) >> I could not let that like that. >> >> Stef >> >> On Aug 27, 2010, at 11:11 AM, Marcus Denker wrote: >> >>> >>> On Aug 27, 2010, at 11:02 AM, StÃ©phane Ducasse wrote: >>> >>>> We are not consistent: >>>> >>>> Â Â Â do we want >>>> Â Â Â Â Â Â Â renameInstVar:to: >>>> Â Â Â or >>>> Â Â Â Â Â Â Â removeInstVarName: >>>> Â Â Â Â Â Â Â Â Â Â Â I prefer that. >>>> >>>> We have addInstVarName: >>>> Â Â Â -> should be addInstVarNamed: from my taste >>>> >>>> >>>> Now we want to have first class instance variable in the near future so I would like to have >>>> >>>> Â Â Â addInstVar: aVariable >>>> Â Â Â addInstVarNamed: aString >>>> >>>> What do you think? >>>> >>> >>> Yes! >>> >>> >>> -- >>> Marcus Denker Â -- http://www.marcusdenker.de >>> INRIA Lille -- Nord Europe. Team RMoD. >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> Pharo-project(a)lists.gforge.inria.fr >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > > -- > Lukas Renggli > www.lukas-renggli.ch > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Jorge Ressia www.jorgeressia.com

Aug. 27, 2010

Re: [Pharo-project] class API question

by Lukas Renggli

I think it would be beneficial to adapt the protocol of the refactoring browser. It especially spells out all the names: #addInstanceVariable: #allInstanceVariableNames #instanceVariableNames Lukas On 27 August 2010 11:50, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > done :) > I could not let that like that. > > Stef > > On Aug 27, 2010, at 11:11 AM, Marcus Denker wrote: > >> >> On Aug 27, 2010, at 11:02 AM, StÃ©phane Ducasse wrote: >> >>> We are not consistent: >>> >>> Â Â Â do we want >>> Â Â Â Â Â Â Â renameInstVar:to: >>> Â Â Â or >>> Â Â Â Â Â Â Â removeInstVarName: >>> Â Â Â Â Â Â Â Â Â Â Â I prefer that. >>> >>> We have addInstVarName: >>> Â Â Â -> should be addInstVarNamed: from my taste >>> >>> >>> Now we want to have first class instance variable in the near future so I would like to have >>> >>> Â Â Â addInstVar: aVariable >>> Â Â Â addInstVarNamed: aString >>> >>> What do you think? >>> >> >> Yes! >> >> >> -- >> Marcus Denker Â -- http://www.marcusdenker.de >> INRIA Lille -- Nord Europe. Team RMoD. >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Lukas Renggli www.lukas-renggli.ch

Aug. 27, 2010

Re: [Pharo-project] hasInstanceVariableNamed: hasClassVariableNamed:

by StÃ©phane Ducasse

> *Cough* deprecate *cough* ;) what you got a flu :) Of course this is done and I would like to have a refactoring called renameDeprecate :) Lukas? :) Stef > > Cheers, > Henry > > On Aug 27, 2010, at 10:51 44AM, StÃ©phane Ducasse wrote: > >> for example >> we have >> addClassVarName: >> classVarNamed: >> classVarNamed:put: >> removeClassVarName: >> >> I will rename >> addClassVarName: >> removeClassVarName: >> >> to be >> addClassVarNamed: >> removeClassVarNamed: >> >> consistency.... >> >> Stef >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 27, 2010

Re: [Pharo-project] class API question

by StÃ©phane Ducasse

done :) I could not let that like that. Stef On Aug 27, 2010, at 11:11 AM, Marcus Denker wrote: > > On Aug 27, 2010, at 11:02 AM, StÃ©phane Ducasse wrote: > >> We are not consistent: >> >> do we want >> renameInstVar:to: >> or >> removeInstVarName: >> I prefer that. >> >> We have addInstVarName: >> -> should be addInstVarNamed: from my taste >> >> >> Now we want to have first class instance variable in the near future so I would like to have >> >> addInstVar: aVariable >> addInstVarNamed: aString >> >> What do you think? >> > > Yes! > > > -- > Marcus Denker -- http://www.marcusdenker.de > INRIA Lille -- Nord Europe. Team RMoD. > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 27, 2010

[Pharo-project] could we fix MorphicModel

by stephane ducasse

gary? Do you have surgeon knife? Because I do not see why slider need to depend on that wonderfully designed code. use: cachedSelector orMakeModelSelectorFor: selectorBody in: selectorBlock | selector | model ifNil: [^ nil]. cachedSelector ifNil: ["Make up selector from slotname if any" selector := (slotName ifNil: [selectorBody] ifNotNil: [slotName , selectorBody]) asSymbol. (model class canUnderstand: selector) ifFalse: [(self confirm: 'Shall I compile a null response for' , Character cr asString , model class name , '>>' , selector) ifFalse: [self halt]. model class compile: (String streamContents: [:s | selector keywords doWithIndex: [:k :i | s nextPutAll: k , ' arg' , i printString]. s cr; nextPutAll: '"Automatically generated null response."'. s cr; nextPutAll: '"Add code below for appropriate behavior..."'.]) classified: 'input events' notifying: nil]] ifNotNil: [selector := cachedSelector]. ^ selectorBlock value: selector

Aug. 27, 2010

Re: [Pharo-project] hasInstanceVariableNamed: hasClassVariableNamed:

by Henrik Johansen

*Cough* deprecate *cough* ;) Cheers, Henry On Aug 27, 2010, at 10:51 44AM, StÃ©phane Ducasse wrote: > for example > we have > addClassVarName: > classVarNamed: > classVarNamed:put: > removeClassVarName: > > I will rename > addClassVarName: > removeClassVarName: > > to be > addClassVarNamed: > removeClassVarNamed: > > consistency.... > > Stef > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 27, 2010

Re: [Pharo-project] hasInstanceVariableNamed: hasClassVariableNamed:

by Marcus Denker

On Aug 27, 2010, at 10:31 AM, StÃ©phane Ducasse wrote: > what about > > ClassDescription>>chooseInstVarThenDo: aBlock > > ClassDescription>>chooseInstVarAlphabeticallyThenDo: aBlock > > > ClassDescription>>chooseClassVarName > > should I really comment.... I will fix them. > This should not be in ClassDescription... the only client is SystemNavigation. -- Marcus Denker -- http://www.marcusdenker.de INRIA Lille -- Nord Europe. Team RMoD.

Aug. 27, 2010

Re: [Pharo-project] class API question

by Marcus Denker

On Aug 27, 2010, at 11:02 AM, StÃ©phane Ducasse wrote: > We are not consistent: > > do we want > renameInstVar:to: > or > removeInstVarName: > I prefer that. > > We have addInstVarName: > -> should be addInstVarNamed: from my taste > > > Now we want to have first class instance variable in the near future so I would like to have > > addInstVar: aVariable > addInstVarNamed: aString > > What do you think? > Yes! -- Marcus Denker -- http://www.marcusdenker.de INRIA Lille -- Nord Europe. Team RMoD.

Aug. 27, 2010

Re: [Pharo-project] [SPAM] About webclient.ppm.73

by StÃ©phane Ducasse

this is in the inbox Stef On Aug 27, 2010, at 10:47 AM, Sven Van Caekenberghe wrote: > I could have a look, if you tell me where I can find it... > > On 27 Aug 2010, at 10:22, stephane ducasse wrote: > >> hi philippe >> >> I wanted to integrate it in 1.2 but I have 5 conflicts. Should I juste load? >> Is it for 1.2? >> >> Stef >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Aug. 27, 2010

[Pharo-project] class API question

by StÃ©phane Ducasse

We are not consistent: do we want renameInstVar:to: or removeInstVarName: I prefer that. We have addInstVarName: -> should be addInstVarNamed: from my taste Now we want to have first class instance variable in the near future so I would like to have addInstVar: aVariable addInstVarNamed: aString What do you think? Stef

Aug. 27, 2010