- Pharo-users - lists.pharo.org

Re: [Pharo-users] mentor question 4

by StÃ©phane Ducasse

> On 5 May 2020, at 16:16, Richard O'Keefe <raoknz(a)gmail.com> wrote: > > By the way, while playing with this problem, I ran into a moderately > painful issue. > > There is a reason that Smalltalk has both #printString (to get a > printable representation of an object) and #asString (to convert a > sequence to another kind of sequence with the same elements.) If I > *want* #printString, I know where to find it. The definition in my > Smalltalk no reads > > asString > "What should #($a $b $c) do? > - Blue Book, Inside Smalltalk, Apple Smalltalk-80: > there is no #asString. > - ANSI, VW, Dolphin, CSOM: > #asString is defined on characters and strings > (and things like file names and URIs that are sort of strings), > so expect an error report. > - VisualAge Smalltalk: > '($a $b $c)' > - Squeak and Pharo: > '#($a $b $c)' > - GNU Smalltalk, Smalltalk/X, and astc: > 'abc' > I don't intend any gratuitous incompatibility, but when there > is no consensus to be compatible with, one must pick something, > and this seems most useful. > " > ^String withAll: self > > Does anyone here know WHY Squeak and Pharo do what they do here? Oops I did not see the quotes on my screen.. #( a b c) asString >>> '#(#a #b #c)â this is unclear to me why this is not good but I have no strong opinion that this is good. I worked on printString for literals because I wanted to have self evaluating properties for basic literal like in Scheme and others. where #t >>> #t And I payed attention that we get the same for literal arrays. Now the conversion is open to me. #($a $b $c) asString >>> '#($a $b $c)â In fact I do not really understand why a string #($a $b $c) asString would be '(a b c)â and its use if this is to nicely display in the ui I would have displayString doing it. S. > > On Wed, 6 May 2020 at 01:20, Richard O'Keefe <raoknz(a)gmail.com> wrote: >> >> The irony is that the code I was responding to ISN'T obviously correct. >> Indeed, I found it rather puzzling. >> The problem specification says that the input string may contain digits >> AND SPACES. The original message includes this: >> >> Strings of length 1 or less are not valid. Spaces are allowed in the >> input, but they should be stripped before checking. All other >> non-digit characters are disallowed. >> >> Now it isn't clear what "disallowed" means. I took it to mean "may occur and >> should simply mean the input is rejected as invalid." Perhaps "may not occur" >> was the intention. So we shall not quibble about such characters. >> >> But I can't for the life of me figure out how Trygve's code checks for spaces. >> One reason this is an issue is that the behaviour of #digitValue is not >> consistent between systems. >> Character space digitValue >> does not exist in the ANSI standard >> answers -1 in many Smalltalks (which is a pain) >> answers a positive integer that can't be mistake for a digit in my Smalltalk >> raises an exception in some Smalltalks. >> >> This is a comment I now have in my Smalltalk library for #digitValue >> "This is in the Blue Book, but unspecified on non-digits. >> Squeak, Pharo, Dolphin, VW, VAST, and Apple Smalltalk-80 >> answer -1 for characters that are not digits (or ASCII letters), >> which is unfortunate but consistent with Inside Smalltalk >> which specifies this result for non-digits. >> ST/X and GST raise an exception which is worse. >> Digitalk ST/V documentation doesn't specify the result. >> This selector is *much* easier to use safely if it >> returns a 'large' (>= 36) value for non-digits." >> >> Let's compare three versions, the two I compared last time, >> and the "version A" code I discussed before, which to my mind >> is fairly readable. >> >> "Don't add slowness": 1 (normalised time) >> "Trygve's code": 6.5 >> "High level code": 30.6 (or 4.7 times slower than Trygve's) >> >> Here's the "High level code". >> ^(aString allSatisfy: [:each | each isSpace or: [each isDigit]]) and: [ >> |digitsReverse| >> digitsReverse := (aString select: [:each | each isDigit]) reverse. >> digitsReverse size > 1 and: [ >> |evens odds evenSum oddSum| >> odds := digitsReverse withIndexSelect: [:y :i | i odd]. >> evens := digitsReverse withIndexSelect: [:x :i | i even]. >> oddSum := odds detectSum: [:y | y digitValue]. >> evenSum := evens detectSum: [:x | >> #(0 2 4 6 8 1 3 5 7 9) at: x digitValue + 1]. >> (oddSum + evenSum) \\ 10 = 0]] >> >> This is the kind of code I was recommending that Roelof write. >> >> As a rough guide, by counting traversals (including ones inside existing >> methods), I'd expect the "high level" code to be at least 10 times slower >> than the "no added slowness" code. >> >> We are in vehement agreement that there is a time to write high level >> really obvious easily testable and debuggable code, and that's most >> of the time, especially with programming exercises. >> >> I hope that we are also in agreement that factors of 30 (or even 6) >> *can* be a serious problem. I mean, if I wanted something that slow, >> I'd use Ruby. >> >> I hope we are also agreed that (with the exception of investigations >> like this one) the time to hack on something to make it faster is AFTER >> you have profiled it and determined that you have a problem. >> >> But I respectfully suggest that there is a difference taking slowness OUT >> and simply not going out of your way to add slowness in the first place. >> >> I'd also like to remark that my preference for methods that traverse a >> sequence exactly once has more to do with Smalltalk protocols than >> with efficiency. If the only method I perform on an object is #do: >> the method will work just as well for readable streams as for >> collections. If the only method I perform on an object is #reverseDo: >> the method will work just as well for Read[Write]Streams as for >> SequenceReadableCollections, at least in my library. It's just like >> trying to write #mean so that it works for Durations as well as Numbers. >> >> Oh heck, I suppose I should point out that much of the overheads in >> this case could be eliminated by a Self-style compiler doing dynamic >> inlining + loop fusion. There's no reason *in principle*, given enough >> people, money, and time, that the differences couldn't be greatly >> reduced in Pharo. >> >> On Tue, 5 May 2020 at 21:50, Trygve Reenskaug <trygver(a)ifi.uio.no> wrote: >>> >>> Richard, >>> >>> Thank you for looking at the code. It is comforting to learn that the code has been executed for a large number of examples without breaking. The code is not primarily written for execution but for being read and checked by the human end user. It would be nice if we could also check that it gave the right answers, but I don't know how to do that. >>> >>> The first question is: Can a human domain expert read the code and sign their name for its correctness? >>> >>> >>> When this is achieved, a programming expert will transcribe the first code to a professional quality program. This time, the second code should be reviewed by an independent programmer who signs their name for its correct transcription from the first version. >>> >>> --Trygve >>> >>> PS: In his 1991 Turing Award Lecture, Tony Hoare said: "There are two ways of constructing a software design: One way is to make it so simple that there are obviously no deficiencies and the other is to make it so complicated that there are no obvious deficiencies. The first method is far more difficult." >>> >>> --Trygve >>> >>> On tirsdag.05.05.2020 04:41, Richard O'Keefe wrote: >>> >>> As a coding experiment, I adapted Trygve Reenskoug's code to my >>> Smalltalk compiler, put in my code slightly tweaked, and benchmarked >>> them on randomly generated data. >>> >>> Result: a factor of 6.3. >>> >>> In Squeak it was a factor of ten. >>> >>> I had not, in all honesty, expected it to to be so high. >>> >>> On Tue, 5 May 2020 at 02:00, Trygve Reenskaug <trygver(a)ifi.uio.no> wrote: >>> >>> A coding experiment. >>> Consider a Scrum development environment. Every programming team has an end user as a member. >>> The team's task is to code a credit card validity check. >>> A first goal is that the user representative shall read the code and agree that it is a correct rendering of their code checker: >>> >>> luhnTest: trialNumber >>> | s1 odd s2 even charValue reverse | >>> ----------------------------------------------- >>> " Luhn test according to Rosetta" >>> "Reverse the order of the digits in the number." >>> reverse := trialNumber reversed. >>> "Take the first, third, ... and every other odd digit in the reversed digits and sum them to form the partial sum s1" >>> s1 := 0. >>> odd := true. >>> reverse do: >>> [:char | >>> odd >>> ifTrue: [ >>> s1 := s1 + char digitValue. >>> ]. >>> odd := odd not >>> ]. >>> "Taking the second, fourth ... and every other even digit in the reversed digits: >>> Multiply each digit by two and sum the digits if the answer is greater than nine to form partial sums for the even digits" >>> "The subtracting 9 gives the same answer. " >>> "Sum the partial sums of the even digits to form s2" >>> s2 := 0. >>> even := false. >>> reverse do: >>> [:char | >>> even >>> ifTrue: [ >>> charValue := char digitValue * 2. >>> charValue > 9 ifTrue: [charValue := charValue - 9]. >>> s2 := s2 + charValue >>> ]. >>> even := even not >>> ]. >>> "If s1 + s2 ends in zero then the original number is in the form of a valid credit card number as verified by the Luhn test." >>> ^(s1 + s2) asString last = $0 >>> --------------------------------- >>> Once this step is completed, the next step will be to make the code right without altering the algorithm (refactoring). The result should be readable and follow the team's conventions. >>> >>> >>> P.S. code attached. >>> >>> >>> -- >>> >>> The essence of object orientation is that objects collaborate to achieve a goal. >>> Trygve Reenskaug mailto: trygver(a)ifi.uio.no >>> Morgedalsvn. 5A http://folk.uio.no/trygver/ >>> N-0378 Oslo http://fullOO.info >>> Norway Tel: (+47) 468 58 625 > -------------------------------------------- StÃ©phane Ducasse http://stephane.ducasse.free.fr / http://www.pharo.org 03 59 35 87 52 Assistant: Julie Jonas FAX 03 59 57 78 50 TEL 03 59 35 86 16 S. Ducasse - Inria 40, avenue Halley, Parc Scientifique de la Haute Borne, BÃ¢t.A, Park Plaza Villeneuve d'Ascq 59650 France

