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

Suggestion on naming: shorter names
by Sebastian Sastre Oct. 26, 2013

Oct. 26, 2013

Hi there, I've saw this: https://pharo.fogbugz.com/default.asp?W64 that you introduced: valueWithInterval: aDelay valueAfterWaiting: aDelay which is a cool feature. I only suggest that it could have shorter names. Using: valueAfter: aDelay and valueEvery: aDelay would make it look like: [Transcript cr; show: 'Hello World!' ] valueAfter: 10 seconds. [Transcript cr; show: 'tick-tack...' ] valueEvery: 10 seconds. In general, and when they don't introduce ambiguity or unacceptable vagueness, shorter names help typing, make the autocompleter find hits faster and unload the memory bandwidth of our brains Keep the great work! sebastian o/

3 2

[pharo-project/pharo-core]
by GitHub Oct. 26, 2013

Oct. 26, 2013

Branch: refs/tags/30528 Home: https://github.com/pharo-project/pharo-core

1 0

[pharo-project/pharo-core] e89b4a: 30528
by GitHub Oct. 26, 2013

Oct. 26, 2013

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: e89b4ac70d5cef761667317d58a4496d443d8a4e https://github.com/pharo-project/pharo-core/commit/e89b4ac70d5cef761667317d… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2013-10-26 (Sat, 26 Oct 2013) Changed paths: M Collections-Unordered.package/Dictionary.class/instance/testing/includesKey_.st A Gofer-Core.package/extension/ZnUrl/instance/mcRepositoryAsUser_withPassword_.st M SUnit-Core.package/TAssertable.class/instance/asserting/shouldnt_raise_.st M SUnit-Core.package/TAssertable.class/instance/asserting/shouldnt_raise_description_.st M SUnit-Core.package/TAssertable.class/instance/asserting/shouldnt_raise_whoseDescriptionDoesNotInclude_description_.st M SUnit-Core.package/TAssertable.class/instance/asserting/shouldnt_raise_whoseDescriptionIncludes_description_.st R SUnit-Core.package/TAssertable.class/instance/asserting/validateShouldntException_.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script183.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30528.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 30528 12019 Optimize Dictionary>>#includesKey: https://pharo.fogbugz.com/f/cases/12019 12018 Gofer failing due to ZnUrl missing mcRepositoryAsUser:withPassword: https://pharo.fogbugz.com/f/cases/12018 11989 shouldnt:raise: Error from issue #11876 is too offensive https://pharo.fogbugz.com/f/cases/11989 http://files.pharo.org/image/30/30528.zip

1 0

[pharo-project/pharo-core]
by GitHub Oct. 26, 2013

Oct. 26, 2013

Branch: refs/tags/30527 Home: https://github.com/pharo-project/pharo-core

1 0

[pharo-project/pharo-core] 0311e1: 30527
by GitHub Oct. 26, 2013

Oct. 26, 2013

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: 0311e14ecbcf52feb4a3aab0a99890ebcbe3b433 https://github.com/pharo-project/pharo-core/commit/0311e14ecbcf52feb4a3aab0… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2013-10-26 (Sat, 26 Oct 2013) Changed paths: R Kernel.package/MethodDictionary.class/instance/accessing/includesKey_.st M Network-Url.package/FileUrl.class/instance/private-initialization/privateInitializeFromText_.st M Network-Url.package/GenericUrl.class/class/parsing/absoluteFromText_.st M Network-Url.package/GenericUrl.class/instance/parsing/privateInitializeFromText_.st M Network-Url.package/HierarchicalUrl.class/instance/parsing/privateInitializeFromText_.st M Network-Url.package/Url.class/README.md R Network-Url.package/Url.class/class/instance creation/absoluteFromFileNameOrUrlString_.st A Network-Url.package/Url.class/class/instance creation/new.st M Network-Url.package/Url.class/instance/converting/asUrl.st R Network-Url.package/Url.class/instance/converting/asUrlRelativeTo_.st M Network-Url.package/Url.class/instance/parsing/newFromRelativeText_.st M Network-Url.package/extension/AbstractFileReference/instance/asUrl.st M Network-Url.package/extension/FileStream/instance/asUrl.st M Network-Url.package/extension/Path/instance/asUrl.st M Network-Url.package/extension/String/instance/asUrl.st R Network-Url.package/extension/String/instance/asUrlRelativeTo_.st R NetworkTests.package/FileUrlTest.class/instance/testing/testMatchingSchemesToSubclasses.st A NetworkTests.package/FileUrlTest.class/instance/testing/testNew.st R NetworkTests.package/GenericUrlTest.class/README.md R NetworkTests.package/GenericUrlTest.class/definition.st R NetworkTests.package/GenericUrlTest.class/instance/testing/testAsString.st M NetworkTests.package/HierarchicalUrlTest.class/instance/testing/testAsString.st A NetworkTests.package/HierarchicalUrlTest.class/instance/testing/testNew.st R NetworkTests.package/UrlTest.class/instance/testing/testFromFileNameOrUrlString.st R NetworkTests.package/UrlTest.class/instance/tests/testAbsoluteBrowser.st M NetworkTests.package/UrlTest.class/instance/tests/testAbsoluteFILE.st M NetworkTests.package/UrlTest.class/instance/tests/testAbsoluteFILE2.st M NetworkTests.package/UrlTest.class/instance/tests/testAbsoluteFILE3.st M NetworkTests.package/UrlTest.class/instance/tests/testAbsoluteHTTP.st M NetworkTests.package/UrlTest.class/instance/tests/testAbsolutePortErrorFix.st M NetworkTests.package/UrlTest.class/instance/tests/testAbsoluteTELNET.st M NetworkTests.package/UrlTest.class/instance/tests/testCombineWithRelative.st A NetworkTests.package/UrlTest.class/instance/tests/testNew.st R NetworkTests.package/UrlTest.class/instance/tests/testRelativeFILE.st M NetworkTests.package/UrlTest.class/instance/tests/testRelativeHTTP.st M NetworkTests.package/UrlTest.class/instance/tests/testUsernamePassword.st M NetworkTests.package/UrlTest.class/instance/tests/testUsernamePasswordEncoded2.st M NetworkTests.package/UrlTest.class/instance/tests/testUsernamePasswordPrintingEncoded.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script182.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30527.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M System-Support.package/SmalltalkImage.class/class/initialization/initializeShutDownList.st R Text-Core.package/Text.class/instance/converting/asUrlRelativeTo_.st M Tools.package/MethodFinder.class/instance/initialize/initialize2.st Log Message: ----------- 30527 7024 There are three URL implementation: Url, Uri and ZnUrl https://pharo.fogbugz.com/f/cases/7024 12017 remove #includesKey: override in MethodDictionary https://pharo.fogbugz.com/f/cases/12017 http://files.pharo.org/image/30/30527.zip

