February 2011 - Pharo-dev - lists.pharo.org

Re: [Pharo-project] can you tell us on the slugginess of pharo for Kinect

by Ricardo Moran

Yes, the problem was that WorldState>>#runStepMethodsIn: processed only 1 deferred UI message per cycle, which made the kinect server update really slow because I wrapped all ui changes inside a WorldState>>#addDeferredUIMessage:. I added the issue 3562<http://code.google.com/p/pharo/issues/detail?id=3562> with a proposed fix I took from the Etoys image. I now realize I didn't mentioned the connection with Kinect in the issue description... my bad. The issue is closed now so it seems the fix was accepted :) Cheers! Richo On Sun, Feb 6, 2011 at 6:42 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr>wrote: > Hi ricardo > > you mentioned that pharo is really slow for using kinect and can you tell > us more? > > TX > Stef

Feb. 7, 2011

Re: [Pharo-project] [Hudson] Regression Testing

by Serge Stinckwich

Impressive !!! I like that. On Sun, Feb 6, 2011 at 9:31 PM, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > Â Â Â Â https://pharo-ic.lille.inria.fr/hudson/view/Pharo/job/Pharo%20Core%201.3%20… > > The idea is that there are lots of things that are almost never tested because the tests would take too long. > *and* because the tests do something to the image that is not wanted for a normal build. > > Examples > > Â Â Â Â -> recompile everything, run tests again > Â Â Â Â -> decompile everything to text, compile and install > Â Â Â Â -> format everything and compile > Â Â Â Â -> condense changes > Â Â Â Â -> write new sources > Â Â Â Â -> run without sources > Â Â Â Â -> run without sources but encoded temps > Â Â Â Â -> ..... what else??? > > The idea is that this CI Server Project runs all these > after every build, but not as part of main build (so it does not lengthen the build process). > > For now, this just has one test that recompiled the image and does nothing else. > > -- > Marcus Denker Â -- http://www.marcusdenker.de > INRIA Lille -- Nord Europe. Team RMoD. > > > -- Serge Stinckwich UMI UMMISCO 209 (IRD/UPMC), Hanoi, Vietnam Every DSL ends up being Smalltalk http://doesnotunderstand.org/

Feb. 7, 2011

Re: [Pharo-project] Issue 3661 in pharo: fixed the calculation of base in PNGReadWriter>>copyPixelsGray: (floor vs ceil)

by pharo＠googlecode.com

Updates: Status: Fixed Labels: -Type-Squeak Comment #2 on issue 3661 by stephane...(a)gmail.com: fixed the calculation of base in PNGReadWriter>>copyPixelsGray: (floor vs ceil) http://code.google.com/p/pharo/issues/detail?id=3661 I verified it and it works. Attachments: PNGReadWriter-copyPixelsGray.st 2.8 KB

Feb. 6, 2011

Re: [Pharo-project] Issue 3661 in pharo: fixed the calculation of base in PNGReadWriter>>copyPixelsGray: (floor vs ceil)

by pharo＠googlecode.com

Comment #1 on issue 3661 by stephane...(a)gmail.com: fixed the calculation of base in PNGReadWriter>>copyPixelsGray: (floor vs ceil) http://code.google.com/p/pharo/issues/detail?id=3661 drop this file to verify Note that the fixes shown is just a fix of a previous changes so use the complete method Attachments: 00666.png 15.9 KB

Feb. 6, 2011

[Pharo-project] Issue 3680 in pharo: truncated file support in PNMReader

by pharo＠googlecode.com

Status: FixedWaitingToBePharoed Owner: stephane...(a)gmail.com New issue 3680 by stephane...(a)gmail.com: truncated file support in PNMReader http://code.google.com/p/pharo/issues/detail?id=3680 this was not integrated in Squeak but it may be useful. We should check the license. Name: Graphics-jdr.177 Author: jdr Time: 21 January 2011, 11:48:51.942 am UUID: 82017dd9-eb5e-4170-b183-14122bb13439 Ancestors: Graphics-jdr.176 Bug reading a truncated file, added pgm extension =============== Diff against Graphics-jdr.176 =============== Item was changed: ----- Method: PNMReadWriter class>>typicalFileExtensions (in category 'image reading/writing') ----- typicalFileExtensions "Answer a collection of file extensions (lowercase) which files that I can read might commonly have" + ^#('pbm' 'pgm' 'pnm' 'ppm' 'pam')! - ^#('pbm' 'pnm' 'ppm' 'pam')! Item was changed: ----- Method: PNMReadWriter>>cleanLine (in category 'reading') ----- cleanLine "upTo LF or CR, tab as space" | line loop b | + stream atEnd ifTrue:[self error:'End of Data']. + line := WriteStream with: ''. loop := true. [loop] whileTrue: [ b := stream next. b ifNil:[ loop := false "EOS" ] ifNotNil: [ (b = (Character cr) or:[b = Character lf]) ifTrue:[ loop := false. ] ifFalse:[ b = (Character tab) ifTrue:[b := Character space]. line nextPut: b. ] ] ]. ^line contents!