May 11, 2020

Re: [Pharo-users] mentor question 4

by StÃ©phane Ducasse

> By the way, while playing with this problem, I ran into a moderately > painful issue. > > There is a reason that Smalltalk has both #printString (to get a > printable representation of an object) and #asString (to convert a > sequence to another kind of sequence with the same elements.) If I > *want* #printString, I know where to find it. The definition in my > Smalltalk no reads > > asString > "What should #($a $b $c) do? > - Blue Book, Inside Smalltalk, Apple Smalltalk-80: > there is no #asString. > - ANSI, VW, Dolphin, CSOM: > #asString is defined on characters and strings > (and things like file names and URIs that are sort of strings), > so expect an error report. > - VisualAge Smalltalk: > '($a $b $c)' > - Squeak and Pharo: > '#($a $b $c)' > - GNU Smalltalk, Smalltalk/X, and astc: > 'abc' > I don't intend any gratuitous incompatibility, but when there > is no consensus to be compatible with, one must pick something, > and this seems most useful. > " > ^String withAll: self The asString looks wrong to me. I will report it as a bug. Now in Pharo we should really introduce display string > #($a $b $c) > > Does anyone here know WHY Squeak and Pharo do what they do here? > > On Wed, 6 May 2020 at 01:20, Richard O'Keefe <raoknz(a)gmail.com> wrote: >> >> The irony is that the code I was responding to ISN'T obviously correct. >> Indeed, I found it rather puzzling. >> The problem specification says that the input string may contain digits >> AND SPACES. The original message includes this: >> >> Strings of length 1 or less are not valid. Spaces are allowed in the >> input, but they should be stripped before checking. All other >> non-digit characters are disallowed. >> >> Now it isn't clear what "disallowed" means. I took it to mean "may occur and >> should simply mean the input is rejected as invalid." Perhaps "may not occur" >> was the intention. So we shall not quibble about such characters. >> >> But I can't for the life of me figure out how Trygve's code checks for spaces. >> One reason this is an issue is that the behaviour of #digitValue is not >> consistent between systems. >> Character space digitValue >> does not exist in the ANSI standard >> answers -1 in many Smalltalks (which is a pain) >> answers a positive integer that can't be mistake for a digit in my Smalltalk >> raises an exception in some Smalltalks. >> >> This is a comment I now have in my Smalltalk library for #digitValue >> "This is in the Blue Book, but unspecified on non-digits. >> Squeak, Pharo, Dolphin, VW, VAST, and Apple Smalltalk-80 >> answer -1 for characters that are not digits (or ASCII letters), >> which is unfortunate but consistent with Inside Smalltalk >> which specifies this result for non-digits. >> ST/X and GST raise an exception which is worse. >> Digitalk ST/V documentation doesn't specify the result. >> This selector is *much* easier to use safely if it >> returns a 'large' (>= 36) value for non-digits." >> >> Let's compare three versions, the two I compared last time, >> and the "version A" code I discussed before, which to my mind >> is fairly readable. >> >> "Don't add slowness": 1 (normalised time) >> "Trygve's code": 6.5 >> "High level code": 30.6 (or 4.7 times slower than Trygve's) >> >> Here's the "High level code". >> ^(aString allSatisfy: [:each | each isSpace or: [each isDigit]]) and: [ >> |digitsReverse| >> digitsReverse := (aString select: [:each | each isDigit]) reverse. >> digitsReverse size > 1 and: [ >> |evens odds evenSum oddSum| >> odds := digitsReverse withIndexSelect: [:y :i | i odd]. >> evens := digitsReverse withIndexSelect: [:x :i | i even]. >> oddSum := odds detectSum: [:y | y digitValue]. >> evenSum := evens detectSum: [:x | >> #(0 2 4 6 8 1 3 5 7 9) at: x digitValue + 1]. >> (oddSum + evenSum) \\ 10 = 0]] >> >> This is the kind of code I was recommending that Roelof write. >> >> As a rough guide, by counting traversals (including ones inside existing >> methods), I'd expect the "high level" code to be at least 10 times slower >> than the "no added slowness" code. >> >> We are in vehement agreement that there is a time to write high level >> really obvious easily testable and debuggable code, and that's most >> of the time, especially with programming exercises. >> >> I hope that we are also in agreement that factors of 30 (or even 6) >> *can* be a serious problem. I mean, if I wanted something that slow, >> I'd use Ruby. >> >> I hope we are also agreed that (with the exception of investigations >> like this one) the time to hack on something to make it faster is AFTER >> you have profiled it and determined that you have a problem. >> >> But I respectfully suggest that there is a difference taking slowness OUT >> and simply not going out of your way to add slowness in the first place. >> >> I'd also like to remark that my preference for methods that traverse a >> sequence exactly once has more to do with Smalltalk protocols than >> with efficiency. If the only method I perform on an object is #do: >> the method will work just as well for readable streams as for >> collections. If the only method I perform on an object is #reverseDo: >> the method will work just as well for Read[Write]Streams as for >> SequenceReadableCollections, at least in my library. It's just like >> trying to write #mean so that it works for Durations as well as Numbers. >> >> Oh heck, I suppose I should point out that much of the overheads in >> this case could be eliminated by a Self-style compiler doing dynamic >> inlining + loop fusion. There's no reason *in principle*, given enough >> people, money, and time, that the differences couldn't be greatly >> reduced in Pharo. >> >> On Tue, 5 May 2020 at 21:50, Trygve Reenskaug <trygver(a)ifi.uio.no> wrote: >>> >>> Richard, >>> >>> Thank you for looking at the code. It is comforting to learn that the code has been executed for a large number of examples without breaking. The code is not primarily written for execution but for being read and checked by the human end user. It would be nice if we could also check that it gave the right answers, but I don't know how to do that. >>> >>> The first question is: Can a human domain expert read the code and sign their name for its correctness? >>> >>> >>> When this is achieved, a programming expert will transcribe the first code to a professional quality program. This time, the second code should be reviewed by an independent programmer who signs their name for its correct transcription from the first version. >>> >>> --Trygve >>> >>> PS: In his 1991 Turing Award Lecture, Tony Hoare said: "There are two ways of constructing a software design: One way is to make it so simple that there are obviously no deficiencies and the other is to make it so complicated that there are no obvious deficiencies. The first method is far more difficult." >>> >>> --Trygve >>> >>> On tirsdag.05.05.2020 04:41, Richard O'Keefe wrote: >>> >>> As a coding experiment, I adapted Trygve Reenskoug's code to my >>> Smalltalk compiler, put in my code slightly tweaked, and benchmarked >>> them on randomly generated data. >>> >>> Result: a factor of 6.3. >>> >>> In Squeak it was a factor of ten. >>> >>> I had not, in all honesty, expected it to to be so high. >>> >>> On Tue, 5 May 2020 at 02:00, Trygve Reenskaug <trygver(a)ifi.uio.no> wrote: >>> >>> A coding experiment. >>> Consider a Scrum development environment. Every programming team has an end user as a member. >>> The team's task is to code a credit card validity check. >>> A first goal is that the user representative shall read the code and agree that it is a correct rendering of their code checker: >>> >>> luhnTest: trialNumber >>> | s1 odd s2 even charValue reverse | >>> ----------------------------------------------- >>> " Luhn test according to Rosetta" >>> "Reverse the order of the digits in the number." >>> reverse := trialNumber reversed. >>> "Take the first, third, ... and every other odd digit in the reversed digits and sum them to form the partial sum s1" >>> s1 := 0. >>> odd := true. >>> reverse do: >>> [:char | >>> odd >>> ifTrue: [ >>> s1 := s1 + char digitValue. >>> ]. >>> odd := odd not >>> ]. >>> "Taking the second, fourth ... and every other even digit in the reversed digits: >>> Multiply each digit by two and sum the digits if the answer is greater than nine to form partial sums for the even digits" >>> "The subtracting 9 gives the same answer. " >>> "Sum the partial sums of the even digits to form s2" >>> s2 := 0. >>> even := false. >>> reverse do: >>> [:char | >>> even >>> ifTrue: [ >>> charValue := char digitValue * 2. >>> charValue > 9 ifTrue: [charValue := charValue - 9]. >>> s2 := s2 + charValue >>> ]. >>> even := even not >>> ]. >>> "If s1 + s2 ends in zero then the original number is in the form of a valid credit card number as verified by the Luhn test." >>> ^(s1 + s2) asString last = $0 >>> --------------------------------- >>> Once this step is completed, the next step will be to make the code right without altering the algorithm (refactoring). The result should be readable and follow the team's conventions. >>> >>> >>> P.S. code attached. >>> >>> >>> -- >>> >>> The essence of object orientation is that objects collaborate to achieve a goal. >>> Trygve Reenskaug mailto: trygver(a)ifi.uio.no >>> Morgedalsvn. 5A http://folk.uio.no/trygver/ >>> N-0378 Oslo http://fullOO.info >>> Norway Tel: (+47) 468 58 625 > -------------------------------------------- StÃ©phane Ducasse http://stephane.ducasse.free.fr / http://www.pharo.org 03 59 35 87 52 Assistant: Julie Jonas FAX 03 59 57 78 50 TEL 03 59 35 86 16 S. Ducasse - Inria 40, avenue Halley, Parc Scientifique de la Haute Borne, BÃ¢t.A, Park Plaza Villeneuve d'Ascq 59650 France

