December 2014 - Pharo-dev - lists.pharo.org

[pharo-project/pharo-core] e3c61d: 40394
by GitHub Dec. 2, 2014

Dec. 2, 2014

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: e3c61d5e96987e907d531dafe966378d863112aa https://github.com/pharo-project/pharo-core/commit/e3c61d5e96987e907d531daf… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-12-02 (Tue, 02 Dec 2014) Changed paths: M AST-Core.package/ManifestASTCore.class/definition.st M Fuel.package/ManifestFuel.class/definition.st M Manifest-Core.package/ManifestManifestCore.class/definition.st A Manifest-Core.package/PackageManifest.class/README.md A Manifest-Core.package/PackageManifest.class/class/testing/isManifest.st A Manifest-Core.package/PackageManifest.class/definition.st A Manifest-Core.package/PackageManifest.class/instance/code-critics/rejectClasses.st A Manifest-Core.package/PackageManifest.class/instance/code-critics/rejectRules.st A Manifest-Core.package/PackageManifest.class/instance/comparing/=.st A Manifest-Core.package/PackageManifest.class/instance/comparing/hash.st M Manifest-Core.package/SmalllintManifestChecker.class/instance/manifest/isFalsePositive_forRuleId_versionId_.st M Manifest-Core.package/TheManifestBuilder.class/class/instance creation/of_.st M Manifest-Core.package/TheManifestBuilder.class/class/utils/allManifestClasses.st M Manifest-Core.package/TheManifestBuilder.class/class/utils/manifestClassComment.st M Manifest-Core.package/TheManifestBuilder.class/definition.st M Manifest-Core.package/TheManifestBuilder.class/instance/accessing/manifest.st M Manifest-Core.package/TheManifestBuilder.class/instance/adding%2Fremoving/addRejectClass_.st M Manifest-Core.package/TheManifestBuilder.class/instance/adding%2Fremoving/addRejectRule_.st M Manifest-Core.package/TheManifestBuilder.class/instance/adding%2Fremoving/cleanUp.st M Manifest-Core.package/TheManifestBuilder.class/instance/adding%2Fremoving/removeRejectClass_.st M Manifest-Core.package/TheManifestBuilder.class/instance/adding%2Fremoving/removeRejectRule_.st M Manifest-Core.package/TheManifestBuilder.class/instance/comparing/=.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/createManifestNamed_.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/falsePositiveOf_version_.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/hasFalsePositiveOf_version_.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/hasToDoOf_version_.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/hasTruePositiveOf_version_.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/rejectClasses.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/rejectRules.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/toDoOf_version_.st M Manifest-Core.package/TheManifestBuilder.class/instance/manifest/truePositiveOf_version_.st M Manifest-Core.package/TheManifestBuilder.class/instance/private/addAllItem_selector_.st M Manifest-Core.package/TheManifestBuilder.class/instance/private/compileSelector_returnValue_.st M Manifest-Core.package/TheManifestBuilder.class/instance/private/dateOf_forSelector_.st M Manifest-Core.package/TheManifestBuilder.class/instance/private/isObsoleteManifest.st M Manifest-Core.package/TheManifestBuilder.class/instance/private/removeAllItem_selector_.st M Manifest-Core.package/TheManifestBuilder.class/instance/private/removeObsoleteFalsePositiveOf_.st A Manifest-Core.package/extension/RPackage/instance/packageManifest.st M Manifest-Core.package/extension/TBehavior/instance/isManifest.st M Manifest-CriticBrowser.package/ManifestManifestCriticBrowser.class/definition.st R Manifest-Tests.package/ManifestManifestTests.class/class/meta data/rejectClasses.st R Manifest-Tests.package/ManifestManifestTests.class/class/meta data/rejectRules.st M Manifest-Tests.package/ManifestManifestTests.class/definition.st M NautilusRefactoring.package/NautilusRefactoring.class/instance/private-class/privateGenerateSubclassFor_.st M NautilusRefactoring.package/NautilusRefactoring.class/instance/private-class/privateGenerateSuperClassFor_.st M NautilusRefactoring.package/NautilusRefactoring.class/instance/private-class/privateInsertSubclassFor_.st M NautilusRefactoring.package/NautilusRefactoring.class/instance/private-class/privateInsertSuperclassFor_.st M OpalCompiler-Core.package/ManifestOpalCompilerCore.class/definition.st A Refactoring-Core.package/RBRemoveMethodRefactoring.class/instance/preconditions/checkBrowseAllOccurences_.st A Refactoring-Core.package/RBRemoveMethodRefactoring.class/instance/preconditions/checkBrowseOccurrenceOf_in_.st A Refactoring-Core.package/RBRemoveMethodRefactoring.class/instance/preconditions/checkBrowseOccurrences_.st M Refactoring-Core.package/RBRemoveMethodRefactoring.class/instance/preconditions/checkReferencesToAnyOf_.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script394.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40394.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M StartupPreferences.package/StartupPreferencesLoader.class/instance/accessing/add_.st M UserManager-Tests.package/ManifestKeyChainTest.class/definition.st Log Message: ----------- 40394 14556 Error: Instances of UndefinedObject are not indexable https://pharo.fogbugz.com/f/cases/14556 14505 Better integration of Package Manifest https://pharo.fogbugz.com/f/cases/14505 10719 Problem removing methods with more than one implementor using suggestions https://pharo.fogbugz.com/f/cases/10719 12594 #runOnce: of StartupAction not working https://pharo.fogbugz.com/f/cases/12594 http://files.pharo.org/image/40/40394.zip

