April 2016 - Pharo-dev - lists.pharo.org

[pharo-project/pharo-core] 0ae434: 50716

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 0ae434c0f966d46075dd8cc81ce1ebac483fa5ea https://github.com/pharo-project/pharo-core/commit/0ae434c0f966d46075dd8cc8… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-04-26 (Tue, 26 Apr 2016) Changed paths: M BlueInk-Core.package/BIConfigurableFormatter.class/instance/visiting/visitCascadeNode_.st M ConfigurationOfCatalog.package/ConfigurationOfCatalog.class/instance/symbolic versions/stable_.st A ConfigurationOfCatalog.package/ConfigurationOfCatalog.class/instance/versions/version013_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50715.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50716.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50715.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50716.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Tool-Catalog.package/CatalogSettings.class/class/settings/catalogSettingsOn_.st Log Message: ----------- 50716 18087 Update Catalog 0.13 https://pharo.fogbugz.com/f/cases/18087 18096 haltonce in formatter https://pharo.fogbugz.com/f/cases/18096 http://files.pharo.org/image/50/50716.zip

April 26, 2016

Re: [Pharo-dev] Class allSubInstances size takes 23 seconds to run on Pharo 5 on OS X

by Guille Polito

Hi, Steph, the image logic also has changed in Pharo. There should not be any difference (at least from the list that Eliot gave). Guille -------- Original Message -------- > Wow, thanks for the detailed explanation! > > Cheers, > Bernhard > > >> Am 21.04.2016 um 16:48 schrieb Eliot Miranda <eliot.miranda(a)gmail.com>: >> >> Hi Bernhard, >> >>> On Apr 20, 2016, at 9:56 PM, Bernhard Pieber <bernhard(a)pieber.com> wrote: >>> >>> Hi Eliot, >>> >>> Right, but in the latest Squeak5.1-trunk with the latest CogSpur.r3663.app it takes only 3 milliseconds. I guess itâs just an optimized implementation, there. (Canât look right now.) >> Indeed ;-). In most Smalltalk implementations an object's header contains a direct reference (a pointer) to its class, but in Spur, an object's header contains a "class index" (as does the Dart VM). IIRC this is an approach first tried in a system at Xerox PARC, but what follows, I invented for 64-bit VW, to avoid having 64-bits of class reference in every object. >> >> The object header contains both a 22-bit classIndex, for up to 4 million classes, and a 22-bit identityHash field for 4 million hash values. The classIndex is used in all method caches, which has advantages for the GC, and means that when sending a message via an inline cache, the object's index is fetched, not its class. >> >> Classes are stored in a hidden two-level sparse array (array of arrays), the class table. If the object's class object is required (for the #class message or to do a full message lookup searching the inheritance hierarchy) the classIndex is used to index the class table. >> >> The classes known to the VM, Array, Message, Context, LargeNegativeInteger, LargePositiveInteger have known indices and occupy the first page of the class table. So the VM can allocate instances of these by simply writing a header containing the relevant classIndex. A conventional implementation would have to fetch the class from the specialObjectsArray and write it into the header (and in pre-Spur, there are 31 "compact class indices" of which a handful are used for a subset of the classes). >> >> But how does the VM find the classIndex of an arbitrary class? Searching the table would be very slow. So Spur arranges that a class's identityHash is also its index in the classTable. Classes have their own special identityHash primitive, see Behavior>>#identityHash. Whenever the VM tries to Instantiate a class, or whenever the class's identityHash primitive is run, if the class's identityHash is zero, the VM assigns an unused index in the class table as the identityHash and stores the class in the table. So a class's identityHash is its index in the class table. >> >> To Instantiate a class its identityHash is copied into the classIndex field of the new instance's header. To search for all instances of a class, the VM searches for objects whose classIndex is that class's identityHash. A consequence is that if a class doesn't yet have an identityHash (identityHash = 0) it can't have instances, and so there is no need to scan the heap. >> >> >> Note that you can't (easily) tell from the image. If you send identityHash to a class that will enter it into the table. There's a hidden hasIdentityHash primitive that can be used, and if you're curious I can post code to get at it. Note that this implies all the metaclasses are in the table because they have as their single instance a normal named class, but Metaclasses holds onto its instance via thisClass and so can also avoid traversing the heap. >> >> Some things to try: >> >> {Array. ByteString. Character. Context. LargeNegativeInteger. LargePositiveInteger. Message. SmallFloat64. BoxedFloat64. SmallInteger} collect: #identityHash >> >> and in a pre-Spur system >> >> Smalltalk compactClassesArray select: [:c| Smalltalk specialObjectsArray includes: c] thenCollect: [:c| {c. c indexIfCompact] >> >> Hope this explains what's going on under the covers. >> >>> Cheers, >>> Bernhard >>> >>>> Am 20.04.2016 um 17:43 schrieb Eliot Miranda <eliot.miranda(a)gmail.com>: >>>> >>>> Hi Bernhard, >>>> >>>> >>>>> On Apr 20, 2016, at 5:26 AM, Bernhard Pieber <bernhard(a)pieber.com> wrote: >>>>> >>>>> Dear Pharoers, >>>>> >>>>> I found something strange: >>>>> Time millisecondsToRun: [ Class allSubInstances size ]. â23617" >>>> Because it does an allInstances for Class and all its subclasses, and allInstances visits every object in the heap, so this does thousands of scans of the heap. >>>> >>>> >>>>> I did this on a new Pharo 5 image >>>>> curl get.pharo.org/alpha+vmLatest | bash >>>>> >>>>> Can somebody confirm this on their machine? What might be the reason? >>>>> >>>>> Cheers, >>>>> Bernhard >>> >

April 26, 2016

[pharo-project/pharo-core]

by GitHub

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

April 26, 2016

[pharo-project/pharo-core] b1b738: 50715

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: b1b738b40653feebd993cef3f84c9f4556aca695 https://github.com/pharo-project/pharo-core/commit/b1b738b40653feebd993cef3… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-04-26 (Tue, 26 Apr 2016) Changed paths: M BlueInk-Extras.package/BIInspector.class/class/settings/settingsOn_.st M ConfigurationOfRubric.package/ConfigurationOfRubric.class/instance/symbolic versions/stable_.st A ConfigurationOfRubric.package/ConfigurationOfRubric.class/instance/versions/version2193_.st M ConfigurationOfVersionner.package/ConfigurationOfVersionner.class/instance/symbolic versions/stable_.st A ConfigurationOfVersionner.package/ConfigurationOfVersionner.class/instance/versions/version21312_.st M DebuggerModel.package/DebugSystemSettings.class/class/settings/debugSettingsOn_.st M FreeType.package/FreeTypeSystemSettings.class/class/settings/freeTypeSettingsOn_.st M Monticello.package/MCFileBasedRepository.class/class/settings/monticelloSettingsOn_.st M Morphic-Base.package/ShortcutReminder.class/class/settings/settingsOn_.st M Morphic-Base.package/SystemProgressMorph.class/class/settings/settingOn_.st R NodeNavigation.package/NNavNavigation.class/class/configuration/enableAstHighlightSetting_.st M OpalCompiler-Core.package/AbstractCompiler.class/class/settings/compilerSettingsOn_.st M Rubric.package/RubAbstractTextArea.class/class/settings/rubricSettingsOn_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50714.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50715.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50714.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50715.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st R Settings-Polymorph.package/PolymorphSystemSettings.class/class/morphic/morphicAnimationSettingsOn_.st M Settings-Polymorph.package/PolymorphSystemSettings.class/class/morphic/morphicMenuSettingsOn_.st M Settings-Polymorph.package/PolymorphSystemSettings.class/class/morphic/morphicSettingsOn_.st M Settings-Polymorph.package/PolymorphSystemSettings.class/class/morphic/morphicWindowsSettingsOn_.st M Settings-System.package/SystemSystemSettings.class/class/settings/systemSettingOn_.st M Text-Diff.package/TextDiffBuilder.class/class/settings/differatorSettingsOn_.st M Text-Edition.package/Editor.class/class/settings/editingSettingsOn_.st M Tool-Finder.package/DialogItemsChooserUI.class/class/settings/dialogItemsChooserSettingsOn_.st M UIManager.package/CommandLineUIManager.class/class/settings/uiSettingsOn_.st M Versionner-Spec-Browser.package/VersionnerSpecBrowser.class/class/settings/settingsOn_.st A WorldMenuHelp.package/WorldMenuHelp.class/README.md A WorldMenuHelp.package/WorldMenuHelp.class/class/as yet unclassified/bookName.st A WorldMenuHelp.package/WorldMenuHelp.class/class/as yet unclassified/overview.st A WorldMenuHelp.package/WorldMenuHelp.class/class/as yet unclassified/pages.st A WorldMenuHelp.package/WorldMenuHelp.class/definition.st Log Message: ----------- 50715 18080 New Help about adding World Menu Items https://pharo.fogbugz.com/f/cases/18080 18091 Comment some existing settings https://pharo.fogbugz.com/f/cases/18091 18089 Update Rubric 2.19.3 https://pharo.fogbugz.com/f/cases/18089 18090 Update Versionner 2.13.12 https://pharo.fogbugz.com/f/cases/18090 http://files.pharo.org/image/50/50715.zip

April 26, 2016

Re: [Pharo-dev] [Pharo 5] Some feedbacks

by Stephan Eggermont

On 25-04-16 12:16, Hilaire wrote: > While giving a try to Pharo5, I wrote some notes about how it fells. > It may be useful to share it. Of course it is also very opinionated and > may be rude. But no offense to consider, really. Thanks for the detailed feedback. Stephan

April 26, 2016

[pharo-project/pharo-core]

by GitHub

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

April 26, 2016

[pharo-project/pharo-core] 690d4c: 50714

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 690d4c587a8380a4489d030e3ed9f1796672b1f8 https://github.com/pharo-project/pharo-core/commit/690d4c587a8380a4489d030e… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-04-26 (Tue, 26 Apr 2016) Changed paths: M BlueInk-Core.package/BIPrettyPrinterContext.class/instance/initialization/initialize.st M ContributingToTheCatalogHelp.package/ContributingToTheCatalogHelp.class/class/accessing/bookName.st M HowToContributeHelp.package/HowToContributeHelp.class/class/accessing/bookName.st M OpalCompiler-Core.package/OCASTSemanticAnalyzer.class/instance/error handling/storeIntoReadOnlyVariable_.st M OpalCompiler-Core.package/OCASTSemanticAnalyzer.class/instance/error handling/storeIntoSpecialVariable_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50713.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50714.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50713.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50714.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A Shout.package/SHRBTextStyler.class/instance/visiting/visitAssignmentNode_.st Log Message: ----------- 50714 18070 Syntax error: Cannot store into https://pharo.fogbugz.com/f/cases/18070 18082 Shorter help items https://pharo.fogbugz.com/f/cases/18082 18065 BIPrettyPrinterContext undeclared var in #initialize https://pharo.fogbugz.com/f/cases/18065 http://files.pharo.org/image/50/50714.zip

April 26, 2016

Please add me to the smalltalkhub Pharo group

by Alistair Grant

Hi, I have a proposed enhancement to the CatalogBrowser that I would like to submit via fogbugz. I'm following Stephan Eggermont's video which states: "Ask to be added to the smalltalkhub Pharo group on the mailing list" Is this the right place to make the request (and please add me)? Thanks, Alistair

April 25, 2016

Re: [Pharo-dev] [Pharo 5] Some feedbacks

by Peter UhnÃ¡k

> > I think it is more a question of usability based on the average wished > font size. So all in all the choice of font size can be done objectively > although font size wish is subjective. > Yes, that's why it is configurable. > > Some people will want 9, other 14 or 16. The average is likely in between. > It was 9 since at least Pharo 3, so I am not sure what's the complaint about, considering people were ok with the size for several years (so I am inferring 9/10 should be good for everyone). > > > > > >> Line numbers: where is it used, not in the system browser > > > > This is disabled by default, so it shouldn't be a problem. > > UI noise, but not a big deal. > Considering the L+ and W are literally the smallest elements in the UI I'd hardly call it noise; but this is subjective. > > >> Nautilis: where is the local senders tools? > > > > You need to scope your view (top left corner button "Scoped") > > Oh great to know what does "Scoped". > The tooltips of the "Scoped" button is therefore not helping as there is > no useful information: "Browse scoped the selected packages" > One can check at Dr. Geo what useful tips mean: Most of its tools come > with 3 or even 4 lines tips (see screenshot example) > > I guess professional software company have people to review it all. > A company that cares about first-time users. Once you know what it does, the current tooltip is often completely obvious. And since we don't really have much first-time users that would be willing to share (as often they are overwhelmed by so much different stuff) it can be quite difficult to improve. So if you can propose better tooltips (since you are a pair of fresh eyes), then it would be valuable for new users. > > > >> Spotter > > Why Ctrl+[->]? my intuition tells me [->] as indicated by the blue icon > > arrow, please fix it! > > > > Because -> and <- is already taken. Pretty much any shortcut in the > > system has ctrl/alt/command in it, so I don't see why this only thing in > > the entire system should be different. You can also hover over the > > button to see the shortcut. > > Indeed <- and -> is used to edit the text search but keyboard focus can > open the use of <- -> for navigation. Not being able to use -> is > counter intuitive. > > > Not to mention that you have two different right arrows (ctrl+right, > > ctrl+shift+right), so just because you immediately see one use case > > doesn't mean that it's the only one/best one. There was A LOT of > > discussion on the mailing list regarding Spotter's usability and > shortcuts. > > It is inconsistent: why no need of Ctrl for up and down, but a need for > it for left and right. > Because horizontal navigation in such a way is the same everywhere. But with control you are taking different action. But point above about new/accustomed users holds here as well. > > > > >> Playground > > Why Ctrl+shift+g to open a pane, looks like Ctrl+g is working. > > Nevertheless Ctrl+g is not very handy short cut, it hurts the fingers! > > Can it be changed? > > > > Because it's not the same. do-it-all-and-go (ctrl+shift+g) executes all > > the code and inspects it in the next pane, while ctrl+g (do-it-and-go) > > just the selected block/current line. > > ok. But Ctrl+g is really really not fingers friendly for a short cut you > will use very often. See screenshot, there is so much tension in the > three fingers when doing it. It is torture and I can tell you my hands > are large celtic ones. I guess you need two hands to use comfortably > this shortcut. > Interesting, I press the control with my thumb. As a proper touch-type user I should actually use the opposite shift than the key is on (so right shfit for G, left shift for H, etc.)â¦ I don't know why I type it with one hand. The default meta key on linux is ctrl, so such hand-braking shortcuts are not so uncommon (ctrl+shift+Debug, ctrl+shift+Format). Peter

April 25, 2016

Re: [Pharo-dev] XMLParser changes

by monty

+1 > Sent: Monday, April 25, 2016 at 6:23 AM > From: "Jan Vrany" <jan.vrany(a)fit.cvut.cz> > To: pharo-dev(a)lists.pharo.org > Subject: Re: [Pharo-dev] XMLParser changes > > Not sure whether this was an intention. but strictly speaking,Â > new behavior is correct.Â > See Extensible Markup Language (XML) 1.0 (Fifth Edition),Â > SectionÂ 2.10 White Space Handling: > > "An XML processor must always pass all characters in a document that > are not markup through to the application. A validating XML processor > must also inform the application which of these characters constitute > white space appearing in element content." > > Jan > > > > On Mon, 2016-04-25 at 12:14 +0200, Cyril Ferlicot Delbecque wrote: > > Hi, > > > > last week there was a new stable version of XMLParser and some tests > > broke in some tools. I think that there was a regression in this > > version. > > > > Snippet: > > > > (XMLDOMParser parse: '<?xml version="1.0" encoding="UTF-8"?> > >  > > <Clones> > > Â Â Â Â <ClonedFragment cloneName="test"> > > Â Â Â Â Â Â Â Â <Member fileName="ProgramA"/> > > Â Â Â Â Â Â Â Â <Member fileName="ProgramB"/> > > Â Â Â Â </ClonedFragment> > > Â Â Â Â <ClonedFragment cloneName="test2"> > > Â Â Â Â Â Â Â Â <Member fileName="ProgramA"/> > > Â Â Â Â Â Â Â Â <Member fileName="ProgramB"/> > > Â Â Â Â </ClonedFragment> > > </Clones>') elements first nodes > > > > With the release 2.7.4 we get 2 nodes but in release 2.7.6 we get 5 > > nodes. The 2 previous ones and 3 empty String nodes. > > > > I think this is not what we expect. Correct me if I am wrong :) > > > > > >

April 25, 2016