Feb. 6, 2011

Re: [Pharo-project] Issue 3676 in pharo: Remove #example* class messages from HTTPClient

by pharo＠googlecode.com

Updates: Status: Fixed Labels: Milestone-1.3 Comment #1 on issue 3676 by stephane...(a)gmail.com: Remove #example* class messages from HTTPClient http://code.google.com/p/pharo/issues/detail?id=3676 tx!

Feb. 6, 2011

Re: [Pharo-project] [COTDC] 3 - HTTPClient

by StÃ©phane Ducasse

tx!! Stef On Feb 6, 2011, at 10:52 PM, Francisco Ortiz PeÃ±aloza wrote: > Done, see http://code.google.com/p/pharo/issues/detail?id=3676 > > On Sun, Feb 6, 2011 at 7:33 PM, StÃ©phane Ducasse > <stephane.ducasse(a)inria.fr> wrote: >> lol >> >> On Feb 6, 2011, at 9:42 PM, Philippe Marschall wrote: >> >>> On 06.02.2011 18:01, laurent laffont wrote: >>>> Comment Of The Day Contest - One Day One Comment >>>> Rules: Each day a not commented class is elected. Each day the best >>>> comment will be integrated with name of the author(s). >>>> >>>> >>>> Today: HTTPClient >>> >>> <trollmode> >>> I'm a good example of how not to write code. Turn around, walk away, >>> fast. I'm fully of bugs. >>> >>> Somebody at some point thought that HTTP is a simple "hacker with >>> telnet"-protocol. It's not. >>> </trollmode> >>> >>> Philippe >>> >>> >> >> >> >

Feb. 6, 2011

[Pharo-project] Issue 3679 in pharo: Refactored and extended CompilerExceptionsTest

by pharo＠googlecode.com

Status: FixedWaitingToBePharoed Owner: stephane...(a)gmail.com CC: laurent....(a)gmail.com Labels: Milestone-1.3 Difficulty-Easy New issue 3679 by stephane...(a)gmail.com: Refactored and extended CompilerExceptionsTest http://code.google.com/p/pharo/issues/detail?id=3679 Levente Uzonyi uploaded a new version of Tests to project The Trunk: http://source.squeak.org/trunk/Tests-ul.110.mcz ==================== Summary ==================== Name: Tests-ul.110 Author: ul Time: 12 January 2011, 2:03:05.15 pm UUID: f26a4053-7b86-3348-954f-5f1ba3480af1 Ancestors: Tests-cmm.109 Refactored and extended CompilerExceptionsTest. One test case is failing. The corresponding bug report is here: http://bugs.squeak.org/view.php?id=7572 . =============== Diff against Tests-cmm.109 =============== Item was changed: TestCase subclass: #CompilerExceptionsTest + instanceVariableNames: 'text' - instanceVariableNames: '' classVariableNames: '' poolDictionaries: '' category: 'Tests-Compiler'! Item was added: + ----- Method: CompilerExceptionsTest>>compile: (in category 'private') ----- + compile: sourceString + + text := sourceString. + self class + compileSilently: text + classified: 'generated' + notifying: self! Item was added: + ----- Method: CompilerExceptionsTest>>compiling:shouldRaise: (in category 'private') ----- + compiling: sourceCode shouldRaise: exceptionClass + + self should: [ self compile: sourceCode ] raise: exceptionClass! Item was removed: - ----- Method: CompilerExceptionsTest>>griffle (in category 'as yet unclassified') ----- - griffle | goo |! Item was added: + ----- Method: CompilerExceptionsTest>>removeGeneratedMethods (in category 'private') ----- + removeGeneratedMethods + + self class removeCategory: 'generated'! Item was added: + ----- Method: CompilerExceptionsTest>>setUp (in category 'running') ----- + setUp + + self removeGeneratedMethods! Item was added: + ----- Method: CompilerExceptionsTest>>tearDown (in category 'running') ----- + tearDown + + self removeGeneratedMethods! Item was added: + ----- Method: CompilerExceptionsTest>>testAmbiguousSelector (in category 'tests') ----- + testAmbiguousSelector + + self + compiling: 'griffle ^1--1' + shouldRaise: AmbiguousSelector; + compiling: 'griffle ^1@-1' + shouldRaise: AmbiguousSelector; + compiling: 'griffle ^1+-1' + shouldRaise: AmbiguousSelector ! Item was changed: ----- Method: CompilerExceptionsTest>>testUndeclaredVariable (in category 'tests') ----- testUndeclaredVariable + self + compiling: 'griffle ^ goo' + shouldRaise: UndeclaredVariable; + compiling: 'griffle ^ [ goo ] value' + shouldRaise: UndeclaredVariable! - should: - [self class - compile: 'griffle ^ goo' - notifying: self] - raise: UndeclaredVariable! Item was changed: ----- Method: CompilerExceptionsTest>>testUndefinedVariable (in category 'tests') ----- testUndefinedVariable + self + compiling: 'griffle | goo | ^ goo' + shouldRaise: UndefinedVariable; + compiling: 'griffle [ | goo | ^ goo ] value' + shouldRaise: UndefinedVariable! - should: - [self class - compile: 'griffle | goo | ^ goo' - notifying: self] - raise: UndefinedVariable! Item was changed: ----- Method: CompilerExceptionsTest>>testUnknownSelector (in category 'tests') ----- testUnknownSelector + self + compiling: 'griffle self reallyHopeThisIsntImplementedAnywhere' + shouldRaise: UnknownSelector; + compiling: 'griffle [ self reallyHopeThisIsntImplementedAnywhere ] value' + shouldRaise: UnknownSelector! - should: - [self class - compile: 'griffle self reallyHopeThisIsntImplementedAnywhere' - notifying: self] - raise: UnknownSelector! Item was changed: ----- Method: CompilerExceptionsTest>>testUnusedVariable (in category 'tests') ----- testUnusedVariable + self + compiling: 'griffle | goo | ^nil' + shouldRaise: UnusedVariable; + compiling: 'griffle ^[ | goo | ]' + shouldRaise: UnusedVariable! - should: - [self class - compile: self unusedVariableSource - notifying: self] - raise: UnusedVariable! Item was changed: ----- Method: CompilerExceptionsTest>>text (in category 'emulating') ----- text + + ^text! - ^ self unusedVariableSource! Item was removed: - ----- Method: CompilerExceptionsTest>>unusedVariableSource (in category 'private') ----- - unusedVariableSource - ^ 'griffle - | goo | - ^ nil'!