May 11, 2020

Re: [Pharo-users] [ANN] Pharo Compendium

by StÃ©phane Ducasse

In the past we added metadata to manifest. And the compedium could take advantage of that. We will have to work on it (end of the month). S > On 4 May 2020, at 01:33, Torsten Bergmann <astares(a)gmx.de> wrote: > > Yes - I was developing on P9 and never tried it in Pharo 8. But I checked now - seems to work without > any problem there as well. > > Thx > T. > > > Gesendet: Sonntag, 03. Mai 2020 um 22:36 Uhr > Von: "StÃ©phane Ducasse" <stephane.ducasse(a)inria.fr> > An: "Any question about pharo is welcome" <pharo-users(a)lists.pharo.org> > Betreff: Re: [Pharo-users] [ANN] Pharo Compendium > > It can probably work in P8 because parts of Spec20 are working in P8 but not all the features and widgets like tables and others. > But torsten will tell us. > > On 3 May 2020, at 17:34, Esteban Maringolo <emaringolo@gmail.com[mailto:emaringolo@gmail.com]> wrote: > > Excellent Torsten! > > I guess it will not work in Pharo 8 because of the Spec 2 requirement. Right? > > Regards! > > Esteban A. Maringolo > > -------------------------------------------- StÃ©phane Ducasse http://stephane.ducasse.free.fr / http://www.pharo.org 03 59 35 87 52 Assistant: Julie Jonas FAX 03 59 57 78 50 TEL 03 59 35 86 16 S. Ducasse - Inria 40, avenue Halley, Parc Scientifique de la Haute Borne, BÃ¢t.A, Park Plaza Villeneuve d'Ascq 59650 France

