- Pharo-dev - lists.pharo.org

Re: [Pharo-project] Replacing 'ReadStream on smth:' by 'smth readStream'
by Bill Schwab May 30, 2008

May 30, 2008

Damien, Interesting! As an aside, I have a love/hate relationship with the RB, and almost never let it rewrite any of my Dolphin code. The XP guys will recoil in horror at this, but I have too often been saved by 18 month old comments to have them disrespectfully scattered around my source. If that is fixed, I will become a believer. I *do* use pull up/down to great advantage, and will sometimes use other refactorings on new code that has not had time to accumulate valuable comments. The above said, I have considered doing something like your rewrite to fix stream method naming snags. Consider #next. As we have discussed, I think it should be named #nextOrNil, allowing #next to behave as it does in Dolphin. Yes, I have a bias toward Dolphin, but I also consider its streams to be elegantly/correctly built. One way to fix it would be to transform #next into #nextOrNil (which is what Squeak's #next does), and then add a #next that signals EndOfStream on exhaustion; EndOfStream would have no default action in this scenario. If we could further transform any idioms such as aStream next ifNotNil:[ :x | ... ] into aStream nextIfPresent:[ :x | ... ] etc., that could be great if we did it well. Even a clean change to #nextOrNil would allow us to make an otherwise breaking change as painlessly as possible. FWIW, I have no direct problem with #on:, though I see where you are going. We might get a similar benefit by simply changing ReadStream in the system dictionary, making it an alias for NSReadStream?? Bill Wilhelm K. Schwab, Ph.D. University of Florida Department of Anesthesiology PO Box 100254 Gainesville, FL 32610-0254 Email: bschwab(a)anest.ufl.edu Tel: (352) 846-1285 FAX: (352) 392-7029 >>> damien.cassou(a)gmail.com 05/30/08 4:47 AM >>> Hi, I've just published in the pharo inbox a package for the System category. This change has been generated automatically through the rewriter: ParseTreeRewriter new replace: 'ReadStream on: ``@object' with: '``@object readStream'; yourself The formatting of the changed method is completely different due to the use of the refactoring browser. This makes the diffs not readable. Even if it is a real problem, I don't see any simple solution. What do you think? -- Damien Cassou Peter von der AhÃ©: Â«I'm beginning to see why Gilad wished us good luckÂ». (http://blogs.sun.com/ahe/entry/override_snafu) _______________________________________________ Pharo-project mailing list Pharo-project(a)lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

2 2

Re: [Pharo-project] [ANN][Pharo-dev] First release
by Bill Schwab May 30, 2008

May 30, 2008

Damien, Probably. Unfortunately, I would have to try each release to see whether it would help. Having the RB, Seaside, and ideally Pinesoft enhancements would be great. My personal tastes often lean away from the bleeding edge browsers and programming assistants. If the basics are still available, I should be happy. If nothing else, it should be interesting to see what does and does not work, especially as we fix what needs to be fixed. Bill Wilhelm K. Schwab, Ph.D. University of Florida Department of Anesthesiology PO Box 100254 Gainesville, FL 32610-0254 Email: bschwab(a)anest.ufl.edu Tel: (352) 846-1285 FAX: (352) 392-7029 >>> damien.cassou(a)gmail.com 05/30/08 9:06 AM >>> Hi, I've published the first version of Pharo-dev. It is hosted on gforge as usual: https://gforge.inria.fr/frs/?group_id=1299. I will try to generate new pharo-dev images when I generate squeak-dev images if anyone is interested. Are you? Bye -- Damien Cassou Peter von der AhÃ©: Â«I'm beginning to see why Gilad wished us good luckÂ». (http://blogs.sun.com/ahe/entry/override_snafu) _______________________________________________ Pharo-project mailing list Pharo-project(a)lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

1 0

Re: [Pharo-project] Replacing 'ReadStream on smth:' by 'smth readStream'
by Bill Schwab May 30, 2008

May 30, 2008

Adrian, Damien, I suspect that Damien's goal is to simplify replacing ReadStream. If we can do that by ReadStream := SomethingElse. then #on: might be fine as it is. I do suspect that the RB is the best and most practical way to cope with #next. Bill Wilhelm K. Schwab, Ph.D. University of Florida Department of Anesthesiology PO Box 100254 Gainesville, FL 32610-0254 Email: bschwab(a)anest.ufl.edu Tel: (352) 846-1285 FAX: (352) 392-7029 >>> adi(a)netstyle.ch 05/30/08 8:18 AM >>> Have you verified that your change does not inadvertently modify any behavior? Its not likely, but changes like this can introduce subtle bugs. For instance, Matrix overrides readStream. So if there exists any use of "ReadStream on: aMatrix" that is there to bypass the overridden behavior, the code will break. Cheers, Adrian On May 30, 2008, at 10:47 , Damien Cassou wrote: > Hi, > > I've just published in the pharo inbox a package for the System > category. This change has been generated automatically through the > rewriter: > > ParseTreeRewriter new > replace: 'ReadStream on: ``@object' with: '``@object readStream'; > yourself > > The formatting of the changed method is completely different due to > the use of the refactoring browser. This makes the diffs not readable. > Even if it is a real problem, I don't see any simple solution. What do > you think? > > -- > Damien Cassou > Peter von der AhÃ©: Â«I'm beginning to see why Gilad wished us good > luckÂ». (http://blogs.sun.com/ahe/entry/override_snafu) > > _______________________________________________ > 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

1 0

[Pharo-project] new to pharo
by Alain Plantec May 30, 2008

May 30, 2008

Hi all, I would like to help. MVC removeall is a task I can achieve. now the question is how do I must proceed? what kind of result ? .cs, .mcz, script ... thanks alain

2 1

[Pharo-project] Some more cleaning...
by Alexandre Bergel May 28, 2008

May 28, 2008

Dear all, I put two changes in the PharoInBox that remove some unused and obsolete code related to MVC and the menu on Projects. Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

2 1

[Pharo-project] Call for help
by StÃ©phane Ducasse May 28, 2008

May 28, 2008

Hi guys I saw that PackageInfo does not have any tests. It would be great if someone could write a couple of tests. stef

5 18

[Pharo-project] Why 3 projects ?
by Serge Stinckwich May 28, 2008

May 28, 2008

Stef, i was wondering : why not use just one SqueakSource project (Pharo) instead of three (Pharo, PharoInbox, PharoTreated) ? You are worrying about the scalability ? You just need to select from time to time which package will be blessed and include them in the official update list. -- Serge Stinckwich http://blog.doesnotunderstand.org/

3 3

[Pharo-project] Message for admin
by Alexandre Bergel May 27, 2008

May 27, 2008

I spent some time on trying to set the mailing list email the default Reply-To: field, but obviously I failed. Is there any mailman guru here? Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

4 3

[Pharo-project] Test
by Alexandre Bergel May 27, 2008

May 27, 2008

-- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

1 0

[Pharo-project] Pharo archive on gmane
by Serge Stinckwich May 26, 2008

May 26, 2008

Pharo mailing-list is now archived on gmane.org : http://dir.gmane.org/gmane.comp.lang.smalltalk.pharo.devel -- Serge Stinckwich http://blog.doesnotunderstand.org/

2 1