August 2013 - Pharo-dev - lists.pharo.org

[regression reporter]regression occurred
by no-reply＠ci.inria.fr Aug. 27, 2013

Aug. 27, 2013

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/label=win/… 1 regressions found. Zinc.Zodiac.ZnHTTPSTests.testGetPharoVersion

1 0

[update 3.0] #30345
by Marcus Denker Aug. 27, 2013

Aug. 27, 2013

30345 ----- 11462 Wrong comment in testMethodsOfTheClassShouldNotBeRepeatedInItsSuperclasses https://pharo.fogbugz.com/f/cases/11462 11463 RBRefactoringChangeTests >> #testAddMetaclassPattern takes too much time https://pharo.fogbugz.com/f/cases/11463 11460 TabModel>>closeable: is not working https://pharo.fogbugz.com/f/cases/11460 Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/Tests-MarcusDenker.615.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Widgets-MarcusDenker.229… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Refactoring-Tests-Changes-Mar…

2 1

Can someone explain me what this code doing?
by Igor Stasenko Aug. 26, 2013

Aug. 26, 2013

NewValueHolder>>contents: anObject | oldValue | lock ifTrue: [ ^ self ]. lock := true. oldValue := contents. contents := anObject. [ self contentsChanged: oldValue ] ensure: [ lock := false ]. ^ contents it is really interesting.. it appears you cannot change the contents of value holder (as you can see it will simply ignore it) when it 'locked' during announcing the change event. i know, this will prevent it from entering recursive loop, but i would instead throw an error in such case(s) because code which modifying the value as a reaction on value change is something really stupid and must be fixed, instead of putting workaround which will block the value holder from accepting new value(s).. and that is really bad: - notification(s) about some property change should not block the value from being able to be changed -- Best regards, Igor Stasenko.

1 0

Need of a review
by Damien Cassou Aug. 26, 2013

Aug. 26, 2013

Could someone please review this change? Extract the logic that opens the .sources and .changes files This logic is currently in the SmalltalkImage class. The code is hard to read and enhance. Extracting the logic to its own class will fix these issues. https://pharo.fogbugz.com/default.asp?11426 -- Damien Cassou http://damiencassou.seasidehosting.st "Success is the ability to go from one failure to another without losing enthusiasm." Winston Churchill

2 2

Re: [Pharo-dev] what is the new way to do Smalltalk at: #MyClass?
by Fernando Olivero Aug. 26, 2013

Aug. 26, 2013

I prefer to evaluate Smalltalk globals classNamed: #MyClass Fernando On Sat, Aug 24, 2013 at 11:57 AM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > > Is it > > asClass? > > Stef >

10 21

[update 3.0] #30344
by Marcus Denker Aug. 26, 2013

Aug. 26, 2013

30344 ----- 6821 Inconsistent String>>asDate behaviour https://pharo.fogbugz.com/f/cases/6821 11446 recategorize compareWith:ifLesser:ifEqual:ifGreater: https://pharo.fogbugz.com/f/cases/11446 11431 #sorted: is broken for some collections https://pharo.fogbugz.com/f/cases/11431 11459 Failing test: .testLocalMethodsOfTheClassShouldNotBeRepeatedInItsTraits https://pharo.fogbugz.com/f/cases/11459 (this changes Athens, I will commit this back into the Athens repo later) Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/Tabs-MarcusDenker.29.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-MarcusDenker.1575.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/CollectionsTests-MarcusDenker… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Collections-Traits-MarcusDenk… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Collections-Sequenceable-Marc… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Athens-Cairo-MarcusDenker.51.…

1 0

[update 3.0] #30343
by Marcus Denker Aug. 26, 2013

Aug. 26, 2013

30343 ----- 11447 Improve Chronology https://pharo.fogbugz.com/f/cases/11447 11456 RBLiteralValueNode does not Understand shortPrintOn: https://pharo.fogbugz.com/f/cases/11456 Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/Tools-MarcusDenker.1219.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/System-Support-MarcusDenker.9… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Network-Mail-MarcusDenker.21.… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Morphic-Base-MarcusDenker.76.… http://smalltalkhub.com/mc/Pharo/Pharo30/main/KernelTests-MarcusDenker.552.… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-MarcusDenker.1574.diff

2 1

[regression reporter]regression occurred
by no-reply＠ci.inria.fr Aug. 26, 2013

Aug. 26, 2013

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/label=win/… 1 regressions found. Tests.Release.ReleaseTest.testLocalMethodsOfTheClassShouldNotBeRepeatedInItsTraits

1 0

[regression reporter]regression occurred
by no-reply＠ci.inria.fr Aug. 26, 2013

Aug. 26, 2013

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/label=mac/… 1 regressions found. Tests.Release.ReleaseTest.testLocalMethodsOfTheClassShouldNotBeRepeatedInItsTraits

1 0

[regression reporter]regression occurred
by no-reply＠ci.inria.fr Aug. 26, 2013

Aug. 26, 2013

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/label=linu… 1 regressions found. Tests.Release.ReleaseTest.testLocalMethodsOfTheClassShouldNotBeRepeatedInItsTraits

1 0