Squeak-dev June 2011

squeak-dev@lists.squeakfoundation.org

77 participants
236 discussions

Re: Self new in arrowPrototype
by Rodney Polkinghorne 07 Jun '11

07 Jun '11

Ralph Johnson, replying to me: > "new" should just copy the existing prototype. Thanks. That makes general sense, but I'm still confused about some details. Suppose I want to define IndicatorMorph, an arrow-like subclass of PolygonMorph that will be attached to a DialMorph. 1. Presumably, IndicatorMorph class>>initialize constructs a prototype by sending arrowPrototype to PolygonMorph class. Where should this be stored? 2. IndicatorMorph>>new copies the prototype, giving another PolygonMorph. Then a miracle occurs. Finally, the new method has an IndicatorMorph, which it answers. How do mortal programers reproduce the miracle? If this is covered in Objects for Dummies, please tell me where to find a copy. > I assume that PolygonMorph doesn't define "new". If it did, would > you want this method to call it? PolygonMorph does define new - it answers a brightly coloured triangle. The arrowPrototype method is just like new, except it has arrows: #forward instead of arrows: #none, and a few things like that. > What would happen if you made a subclass of PolygonMorph? When I tried it, its new method went into an infinite chain of messages, and I emailed the list. > I bet that the purpose of using the > prototype pattern here is to avoid having to make subclasses of > PolygonMorph In that case, how should IndicatorMorph work? I can duplicate PolygonMorph>>arrowPrototype and call it IndicatorMorph>>new, but that seems wrong for all kinds of reasons. Thanks for your help. Rodney

1 0

Daily Commit Log
by commits＠source.squeak.org 07 Jun '11

07 Jun '11