Feb. 6, 2011

[Pharo-project] Issue 3678 in pharo: Refactored and extended CompilerExceptionsTest

by pharo＠googlecode.com

Status: FixedWaitingToBePharoed Owner: stephane...(a)gmail.com CC: laurent....(a)gmail.com Labels: Milestone-1.3 Difficulty-Easy New issue 3678 by stephane...(a)gmail.com: Refactored and extended CompilerExceptionsTest http://code.google.com/p/pharo/issues/detail?id=3678 Levente Uzonyi uploaded a new version of Tests to project The Trunk: http://source.squeak.org/trunk/Tests-ul.110.mcz ==================== Summary ==================== Name: Tests-ul.110 Author: ul Time: 12 January 2011, 2:03:05.15 pm UUID: f26a4053-7b86-3348-954f-5f1ba3480af1 Ancestors: Tests-cmm.109 Refactored and extended CompilerExceptionsTest. One test case is failing. The corresponding bug report is here: http://bugs.squeak.org/view.php?id=7572 . =============== Diff against Tests-cmm.109 =============== Item was changed: TestCase subclass: #CompilerExceptionsTest + instanceVariableNames: 'text' - instanceVariableNames: '' classVariableNames: '' poolDictionaries: '' category: 'Tests-Compiler'! Item was added: + ----- Method: CompilerExceptionsTest>>compile: (in category 'private') ----- + compile: sourceString + + text := sourceString. + self class + compileSilently: text + classified: 'generated' + notifying: self! Item was added: + ----- Method: CompilerExceptionsTest>>compiling:shouldRaise: (in category 'private') ----- + compiling: sourceCode shouldRaise: exceptionClass + + self should: [ self compile: sourceCode ] raise: exceptionClass! Item was removed: - ----- Method: CompilerExceptionsTest>>griffle (in category 'as yet unclassified') ----- - griffle | goo |! Item was added: + ----- Method: CompilerExceptionsTest>>removeGeneratedMethods (in category 'private') ----- + removeGeneratedMethods + + self class removeCategory: 'generated'! Item was added: + ----- Method: CompilerExceptionsTest>>setUp (in category 'running') ----- + setUp + + self removeGeneratedMethods! Item was added: + ----- Method: CompilerExceptionsTest>>tearDown (in category 'running') ----- + tearDown + + self removeGeneratedMethods! Item was added: + ----- Method: CompilerExceptionsTest>>testAmbiguousSelector (in category 'tests') ----- + testAmbiguousSelector + + self + compiling: 'griffle ^1--1' + shouldRaise: AmbiguousSelector; + compiling: 'griffle ^1@-1' + shouldRaise: AmbiguousSelector; + compiling: 'griffle ^1+-1' + shouldRaise: AmbiguousSelector ! Item was changed: ----- Method: CompilerExceptionsTest>>testUndeclaredVariable (in category 'tests') ----- testUndeclaredVariable + self + compiling: 'griffle ^ goo' + shouldRaise: UndeclaredVariable; + compiling: 'griffle ^ [ goo ] value' + shouldRaise: UndeclaredVariable! - should: - [self class - compile: 'griffle ^ goo' - notifying: self] - raise: UndeclaredVariable! Item was changed: ----- Method: CompilerExceptionsTest>>testUndefinedVariable (in category 'tests') ----- testUndefinedVariable + self + compiling: 'griffle | goo | ^ goo' + shouldRaise: UndefinedVariable; + compiling: 'griffle [ | goo | ^ goo ] value' + shouldRaise: UndefinedVariable! - should: - [self class - compile: 'griffle | goo | ^ goo' - notifying: self] - raise: UndefinedVariable! Item was changed: ----- Method: CompilerExceptionsTest>>testUnknownSelector (in category 'tests') ----- testUnknownSelector + self + compiling: 'griffle self reallyHopeThisIsntImplementedAnywhere' + shouldRaise: UnknownSelector; + compiling: 'griffle [ self reallyHopeThisIsntImplementedAnywhere ] value' + shouldRaise: UnknownSelector! - should: - [self class - compile: 'griffle self reallyHopeThisIsntImplementedAnywhere' - notifying: self] - raise: UnknownSelector! Item was changed: ----- Method: CompilerExceptionsTest>>testUnusedVariable (in category 'tests') ----- testUnusedVariable + self + compiling: 'griffle | goo | ^nil' + shouldRaise: UnusedVariable; + compiling: 'griffle ^[ | goo | ]' + shouldRaise: UnusedVariable! - should: - [self class - compile: self unusedVariableSource - notifying: self] - raise: UnusedVariable! Item was changed: ----- Method: CompilerExceptionsTest>>text (in category 'emulating') ----- text + + ^text! - ^ self unusedVariableSource! Item was removed: - ----- Method: CompilerExceptionsTest>>unusedVariableSource (in category 'private') ----- - unusedVariableSource - ^ 'griffle - | goo | - ^ nil'!