1 0

Use #shouldnt:raise: only with specific errors, or evaluate the given expression directly
by Alexandre Bergel Oct. 26, 2013

Oct. 26, 2013

Hi! Just to share some thought. I am now playing with Pharo 3. I have seen TestCase has some additional checks for assertion with exception. I perfectly understand the idea behind this, however, on the other hand, I see nothing suspect with the following assertion (which Roassal's tests are full of): self shouldnt: [ view raw drawOn: tracingCanvas ] raise: Error I have to shut down the method #validateShouldntException: Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

11 22

Pharo-V20.sources
by Johan Brichau Oct. 26, 2013

Oct. 26, 2013

Hi guys, Just a small note that the sources file for Pharo2.0 (and 3.0) is not easy to get when downloading the VM and the image from the website. I had to finally download the âone clickâ and extract the sources file from its resources. I believe they used to be included with the image zips. cheers Johan

7 8

Fwd: use of pointsTo: for includesSelector:
by Eliot Miranda Oct. 26, 2013

Oct. 26, 2013

---------- Forwarded message ---------- From: Eliot Miranda <eliot.miranda(a)gmail.com> Date: Fri, Oct 25, 2013 at 2:18 PM Subject: use of pointsTo: for includesSelector: To: The general-purpose Squeak developers list < squeak-dev(a)lists.squeakfoundation.org>, Pharo Development < pharo-project(a)lists.gforge.inria.fr> Hi All, I just got a little burned. I was searching through an Array of symbols and integers (it is the PrimitiveTable from the VM), wanting to filter-out methods with a with a certain pragma. The code looks like: anArray doWithIndex: [:entry :index| (self whichClassIncludesSelector: entry) ifNotNil: [:c| | m | m := c >> entry. (m pragmaAt: #option:) ifNotNil: [:pragma| (initializationOptions at: (pragma arguments first) ifAbsent: [true]) ifFalse: [anArray at: index put: 0]]]] the error was a keyNotFound error for c >> entry. Turns out entry was the integer 306, a code for a quick primitive that returns some inst var. The question is why did (self whichClassIncludesSelector: entry) evaluate to other than nil given that 306 is /not/ a selector in any of the classes from self on up. Well, it's MethodDictionary's use of pointsTo: that is at fault: MethodDictionary>>includesKey: aSymbol "This override assumes that pointsTo is a fast primitive" aSymbol ifNil: [^ false]. ^ self pointsTo: aSymbol ProtoObject>>pointsTo: anObject "This method returns true if self contains a pointer to anObject, and returns false otherwise" <primitive: 132> 1 to: self class instSize do: [:i | (self instVarAt: i) == anObject ifTrue: [^ true]]. 1 to: self basicSize do: [:i | (self basicAt: i) == anObject ifTrue: [^ true]]. ^ false Turns out that 306 was the tally of one of the method dictionaries along self's superclass chain. This seems to be to be completely bogus. Do we really need crude performance hacks like this any more? -- best, Eliot -- best, Eliot

2 1

AspectS, PHANtom - Aspects in Pharo
by phil＠highoctane.be Oct. 25, 2013

Oct. 25, 2013

Hello, I was wondering what the state of Aspect programming was in Pharo. Googling a bit turned the title mentioned projects. Do we anything loadable and more or less stable for Pharo2.x? I'd like to use some of that. TIA Phil

2 2

[pharo-project/pharo-core]
by GitHub Oct. 25, 2013

Oct. 25, 2013

Branch: refs/tags/30526 Home: https://github.com/pharo-project/pharo-core

1 0