1 0

ubuntu vm build
by Norbert Hartl Dec. 2, 2014

Dec. 2, 2014

Hi, I forgot who is maintaining the ubuntu launchpad ppa vm build. But whoever you are ( :) ) would it be possible to include a vm build for the actual ubuntu release utopic unicorn? thanks, Norbert

2 3

[To be merged] GT-InspectorExtensions-Core-SvenVanCaekenberghe.77
by Sven Van Caekenberghe Dec. 1, 2014

Dec. 1, 2014

In Moose/GToolkit: === Name: GT-InspectorExtensions-Core-SvenVanCaekenberghe.77 Author: SvenVanCaekenberghe Time: 1 December 2014, 9:56:58.104994 pm UUID: 3da8544f-909e-49c6-88a5-f944a5f13b42 Ancestors: GT-InspectorExtensions-Core-TudorGirba.76 New version of Float>>#gtInspectorFloatIn: that shows the new binary presentation available in Pharo 4. Tests on availability of #binaryLiteralString so that this change should remain compatible with Pharo 3. Thx Kris Gybels for this contribution. === To be merged with higher versions (this was based on the version present in Pharo 4 today). Sven

2 3

[pharo-project/pharo-core] 6f6c06: 40393
by GitHub Dec. 1, 2014

Dec. 1, 2014

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: 6f6c06023fa6406371a1e66a33159f03ea6427a2 https://github.com/pharo-project/pharo-core/commit/6f6c06023fa6406371a1e66a… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-12-01 (Mon, 01 Dec 2014) Changed paths: M EmbeddedFreeType.package/EmbeddedFreeTypeFontFontDescription.class/class/accessing/fontContents.st M EmbeddedFreeType.package/EmbeddedFreeTypeFontFontDescription.class/class/accessing/installAllFontsIn_.st M EmbeddedFreeType.package/EmbeddedFreeTypeFontFontDescription.class/class/accessing/installFontsIn_.st M EmbeddedFreeType.package/EmbeddedFreeTypeFontFontDescription.class/class/accessing/originalFileName.st M EmbeddedFreeType.package/EmbeddedFreeTypeFontInstaller.class/class/accessing/current.st M EmbeddedFreeType.package/EmbeddedFreeTypeFontInstaller.class/class/extra fonts registration/registerFont_.st R EmbeddedFreeType.package/OpenSansRegular.class/README.md R EmbeddedFreeType.package/OpenSansRegular.class/class/accessing/fontContents.st R EmbeddedFreeType.package/OpenSansRegular.class/class/accessing/originalFileName.st R EmbeddedFreeType.package/OpenSansRegular.class/class/class initialization/initialize.st R EmbeddedFreeType.package/OpenSansRegular.class/definition.st M EmbeddedFreeType.package/SourceCodeFonts.class/class/accessing/defaultFontName.st A EmbeddedFreeType.package/SourceCodeFonts.class/class/accessing/setSourceCodeFontsDefault.st M EmbeddedFreeType.package/SourceCodeFonts.class/class/font registration/registerFonts_.st A EmbeddedFreeType.package/SourceSansProRegular.class/README.md A EmbeddedFreeType.package/SourceSansProRegular.class/class/accessing/fontContents.st A EmbeddedFreeType.package/SourceSansProRegular.class/class/accessing/originalFileName.st A EmbeddedFreeType.package/SourceSansProRegular.class/class/class initialization/initialize.st A EmbeddedFreeType.package/SourceSansProRegular.class/definition.st M FreeType.package/FreeTypeFontProvider.class/instance/loading and updating/updateEmbeddedFreeTypeFonts.st M FreeType.package/FreeTypeFontProvider.class/instance/loading and updating/updateFromSystem.st M FreeType.package/FreeTypeSettings.class/class/startup/startUp_.st A FreeType.package/FreeTypeSettings.class/class/startup/updateFreeType.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script393.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40393.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 40393 14553 FreeType default fonts not correctly loaded when image initializes https://pharo.fogbugz.com/f/cases/14553 http://files.pharo.org/image/40/40393.zip