Changes to Trunk (http://source.squeak.org/trunk.html) in the last 24 hours: http://lists.squeakfoundation.org/pipermail/packages/2011-June/004742.html Name: Kernel-wiz.592 Ancestors: Kernel-wiz.591 asNonFraction with updated comments. ceiling speed up for positive numbers by mirroring floor code rathar than calling floor. ============================================= http://lists.squeakfoundation.org/pipermail/packages/2011-June/004743.html Name: Kernel-wiz.591 Ancestors: Kernel-eem.590 This is asNonFraction defined only for Number and Fraction but not Object. Fractions are converted to Floats other numbers are left alone. Non numbers including nil will raise a DNU. ============================================= http://lists.squeakfoundation.org/pipermail/packages/2011-June/004744.html Name: Tools-cmm.358 Ancestors: Tools-cmm.357 When reformulating a MessageSet list, only signal a change in the autoSelectString if there is, in fact, an autoSelectString present. =============================================

1 0

[BUG] [] storeString goes into a tight infinite loop.
by Jerome Peace 06 Jun '11

06 Jun '11

[squeak-dev] [BUG] [] storeString goes into a tight infinite loop. Dave and Elliot and TWIMC, Given that you can defend your end of things for not causing the problem. Who should take responsibility for fixing it? So storeString as written has limitations. Where does that permit me as a naive users to type two words that will crash my system when executed? I would expect [] storeString to print '[]'. What else should it print? Why should it try to do more than that? If it can't do that then why can't it give me an error message? Who is supposed to take responsibility for the behavior of these two words. Why does the user have to deal with a crashed image? Where is the documentation explaining what I should do to get what I really wanted? It is all well and good not to take the blame but where is the one who takes responsibility for this? Yours in perplexity and service, --Jerome Peace PS. M7639 [BUG] [] storeString goes into a tight infinite loop.

5 4

Re: [squeak-dev] Re: Testing Seaside in Squeak4.3alpha-11454
by Hannes Hirzel 06 Jun '11

06 Jun '11

On 6/3/11, Dale Henrichs <dhenrich(a)vmware.com> wrote: > > > ----- Original Message ----- > | From: "Levente Uzonyi" <leves(a)elte.hu> > | To: "Chris Muller" <ma.chris.m(a)gmail.com> > | Cc: "The general-purpose Squeak developers list" > <squeak-dev(a)lists.squeakfoundation.org> > | Sent: Friday, June 3, 2011 9:49:46 AM > | Subject: Re: [squeak-dev] Re: Testing Seaside in Squeak4.3alpha-11321 > | > | On Fri, 3 Jun 2011, Chris Muller wrote: > | > <snip> > | So the simple solution is to be explicit and change the installer > | script > | from loading ConfigurationOfOmniBrowser, to load > | ConfigurationOfOmniBrowser-xyz.123. The same configuration should be > | able > | to load OB into Squeak 4.1, 4.2, head, etc. > > Levente, > > I have been toying with the idea of making it possible to specify the > version of the configuration along with the version of the package in > Metacello, for exactly the reasons you are describing ... > > Dale > > Dale Yes, that seems to be a useful addition to Metacello. Note 1 is about doing a load test Note 2 is about the ConfigurationOfSeaside30 as concerns Squeak. The problem is with ConfigurationOfOmniBrowser, which is loaded there, right? --Hannes Note 1: -- for the record (retest): I just did a re-test to load Seaside into a fresh Squeak4.3alpha-11454 image (as of now) with the code snippet from the 'Extending the system' workspace. "Seaside 3.0 http://www.seaside.st" (Installer ss project: 'MetacelloRepository') install: 'ConfigurationOfSeaside30'. (Smalltalk at: #ConfigurationOfSeaside30) load. (Smalltalk at: #WAPharoServerAdaptorBrowser) open. It has the same problem as mentioned at the start of this thread (Buttons in Seaside Control Panel do not work because of a missing fix Andreas Raab has provided in the meantime; not the right version of OmniBrowser loaded). Note 2: >From the class ConfigurationOfSeaside30 method baseline304: spec Is this the method which is picked? Or where do I find the description you are referring to? We have to look at the part 'squeakCommon', where ConfigurationOfOmniBrowser is loaded. There the solution has to come from. baseline304: spec <version: '3.0.4-baseline' imports: #('3.0.4-baseline-commonBaseline' )> spec for: #'common' do: [ spec blessing: #'baseline'. spec description: ''. spec package: 'Seaside-FileSystem' with: [ spec requires: #('Seaside-Core' ); repository: 'http://www.squeaksource.com/Seaside30LGPL'. ]; package: 'Seaside-Tests-FileSystem' with: [ spec requires: #('Seaside-FileSystem' ); repository: 'http://www.squeaksource.com/Seaside30LGPL'. ]. spec group: 'Core' with: #('Base' 'Development' 'RSS-Core' 'Javascript-Core' 'Prototype-Core' 'Scriptaculous-Core' 'JQuery-Core' 'JQuery-UI' 'Seaside-Email' 'Seaside-HTML5' 'Seaside-InternetExplorer' 'Seaside-Examples' 'RSS-Examples' 'Seaside-Tools-Web' 'Scriptaculous-Components' 'Seaside-Swazoo' 'Seaside-Adaptors-Swazoo' 'Seaside-Welcome' 'Seaside-Tools-OmniBrowser' 'Seaside-FileSystem' ); group: 'Tests' with: #('Base Tests' 'Development Tests' 'RSS-Tests-Core' 'Javascript-Tests-Core' 'Seaside-Tests-Email' 'Seaside-Tests-Functional' 'Prototype-Tests-Core' 'Scriptaculous-Tests-Core' 'Scriptaculous-Tests-Components' 'JQuery-Tests-Core' 'JQuery-Tests-UI' 'Seaside-Tests-HTML5' 'Seaside-Tests-InternetExplorer' 'Seaside-Tests-Examples' 'Seaside-Tests-Tools-Web' 'Seaside-Tests-UTF8' 'Seaside-Tests-FileSystem' 'Seaside-Tests-Welcome' ). ]. spec for: #'squeakCommon' do: [ spec project: 'Kom' with: [ spec className: 'ConfigurationOfKomHttpServer'; versionString: #'bleedingEdge'; repository: 'http://www.squeaksource.com/MetacelloRepository' ]; project: 'OB' with: [ spec className: 'ConfigurationOfOmniBrowser'; versionString: #'stable'; loads: #('Core' ); repository: 'http://www.squeaksource.com/MetacelloRepository' ]; project: 'SPort2' with: [ spec className: 'ConfigurationOfSPort2'; versionString: '2.31'; repository: 'http://www.squeaksource.com/MetacelloRepository' ]. spec package: 'Javascript-Core' with: [ spec includes: #('Javascript-Pharo-Core' ). ]; package: 'Seaside-Core' with: [ spec includes: #('Seaside-Pharo-Core' ). ]; package: 'Seaside-Canvas' with: [ spec includes: #('Seaside-Pharo-Canvas' ). ]; package: 'Seaside-Development' with: [ spec includes: #('Seaside-Pharo-Development' ). ]; package: 'Seaside-Email' with: [ spec includes: #('Seaside-Pharo-Email' ). ]; package: 'Seaside-Environment' with: [ spec includes: #('Seaside-Pharo-Environment' ). ]; package: 'Seaside-Flow' with: [ spec includes: #('Seaside-Pharo-Flow' ). ];

2 1

converting lists of integers and ranges of integers from a string
by Ralph Boland 06 Jun '11

06 Jun '11

I am creating a couple of Squeak (and Pharo) classes that I will eventually release. I am deciding on syntax and would like to know Squeakers opinions given that any Squeakers who use these classes will be stuck with the syntax. The first class contains a sorted list of cardinals (non-negative integers) and ranges of cardinals. an instance of this class can be created from a string where the string has as examples: '0,3-5,9,11,17-89' (1) '7-11,15,30-' (2) Note that the second example is correct: The '30-' substring at the end implies the list includes all the cardinals from 30 to plus infinity. I am happy with this syntax. My problem is my second class which is a similar sorted list but which also allows negative integers, The question is how should I store a sorted list of integers and integer ranges in a string such that I can read them in? The obvious way would be as indicated by the example: '-9,-5--3,-1-4,7,11-15' (3) (method 0) This is the same as for the first class except negative numbers are now allowed. However, I consider this syntax difficult to read and it presents difficulties in including a range starting with minus infinity. For example does '-9' indicate the range from minus infinity to 9 or just the number -9? Some possible alternative solutions: 1) Use a different character to indicate ranges such as: '0,3@5,9,11,17@89' [example 1 again] (method 1) and: -9,-5@-3,-1@4,7,11@15 [example 3 again] (method 1) This goes against convention and requires more learning by the user to learn the syntax. 2) Mark the list of negative numbers such as: '[9,5-3,1],0-4,7,11-15' [example 3 again] (method 2a) or: '[1,3-5,9],0-4,7,11-15' [example 3 again] (method 2b) Both methods have the problem that a range starting with a negative and ending with a cardinal must be split into two parts. Method 2b has the advantage that the code from the first class can be used to read the negative numbers in and then again for the cardinal numbers. 3) Use method 1 for negatives but continue to use the minus sign for ranges of cardinals: -9,-5@-3,-1@4,7,11-15 [example 3 again] (method 3) I want to know which of methods 0,1,2a,2b,3 is the best and am also interested in other proposals. Opinions much appreciated. If there is a conventional solution to this problem I would of course like to know what it is and where it is used. Regards, Ralph Boland

