May 2008 - Pharo-dev - lists.pharo.org

[Pharo-project] Re: Package Files has unsaved changes in latest Pharo

by Damien Cassou

On Fri, May 30, 2008 at 2:37 PM, Damien Cassou <damien.cassou(a)gmail.com> wrote: > Hi, > > if you take latest released Pharo, open Monticello, select the package > Files and the repository Pharo, you will see changes in the class > declaration CrLfFileStream. Same for packages Sound and Compiler. Probably others. -- 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)

May 30, 2008

Re: [Pharo-project] [ANN][Pharo-dev] First release

by Damien Cassou

On Fri, May 30, 2008 at 3:54 PM, Alexandre Bergel <alexandre(a)bergel.eu> wrote: > I agree with Bil, it is hard to say whether people will find Pharo-dev > useful. Just give a try :-) I was thinking about you guys who are working on Pharo himself. It seems you all load OB, RB and the other manually. -- 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)

May 30, 2008

Re: [Pharo-project] [ANN][Pharo-dev] First release

by Alexandre Bergel

I agree with Bil, it is hard to say whether people will find Pharo-dev useful. Just give a try :-) Cheers, Alexandre On 30 May 2008, at 15:06, Damien Cassou wrote: > 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 > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

May 30, 2008

Re: [Pharo-project] Replacing 'ReadStream on smth:' by 'smth readStream'

by Damien Cassou

On Fri, May 30, 2008 at 2:13 PM, Bill Schwab <BSchwab(a)anest.ufl.edu> wrote: > 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. This problem has been fixed some time ago IIRC. -- 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)

May 30, 2008

Re: [Pharo-project] Replacing 'ReadStream on smth:' by 'smth readStream'

by Damien Cassou

On Fri, May 30, 2008 at 2:13 PM, Bill Schwab <BSchwab(a)anest.ufl.edu> wrote: > 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; I agree. > 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?? Even without talking about Nile, I think it is a good thing to avoid direct references to ReadStream. This also renders the code more readable. A third benefit is that #readStream could now be implement differently on different kind of streamable things. In Nile for example, String overrides #readStream to return an improved ReadStream for strings. -- 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)

May 30, 2008

Re: [Pharo-project] Replacing 'ReadStream on smth:' by 'smth readStream'

by Damien Cassou

On Fri, May 30, 2008 at 2:18 PM, Adrian Lienhard <adi(a)netstyle.ch> wrote: > 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. It should not break anything (if it does, the test will fail and I will fix things :-)). In fact, ReadStream only deals with SequenceableStreams so it should be ok. Do you agree? -- 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)

May 30, 2008

Re: [Pharo-project] [ANN][Pharo-dev] First release

by Bill Schwab

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

May 30, 2008

[Pharo-project] [ANN][Pharo-dev] First release

by Damien Cassou

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)

May 30, 2008

Re: [Pharo-project] Replacing 'ReadStream on smth:' by 'smth readStream'

by Bill Schwab

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

May 30, 2008

[Pharo-project] Package Files has unsaved changes in latest Pharo

by Damien Cassou

Hi, if you take latest released Pharo, open Monticello, select the package Files and the repository Pharo, you will see changes in the class declaration CrLfFileStream. -- 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)

May 30, 2008