May 11, 2020

Re: [Pharo-users] [Seaside] ZincStaticServerAdaptor starts with different port

by Esteban Maringolo

Hi Sven, I know it is convenient for development, and for deployment I create it on startup via a startup script (that piece of code is from it). But what it is weird (or at least unexpected), and what I'm reporting is that in development the adaptor is restarted at a different port than the one I saved the image with. Esteban A. Maringolo On Sun, May 10, 2020 at 6:50 AM Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > > WAServerManager class>>#startUp sends #startUp to all registered adaptors > > If you saved an image with a running adaptor in it, it will be restarted when the image comes up. This is cool and works fine, but I find it better suited to development than deployment. When deploying I make sure there are no servers of any kind running, and then start them manually and explicitly in a startup script. > > So if something starts up automagically, you must have done something. > > BTW, 80 is a restricted port, but I guess you already know that. > > > On 10 May 2020, at 00:14, Esteban Maringolo <emaringolo(a)gmail.com> wrote: > > > > Hi, > > > > I have a ZnZincStaticServerAdaptor, that after quitting the image and > > starting it again starts in a different port. > > > > So I did: > > WAServerManager default adaptors do: [ :each | WAServerManager default > > unregister: each ]. > > (ZnZincStaticServerAdaptor startOn: 8000 andServeFilesFrom: > > (FileLocator imageDirectory / 'static' ) asFileReference). > > > > And get the server running in the port 8000. [see zn8000.png] > > > > But after restarting the image, it is running in the port 80. [see zn80.png] > > > > Why is it so? > > > > Regards, > > <zn8000.png><zn80.png>_______________________________________________ > > seaside mailing list > > seaside(a)lists.squeakfoundation.org > > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside > > _______________________________________________ > seaside mailing list > seaside(a)lists.squeakfoundation.org > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside

May 10, 2020

Re: [Pharo-users] [Seaside] ZincStaticServerAdaptor starts with different port

by Sven Van Caekenberghe

WAServerManager class>>#startUp sends #startUp to all registered adaptors If you saved an image with a running adaptor in it, it will be restarted when the image comes up. This is cool and works fine, but I find it better suited to development than deployment. When deploying I make sure there are no servers of any kind running, and then start them manually and explicitly in a startup script. So if something starts up automagically, you must have done something. BTW, 80 is a restricted port, but I guess you already know that. > On 10 May 2020, at 00:14, Esteban Maringolo <emaringolo(a)gmail.com> wrote: > > Hi, > > I have a ZnZincStaticServerAdaptor, that after quitting the image and > starting it again starts in a different port. > > So I did: > WAServerManager default adaptors do: [ :each | WAServerManager default > unregister: each ]. > (ZnZincStaticServerAdaptor startOn: 8000 andServeFilesFrom: > (FileLocator imageDirectory / 'static' ) asFileReference). > > And get the server running in the port 8000. [see zn8000.png] > > But after restarting the image, it is running in the port 80. [see zn80.png] > > Why is it so? > > Regards, > <zn8000.png><zn80.png>_______________________________________________ > seaside mailing list > seaside(a)lists.squeakfoundation.org > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside

May 10, 2020

ZincStaticServerAdaptor starts with different port

by Esteban Maringolo

Hi, I have a ZnZincStaticServerAdaptor, that after quitting the image and starting it again starts in a different port. So I did: WAServerManager default adaptors do: [ :each | WAServerManager default unregister: each ]. (ZnZincStaticServerAdaptor startOn: 8000 andServeFilesFrom: (FileLocator imageDirectory / 'static' ) asFileReference). And get the server running in the port 8000. [see zn8000.png] But after restarting the image, it is running in the port 80. [see zn80.png] Why is it so? Regards,

May 9, 2020

Re: [Pharo-users] Matching block parameter in rewrite rule

by Cyril Ferlicot D.

Le 08/05/2020 Ã 02:51, Gabriel Cotelli a Ã©critÂ : > Cyril, something likeÂ this: > > | env rewriter result methods changes | > env := RBBrowserEnvironment default forPackages: (RPackageOrganizer > default packages ). > > rewriter := RBParseTreeRewriter new. > rewriter Â > replace:'`@receiver detect: `@arg1 ifFound: [ :`arg2 | `arg2 ] ifNone: > `@arg3' with: '`@receiver detect: `@arg1 ifNone: `@arg3'. > > changes := Array with: (RBSmalllintChecker runRule: > (RBTransformationRule new > rewriteRule: rewriter; > yourself) > onEnvironment: Â env Â ) builder. > > Â (ChangesBrowser changes: changes) open. Thank you! I got it to work :) Now I'll be able to rewrite much more things :) -- Cyril Ferlicot https://ferlicot.fr