4 4

The Trunk: Tools-cmm.358.mcz
by commits＠source.squeak.org 06 Jun '11

06 Jun '11

Chris Muller uploaded a new version of Tools to project The Trunk: http://source.squeak.org/trunk/Tools-cmm.358.mcz ==================== Summary ==================== Name: Tools-cmm.358 Author: cmm Time: 6 June 2011, 9:58:22.054 am UUID: 08000000-1508-381f-1508-381f14000000 Ancestors: Tools-cmm.357 When reformulating a MessageSet list, only signal a change in the autoSelectString if there is, in fact, an autoSelectString present. =============== Diff against Tools-cmm.357 =============== Item was changed: ----- Method: MessageSet>>reformulateList (in category 'message functions') ----- reformulateList "The receiver's messageList has been changed; rebuild it" super reformulateList. self changed: #messageList ; changed: #messageListIndex. self contentsChanged. + autoSelectString ifNotNil: [ self changed: #autoSelect ]! - self changed: #autoSelect!

1 0

Re: [squeak-dev] The Inbox: Kernel-wiz.592.mcz
by Bert Freudenberg 06 Jun '11

06 Jun '11

Uh oh. Your #asNonFraction addition is fine now, I would have committed that. But now I'll have to wait until someone confirms the changed ceiling code is good. Nicolas maybe? - Bert - On 29.05.2011, at 22:32, commits(a)source.squeak.org wrote: > A new version of Kernel was added to project The Inbox: > http://source.squeak.org/inbox/Kernel-wiz.592.mcz > > ==================== Summary ==================== > > Name: Kernel-wiz.592 > Author: wiz > Time: 29 May 2011, 6:40:45.398 pm > UUID: ce3c0c86-e4f2-4bf5-8c2a-efbc708af821 > Ancestors: Kernel-wiz.591 > > asNonFraction with updated comments. > > ceiling speed up for positive numbers by mirroring floor code rathar than calling floor. > > =============== Diff against Kernel-wiz.591 =============== > > Item was changed: > ----- Method: Fraction>>asNonFraction (in category 'converting') ----- > asNonFraction > + "Answer a number equivalent to the receiver that is not a fraction." > - "Answer the receiver asFloat. This is here for the benifit of the copyBits primitive which can handle floats but not fractions. It has to be stated this way so we don't turn integers into floats. See inheritance for backstop" > > + > ^self asFloat! > > Item was changed: > ----- Method: Number>>asNonFraction (in category 'converting') ----- > asNonFraction > + "Answer a number equivalent to the receiver that is not a fraction. See Fraction for active method." > - "Answer the receiver. This is here for the benifit of the copyBits primitive which can handle floats but not fractions. It has to be stated this way so we don't turn integers into floats. See Fraction for active method" > > ^self ! > > Item was changed: > ----- Method: Number>>ceiling (in category 'truncation and round off') ----- > ceiling > + "Answer the integer nearest the receiver toward infinity." > - "Answer the integer nearest the receiver toward positive infinity." > > + | truncation | > + truncation := self truncated. > + self <= 0 ifTrue: [^truncation]. > + self = truncation > + ifTrue: [^truncation] > + ifFalse: [^truncation + 1]! > - self <= 0.0 > - ifTrue: [^self truncated] > - ifFalse: [^self negated floor negated]! > >

