- Pharo-dev - lists.pharo.org

Cleaning up... baseClass, instanceSide, theNonMetaclass
by Stephane Ducasse Feb. 13, 2018

Feb. 13, 2018

Hi We already got this discussion and I plan to address it. Now I discovered that in addition to instanceSide and theNonMetaclass we have - classClass and baseClass - theMetaclass and theNonMetaclass - instanceSide and classSide https://pharo.fogbugz.com/f/cases/20941/deprecate-theMetaclass-and-the-theN… Even if instanceSide and classSide come from UI concerns I think that there are much better than the others. So have you any concerns and prefer one of the other couples? Because we should stop accumulate cruft in the system. We do not need three ways to do the same. Stef

4 4

[Pharo 7.0-dev] Build #539: 21330-CannotDeleteFileException-should-be-thrown-instead-of-FileException-
by ci-pharo-ci-jenkins2＠inria.fr Feb. 13, 2018

Feb. 13, 2018

There is a new Pharo build available! The status of the build #539 was: SUCCESS. The Pull Request #873 was integrated: "21330-CannotDeleteFileException-should-be-thrown-instead-of-FileException-" Pull request url: https://github.com/pharo-project/pharo/pull/873 Issue Url: https://pharo.fogbugz.com/f/cases/21330 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

Point>>#rotateBy:about: has a bad comment
by Julien Feb. 13, 2018

Feb. 13, 2018

Hello, Concerning this case I just opened: https://pharo.fogbugz.com/f/cases/21329/Point-rotateBy-about-has-a-bad-comm… <https://pharo.fogbugz.com/f/cases/21329/Point-rotateBy-about-has-a-bad-comm…> Iâd like your opinion about it. I just copy in this mail what I said on the case: rotateBy: angle about: center "Even though Point.theta is measured CW, this rotates with the more conventional CCW interpretateion of angle." [...] I think this comment does not explain what this method does. What should be something like: This method returns the point obtained after rotating myself around #center point of an #angle given as parameter. The #angle provided as parameter is interpreted as being in radian. Additionally, I think the name of this method is bad, maybe it should be renamed to something more explicit like #rotateBy:around: or #rotateByRadians:around:. Maybe, if this method is renamed as #rotateByRadians:around:, the method #rotateByDegrees:around: should be added as well. Cheers, Julien --- Julien Delplanque Doctorant Ã lâUniversitÃ© de Lille 1 http://juliendelplanque.be/phd.html Equipe Rmod, Inria BÃ¢timent B 40, avenue Halley 59650 Villeneuve d'Ascq NumÃ©ro de tÃ©lÃ©phone: +333 59 35 86 40

2 1

what is <localMetaclassMethod>
by Stephane Ducasse Feb. 13, 2018

Feb. 13, 2018

Hi what is this pragma in metaclass method? Stef

5 9

[Pharo 7.0-dev] Build #538: 21302 Remove unnecessary "self run:" in DictionaryTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 13, 2018

Feb. 13, 2018

There is a new Pharo build available! The status of the build #538 was: SUCCESS. The Pull Request #854 was integrated: "21302 Remove unnecessary "self run:" in DictionaryTest" Pull request url: https://github.com/pharo-project/pharo/pull/854 Issue Url: https://pharo.fogbugz.com/f/cases/21302 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 7.0-dev] Build #537: 21301 Remove unnecessary "self run:" in Base64MimeConverterTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #537 was: SUCCESS. The Pull Request #855 was integrated: "21301 Remove unnecessary "self run:" in Base64MimeConverterTest" Pull request url: https://github.com/pharo-project/pharo/pull/855 Issue Url: https://pharo.fogbugz.com/f/cases/21301 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 7.0-dev] Build #536: 21315 Remove unnecessary "self run:" in ChangeSetClassChangesTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #536 was: FAILURE. The Pull Request #841 was integrated: "21315 Remove unnecessary "self run:" in ChangeSetClassChangesTest" Pull request url: https://github.com/pharo-project/pharo/pull/841 Issue Url: https://pharo.fogbugz.com/f/cases/21315 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 7.0-dev] Build #535: 21299 Remove unnecessary "self run:" in ClassTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #535 was: FAILURE. The Pull Request #857 was integrated: "21299 Remove unnecessary "self run:" in ClassTest" Pull request url: https://github.com/pharo-project/pharo/pull/857 Issue Url: https://pharo.fogbugz.com/f/cases/21299 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 7.0-dev] Build #534: 21294 Remove unnecessary "self run:" in HeapTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #534 was: SUCCESS. The Pull Request #863 was integrated: "21294 Remove unnecessary "self run:" in HeapTest" Pull request url: https://github.com/pharo-project/pharo/pull/863 Issue Url: https://pharo.fogbugz.com/f/cases/21294 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0

[Pharo 7.0-dev] Build #533: 21310 Remove unnecessary "self run:" in WeakMessageSendTest
by ci-pharo-ci-jenkins2＠inria.fr Feb. 12, 2018

Feb. 12, 2018

There is a new Pharo build available! The status of the build #533 was: SUCCESS. The Pull Request #845 was integrated: "21310 Remove unnecessary "self run:" in WeakMessageSendTest" Pull request url: https://github.com/pharo-project/pharo/pull/845 Issue Url: https://pharo.fogbugz.com/f/cases/21310 Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%2…

1 0