Feb. 6, 2011

[Pharo-project] Issue 3677 in pharo: A to:do: loop cannot modify the limit inside the block

by pharo＠googlecode.com

Status: FixedWaitingToBePharoed Owner: stephane...(a)gmail.com Labels: Milestone-1.3 Type-Squeak New issue 3677 by stephane...(a)gmail.com: A to:do: loop cannot modify the limit inside the block http://code.google.com/p/pharo/issues/detail?id=3677 Nicolas Cellier uploaded a new version of Compiler to project The Trunk: http://source.squeak.org/trunk/Compiler-nice.184.mcz ==================== Summary ==================== Name: Compiler-nice.184 Author: nice Time: 12 January 2011, 10:09:27.038 pm UUID: ba12fb9c-6c57-429b-b415-1188e1faba8b Ancestors: Compiler-nice.183 Fix DecompilerTests>>#testDecompileLoopWithMovingLimit A to:do: loop cannot modify the limit inside the block =============== Diff against Compiler-nice.183 =============== Item was changed: ----- Method: MessageNode>>toDoFromWhileWithInit: (in category 'macro transformations') ----- toDoFromWhileWithInit: initStmt "Return nil, or a to:do: expression equivalent to this whileTrue:" | variable increment limit toDoBlock body test | (selector key == #whileTrue: and: [initStmt isAssignmentNode and: [initStmt variable isTemp]]) ifFalse: [^nil]. body := arguments last statements. variable := initStmt variable. increment := body last toDoIncrement: variable. (increment == nil or: [receiver statements size ~= 1]) ifTrue: [^nil]. test := receiver statements first. "Note: test chould really be checked that <= or >= comparison jibes with the sign of the (constant) increment" (test isMessageNode and: [(limit := test toDoLimit: variable) notNil]) ifFalse: [^nil]. + "The block must not overwrite the limit" + (limit isVariableNode and: [body anySatisfy: [:e | e isAssignmentNode and: [e variable = limit]]]) + ifTrue: [^nil]. toDoBlock := BlockNode statements: body allButLast returns: false. toDoBlock arguments: (Array with: variable). variable scope: -1. variable beBlockArg. ^MessageNode new receiver: initStmt value selector: (SelectorNode new key: #to:by:do: code: #macro) arguments: (Array with: limit with: increment with: toDoBlock) precedence: precedence!

Feb. 6, 2011