if somebody wants to check that the fixes I made are ok
and that at:put: and add: return their arguments Stef _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project addReturnValue.1.cs (7K) Download Attachment |
That looks good to me. If no additional tests break after loading it,
I guess it should be clear to integrate. It is important to have a uniform protocol. We should do the same for #nextPut: and #nextPutAll: Lukas 2010/5/8 stephane ducasse <[hidden email]>: > if somebody wants to check that the fixes I made are ok > and that at:put: and add: return their arguments > > > > > > Stef > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Lukas Renggli www.lukas-renggli.ch _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Tx!
Soon in the update. On May 8, 2010, at 5:49 PM, Lukas Renggli wrote: > That looks good to me. If no additional tests break after loading it, > I guess it should be clear to integrate. It is important to have a > uniform protocol. We should do the same for #nextPut: and #nextPutAll: > > Lukas > > 2010/5/8 stephane ducasse <[hidden email]>: >> if somebody wants to check that the fixes I made are ok >> and that at:put: and add: return their arguments >> >> >> >> >> >> Stef >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > > -- > Lukas Renggli > www.lukas-renggli.ch > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by stephane ducasse
I don't have time to scan the mailing list in detail. Was there any discussion that led to this change? Any pointers appreciated. I always thought the comfort of being able to chain selectors like add: outweighs the uniformity.
Norbert On 08.05.2010, at 17:31, stephane ducasse wrote: > if somebody wants to check that the fixes I made are ok > and that at:put: and add: return their arguments > > > <addReturnValue.1.cs> > > Stef_______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
consistency
and this is linked to the work of nicolas to do the same in Squeak. Stef On May 10, 2010, at 10:06 AM, Norbert Hartl wrote: > I don't have time to scan the mailing list in detail. Was there any discussion that led to this change? Any pointers appreciated. I always thought the comfort of being able to chain selectors like add: outweighs the uniformity. > > Norbert > > On 08.05.2010, at 17:31, stephane ducasse wrote: > >> if somebody wants to check that the fixes I made are ok >> and that at:put: and add: return their arguments >> >> >> <addReturnValue.1.cs> >> >> Stef_______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Free forum by Nabble | Edit this page |