May 8, 2020

Re: [Pharo-users] Matching block parameter in rewrite rule

by Gabriel Cotelli

Cyril, something like this: | env rewriter result methods changes | env := RBBrowserEnvironment default forPackages: (RPackageOrganizer default packages ). rewriter := RBParseTreeRewriter new. rewriter replace:'`@receiver detect: `@arg1 ifFound: [ :`arg2 | `arg2 ] ifNone: `@arg3' with: '`@receiver detect: `@arg1 ifNone: `@arg3'. changes := Array with: (RBSmalllintChecker runRule: (RBTransformationRule new rewriteRule: rewriter; yourself) onEnvironment: env ) builder. (ChangesBrowser changes: changes) open. On Thu, May 7, 2020 at 8:22 PM Cyril Ferlicot D. <cyril.ferlicot(a)gmail.com> wrote: > Hi, > > I was wondering if it was possible with the parse tree rewriter to match > a block parameter? > > I would like to be able, for example, to rewrite: > > `#() detect: #odd ifFound: [ :e | e ] ifNone: [ self toto ]` > > as > > `#() detect: #odd ifNone: [ self toto ]` > > But I don't know how to express this rule with the rewrite rules of Pharo. > > I tried multiple things such as: > > '`@receiver detect: `@arg1 ifFound: [ `@arg2 | `@arg2 ] ifNone: `@arg3 ' > -> '`@receiver detect: `@arg1 ifNone: `@arg3' > > or > > '`@receiver detect: `@arg1 ifFound: [ :`@arg2 | `@arg2 ] ifNone: `@arg3 > ' -> '`@receiver detect: `@arg1 ifNone: `@arg3' > > Without sucess. > > Is there a way to do that or is it a limitation of the rewrite syntax? > > -- > Cyril Ferlicot > https://ferlicot.fr > >

May 8, 2020

Matching block parameter in rewrite rule

by Cyril Ferlicot D.

Hi, I was wondering if it was possible with the parse tree rewriter to match a block parameter? I would like to be able, for example, to rewrite: `#() detect: #odd ifFound: [ :e | e ] ifNone: [ self toto ]` as `#() detect: #odd ifNone: [ self toto ]` But I don't know how to express this rule with the rewrite rules of Pharo. I tried multiple things such as: '`@receiver detect: `@arg1 ifFound: [ `@arg2 | `@arg2 ] ifNone: `@arg3 ' -> '`@receiver detect: `@arg1 ifNone: `@arg3' or '`@receiver detect: `@arg1 ifFound: [ :`@arg2 | `@arg2 ] ifNone: `@arg3 ' -> '`@receiver detect: `@arg1 ifNone: `@arg3' Without sucess. Is there a way to do that or is it a limitation of the rewrite syntax? -- Cyril Ferlicot https://ferlicot.fr

May 7, 2020

Issues browsing/saving a class in a package - seems like its unknown

by PAUL DEBRUICKER

Hi - In Pharo 7 I'm having an issue with a particular class where if I browse it in Calypso I can see & navigate among the methods in the class. If I run some of the methods I get dnu errors for methods I can see in the class browser. If I re-save a method that has the dnu without changing anything then Calypso shows two methods in the same class with the same name. If in a playground I try a MyClass recompile Then I get an "Unknown variable..." dialog that asks me to define the class. If I highlight the MyClass in the playground and "browse" it then Calypso finds the class and opens the browser. If I fileout the class then the fileout has all the methods and two copies with the same name of the one I re-saved above. Because of these issues I cannot save the package using Monticello or iceberg. Any ideas what I can do to fix this situation? At a minimum I reckon I can fileout the bad class, delete it from the image, and file it back in, then save. Thanks Paul

May 7, 2020