2 1

[pharo-project/pharo-core]
by GitHub Dec. 1, 2014

Dec. 1, 2014

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

1 0

14353 Delay refactoring (part 2) - change from milliseconds to microseconds
by Ben Coman Dec. 1, 2014

Dec. 1, 2014

The change of Delay from milliseconds to microseconds is ready for human review. (https://pharo.fogbugz.com/default.asp?14353) With this change from milliseconds to microseconds, clock rollover is eliminated, but it seems Delays run at reduced performance. newCodeEnabled FALSE TRUE 128507 101227 delays scheduled per second 100% 79% relative performance 7.8 9.9 microseconds per scheduled delay 100% 127% relative performance I "guess" this is due to the move from 32bit millisecond immediate SmallIntegers to 64bit microsecond boxed LargePositiveIntegers. It would be interesting to check this on Spur64, but for now, is this performance penalty a reasonable trade for eliminating delay clock rollover? The Delay class comment warns of the critical performance its high priority event loop, and on paper a 21% drop in performance seems significant, but actually the main Morph interCyclePause is 50 milliseconds. So... (9.9 - 7.8) / 50000 = 0.004% difference in performance. (Did I get that right?) Comments? P.S. DelayBenchmark results... =============== DelayBenchmark withNewCodeEnabled: true... #ConcurrentDelays: 1000 . Count: 39018 per second #ConcurrentDelays: 2000 . Count: 71906 per second #ConcurrentDelays: 3000 . Count: 98010 per second #ConcurrentDelays: 4000 . Count: 101227 per second #ConcurrentDelays: 5000 . Count: 64728 per second #ConcurrentDelays: 6000 . Count: 90495 per second #ConcurrentDelays: 7000 . Count: 89220 per second #ConcurrentDelays: 8000 . Count: 89585 per second #ConcurrentDelays: 9000 . Count: 88393 per second #ConcurrentDelays: 10000 . Count: 89073 per second DelayBenchmark withNewCodeEnabled: false... #ConcurrentDelays: 1000 . Count: 39498 per second #ConcurrentDelays: 2000 . Count: 77808 per second #ConcurrentDelays: 3000 . Count: 99889 per second #ConcurrentDelays: 4000 . Count: 117632 per second #ConcurrentDelays: 5000 . Count: 128507 per second #ConcurrentDelays: 6000 . Count: 118729 per second #ConcurrentDelays: 7000 . Count: 114437 per second #ConcurrentDelays: 8000 . Count: 114792 per second #ConcurrentDelays: 9000 . Count: 115137 per second #ConcurrentDelays: 10000 . Count: 113973 per second

1 0

Re: [Pharo-dev] [squeak-dev] IdentitySet>>collect:
by Ben Coman Dec. 1, 2014

Dec. 1, 2014

Levente Uzonyi wrote: > The problem with changing #species is that it's overused. There are at > least three different things it's used for in the Collection hierarchy: > - the class of the object returned by #collect: (Set's subclasses are > the exception since 2000 or so, except for WeakSet, but that's really > strange, and should revisited) > - the class of the object returned by #select: and friends > - the class used for comparison of collections So should there be #collectSpecies ? Which defaults to "^self species" and is overidden as required. In the case of Sets, you start with an "unordered collection without duplicates", but since #collect can introduce duplicates, you end up with "an unordered collection with duplicates" --> Bag (per Dave's suggestion) So I think its reasonable that the following two snippets should produce the same result. oc := OrderedCollection newFrom: #( 1 2 3 4 5 ). (oc collect: [ :e | e even ]) occurrencesOf: true. "--> 2" s := Set newFrom: #( 1 2 3 4 5 ). (s collect: [ :e | e even ]) occurrencesOf: true. "--> 2" which you get with the following change... Set>>collectSpecies ^Bag Set>>collect: aBlock | result | result := self collectSpecies new: self size. array do: [:each | each ifNotNil: [result add: (aBlock value: each enclosedSetElement)]]. ^ result > > HashedCollection >> #select: uses #species. Changing IdentitySet >>> #species to return a Set would break #select:, because a ~~ b doesn't > imply a ~= b. IdentitySet is the optimal class for the return value of > IdentitySet >> #select:, because it's guaranteed to be able to hold all > selected values. I think the same applies to all other kind of sets (and > collections, because #select: is simply reducing the number of elements). > > This is not the first time this discussion comes up. There was one in > 2003[1], but the thread is hard to follow. And there was one in 2009[2]. > I think the conclusion was that it's better to leave the method as it > is, because there's no better way to do it. > Sure, we could use #species in #collect:, but it would break quite a lot > of stuff. For example: > > | k | > k := KeyedSet keyBlock: [ :each | each first ]. > k add: #[1]; add: #[2]. > k collect: [ :each | each size ] That works with the above change to produce --> a Bag(1 1) > > Currently this snippet returns "a Set(1)". Using #species in #collect: > (via #copyEmpty) one would get an error, because SmallInteger does not > understand #first. Changing the #species of KeyedSet would break #select:. > > So IMHO when you #collect: from a set, you should always use > #collect:as:. Here is a (partial) list of methods which send #collect: > to a Set: > > Categorizer >> #changeFromCategorySpecs: > ClassFactoryForTestCase >> #createdClassNames > Dictionary >> #unreferencedKeys > MCDependencySorterTest >> > #assertItems:orderAs:withRequired:toLoad:extraProvisions: > MessageNode >> #analyseTempsWithin:rootNode:assignmentPools: > MethodNode >> #referencedValuesWithinBlockExtent: > SetTest >> #testCollect > SetWithNilTest >> #runSetWithNilTestOf: > > There are probably many more, but we should fix all of them. A static > analyzer could help a lot. > > Levente > > [1] > http://lists.squeakfoundation.org/pipermail/squeak-dev/2003-February/052659… > > [2] > http://lists.squeakfoundation.org/pipermail/squeak-dev/2009-November/141016… > > > > > On Wed, 26 Nov 2014, David T. Lewis wrote: > >> It seems to me that Object>>species is intended to handle this sort of >> issue. >> >> For IdentitySet, it answers what Eliot was expecting: >> >> IdentitySet new species ==> IdentitySet >> Set new species ==> Set >> >> However, IdentitySet>>collect: does not make use of this, and it answers >> a Set for the reasons that Levente explained. >> >> Answering an Array or and OrderedCollection would not really make sense, >> because sets are unordered collections (but Bag might be better). Bag is better. >> >> Shouldn't we have an implementation of IdentitySet>>species that answers >> Set (or Bag), with a method comment explaining why this is the case, and >> with all of the collection methods using #species to answer the right >> kind of result? >> >> I note that IdentitySet>>collect: answers a Set, but IdentitySet>select: >> sends #species and therefore answers an IdentitySet. >> >> So I think that if we want the #species of an IdentitySet to be a Set, >> then we should make it so, and give it a method comment to explain the >> rationale. And the #collect: and #select: methods should both answer a >> result of that #species. I think the species of #collect: and #select: are intrinsically different. #collect: is a transformation. >> >> Dave >> >> >> On Thu, Nov 27, 2014 at 01:14:30AM +0100, Levente Uzonyi wrote: >>> Your example hides the problem of ordering - what Tobias is asking >>> about - >>> so here's another: >>> >>> (IdentitySet withAll: #(1 1.0)) collect: [ :each | each class ] >>> >>> If IdentitySet >> #collect: were returning an Array, then what would >>> be the >>> answer? >>> >>> { SmallInteger. Float } or { Float. SmallInteger } ? --> a Bag(SmallInteger Float) >>> >>> If you really want to have the resulting collection have the same size, >>> but avoid the problem with ordering, then what you really need is a Bag. To me that makes the most sense. >>> >>> On Thu, 27 Nov 2014, Frank Lesser wrote: >>> >>>> Hi Tobias, >>>> agree, a problem of "OrderedCollection" >>>> not to break a lot of other things we could return an Array. >>>> but for me collecting has priority. >>>> Frank >>>> >>>> -----Urspr?ngliche Nachricht----- >>>> Von: squeak-dev-bounces(a)lists.squeakfoundation.org >>>> [mailto:squeak-dev-bounces@lists.squeakfoundation.org] Im Auftrag von >>>> Tobias >>>> Pape >>>> Gesendet: Donnerstag, 27. November 2014 00:48 >>>> An: The general-purpose Squeak developers list >>>> Betreff: Re: [squeak-dev] IdentitySet>>collect: >>>> >>>> >>>> On 27.11.2014, at 00:34, Frank Lesser >>>> <frank-lesser(a)lesser-software.com> >>>> wrote: >>>> >>>>> hmm, not convinced >>>>> >>>>> (IdentitySet withAll: #(1 1.0)) collect: [:e| e asInteger ] >>>>> OrderedCollection(1 1 ) >>>>> >>>>> in LSWVST ( one-to-one ), you collect results of evaluating a block on >>>>> objects. >>>>> >>>>> Frank >>>>> maybe I am wrong ... >>>> >>>> Where would the order come from for that _Ordered_Collection? An unordered OrderedCollection --> Bag cheers -ben

2 1

[pharo-project/pharo-core] 8be654: 40392
by GitHub Dec. 1, 2014

Dec. 1, 2014

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: 8be654fa95e7a5c6d96607b7513e3dde8a1ffb74 https://github.com/pharo-project/pharo-core/commit/8be654fa95e7a5c6d96607b7… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-12-01 (Mon, 01 Dec 2014) Changed paths: A Kernel.package/Float.class/class/instance creation/fromIEEE64Bit_.st A Kernel.package/Float.class/instance/printing/binaryLiteralString.st A Kernel.package/Float.class/instance/printing/printBinaryLiteralOn_.st A KernelTests.package/FloatTest.class/instance/tests - printing/testBinaryLiteralString.st A Manifest-Core.package/extension/RBExplicitRequirementMethodsRule/instance/category.st A Nautilus.package/CodeSearchingAcceptor.class/README.md A Nautilus.package/CodeSearchingAcceptor.class/definition.st A Nautilus.package/CodeSearchingAcceptor.class/instance/protocol/accept_notifying_.st A NautilusRefactoring.package/CodeSearchingRule.class/README.md A NautilusRefactoring.package/CodeSearchingRule.class/definition.st A NautilusRefactoring.package/CodeSearchingRule.class/instance/accessing/matcher_.st A NautilusRefactoring.package/CodeSearchingRule.class/instance/accessing/name.st M NautilusRefactoring.package/NautilusRefactoring.class/instance/rewrite code/searchCode.st A NautilusRefactoring.package/extension/NautilusUI/instance/searchCode_.st A Refactoring-Critics.package/RBExplicitRequirementMethodsRule.class/README.md A Refactoring-Critics.package/RBExplicitRequirementMethodsRule.class/definition.st A Refactoring-Critics.package/RBExplicitRequirementMethodsRule.class/instance/accessing/name.st A Refactoring-Critics.package/RBExplicitRequirementMethodsRule.class/instance/accessing/rationale.st A Refactoring-Critics.package/RBExplicitRequirementMethodsRule.class/instance/running/checkClass_.st M SUnit-Core.package/TestCase.class/instance/printing/printOn_.st A SUnit-Core.package/TestCase.class/instance/printing/printTestSelectorDefiningClass_on_.st A SUnit-Core.package/TestCase.class/instance/printing/printTestSelectorOn_.st M SUnit-Tests.package/SUnitTest.class/instance/testing/testFileOutResult.st M SUnit-UI.package/TestRunner.class/instance/browsing/browseSelectedTest_.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script392.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40392.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Spec-Inspector.package/EyeFloatInspector.class/instance/list/addSpecialFields_.st Log Message: ----------- 40392 14550 testTraitExplicitRequirementMethodsMustBeImplementedInTheClassOrInASuperclass failing 200 times https://pharo.fogbugz.com/f/cases/14550 14288 Show binary literal for Floats in inspectors https://pharo.fogbugz.com/f/cases/14288 14551 Pharo 4.0: Fix "Search Code" refactoring menu item in Nautilus https://pharo.fogbugz.com/f/cases/14551 14546 Failed inherited tests should show home class (in TestRunner, like the debugger) https://pharo.fogbugz.com/f/cases/14546 http://files.pharo.org/image/40/40392.zip

2 1

Woden processor occupation and text highlight
by Yuriy Tymchuk Dec. 1, 2014

Dec. 1, 2014

Hi, first of all let me know where I can write questions about Woden (because I have not seen a lot of activity around it in any chat). Woden and Roassal3D have a problem that when the 3D window is shown, text highlighting is not working. Usually itâs not a big deal, but Rubric writes white text on white background :). I think a problem like that was already in Roassal2 and maybe someone remembers how to solve that? Uko

4 4

Woden-GT integration
by Yuriy Tymchuk Dec. 1, 2014

Dec. 1, 2014

Hi everyone, I think that during ESUG Woden was integrated into Toolkit in a way, what you can see a 3D visualization as one of the representations of the view. Is there any additional package that provides this functionality? Uko

4 3