2 2

The Trunk: Kernel-wiz.591.mcz
by commits＠source.squeak.org 06 Jun '11

06 Jun '11

Bert Freudenberg uploaded a new version of Kernel to project The Trunk: http://source.squeak.org/trunk/Kernel-wiz.591.mcz ==================== Summary ==================== Name: Kernel-wiz.591 Author: wiz Time: 29 May 2011, 4:11:53.048 pm UUID: 53586ad8-595c-477a-ac55-19471c286e26 Ancestors: Kernel-eem.590 This is asNonFraction defined only for Number and Fraction but not Object. Fractions are converted to Floats other numbers are left alone. Non numbers including nil will raise a DNU. =============== Diff against Kernel-eem.590 =============== Item was added: + ----- Method: Fraction>>asNonFraction (in category 'converting') ----- + asNonFraction + "Answer the receiver asFloat. This is here for the benifit of the copyBits primitive which can handle floats but not fractions. It has to be stated this way so we don't turn integers into floats. See inheritance for backstop" + + ^self asFloat! Item was added: + ----- Method: Number>>asNonFraction (in category 'converting') ----- + asNonFraction + "Answer the receiver. This is here for the benifit of the copyBits primitive which can handle floats but not fractions. It has to be stated this way so we don't turn integers into floats. See Fraction for active method" + + ^self !

1 0

The Trunk: Kernel-wiz.592.mcz
by commits＠source.squeak.org 06 Jun '11

06 Jun '11

Bert Freudenberg uploaded a new version of Kernel to project The Trunk: http://source.squeak.org/trunk/Kernel-wiz.592.mcz ==================== Summary ==================== Name: Kernel-wiz.592 Author: wiz Time: 29 May 2011, 6:40:45.398 pm UUID: ce3c0c86-e4f2-4bf5-8c2a-efbc708af821 Ancestors: Kernel-wiz.591 asNonFraction with updated comments. ceiling speed up for positive numbers by mirroring floor code rathar than calling floor. =============== Diff against Kernel-eem.590 =============== Item was added: + ----- Method: Fraction>>asNonFraction (in category 'converting') ----- + asNonFraction + "Answer a number equivalent to the receiver that is not a fraction." + + + ^self asFloat! Item was added: + ----- Method: Number>>asNonFraction (in category 'converting') ----- + asNonFraction + "Answer a number equivalent to the receiver that is not a fraction. See Fraction for active method." + + ^self ! Item was changed: ----- Method: Number>>ceiling (in category 'truncation and round off') ----- ceiling + "Answer the integer nearest the receiver toward infinity." - "Answer the integer nearest the receiver toward positive infinity." + | truncation | + truncation := self truncated. + self <= 0 ifTrue: [^truncation]. + self = truncation + ifTrue: [^truncation] + ifFalse: [^truncation + 1]! - self <= 0.0 - ifTrue: [^self truncated] - ifFalse: [^self negated floor negated]!

1 0

MNUs on Windows "right click"
by Frank Shearar 05 Jun '11

05 Jun '11

I've run twice today into a situation where, after having failing tests in the TestRunner, "right click"ing the desktop results in a MessageNotUnderstood: MethodReference>>do:. The "aCollection" being passed around is actually a MethodReference: MessageTrace >> browseAllImplementorsOf:. If you open a Trunk (11458), run the ToolsTest suite, you'll see two failing tests. Click the top failing test, and the debugger pops up. Now yellow-click (?) the desktop, and you'll see the error. Close the failing-test-debugger, and the problem disappears! Reported on Mantis: http://bugs.squeak.org/view.php?id=7642 frank

1 0

Jump to page: