- Pharo-dev - lists.pharo.org

ESUG deadline for Presentation

by stephane ducasse

Hi Happy Smalltalkers Two important points - We are starting to work on the program of ESUG 2022 and we will accept some more talks but do not wait too long. https://esug.github.io/2022-Conference/call2022.html <https://esug.github.io/2022-Conference/call2022.html> - The registration is open and the early rate is nicer than the late one :) https://registration.esug.org <https://registration.esug.org/> But you can sponsor ESUG by registering with late fees from 6th of July. But this is more expensive. S

June 3, 2022

Summary: SelectorTable improvements

by Marcus Denker

Hi, Here is a short summary of all the improvements related to the SelectorTable (which allows the tools to quickly check if a symbol is used as a selector): - It is reset at image build (#cleanUp) - re-creating is fast (#bench shows '16.902 per second' on my machine for rebuildSelectorTable) (we can even think about resetting it on #shutDown: now) - Recording happens at method installation, not compilation - this means we call it far less on code loading - not called anymore for doIts and FFI method creation - optimized: #isSelectorSymbol is ~3.5 times faster in Pharo11 compared to Pharo10 - most speedup due to using a identity dictionary, avoiding costly string comparing - all users of #findInternedSelector: have been changed to send #isSelectorSymbol to the symbol - #findInternedSelector: will be deprecated Marcus

June 3, 2022

This week (22/2022) on the Pharo Issue Tracker

by Marcus Denker

We merged 23 PRs and closed 16 Issue Tracker entries. - Major speedup of the SelectorTable (for testing if a symbol is used as a method name) #someSelector isSelectorSymbol - Simplified Symbol interning - Debugger now can open for clean blocks (enable setting, execute [1halt] value) - We updated Fuel from Version 3 to Version 5 Full list: SymbolTable/SelectorTable ========================= - Simpler (and slightly faster) Symbol>>#intern: #11291 https://github.com/pharo-project/pharo/pull/11291 - 11248-Check-CanSelectorTable-use-WeakIndentitySet #11311 https://github.com/pharo-project/pharo/pull/11311 - 11312-users-of--findInternedSelector-can-use-isSelectorSymbol #11313 https://github.com/pharo-project/pharo/pull/11313 - Improve: Symbol class>>#internSelector: can just record the symbol #11315 https://github.com/pharo-project/pharo/pull/11315 - Do not use #compactSymbolTable (see #11319) #11320 https://github.com/pharo-project/pharo/pull/11320 Clean Blocks ============ - 11186-Clean-Blocks-fix-to-make-debugger-work #11187 https://github.com/pharo-project/pharo/pull/11187 Debugger ======== - Adding debuggerLayout in TStDebuggerExtensionLayout` to be able to choose the layout the debugger extension should have #376 https://github.com/pharo-spec/NewTools/pull/376 - Re enable keybinds on the stack panel of the debugger #371 https://github.com/pharo-spec/NewTools/pull/371 - accesses to debugger session from debugger need refactoring #366 https://github.com/pharo-spec/NewTools/pull/366 - Fixing initial window size in debugger presenter #361 https://github.com/pharo-spec/NewTools/pull/361 Speed ===== - optimize MorphExtension>>#valueOfProperty: a little #11314 https://github.com/pharo-project/pharo/pull/11314 - Improve #indexOfMonth: to not create symbols and compare without using #match #10677 https://github.com/pharo-project/pharo/pull/10677 Features ======== - fix-ReMissingSubclassResponsibilityRule #11303 https://github.com/pharo-project/pharo/pull/11303 - Updated Fuel (and users of Fuel) to 5.0.4 #11133 https://github.com/pharo-project/pharo/pull/11133 - ReNotOptimizedIfRule #7018 https://github.com/pharo-project/pharo/pull/7018 Fixes ===== - Resolves #11184: printStringLimitedTo:Using: will fail to actually respect limit #11185 https://github.com/pharo-project/pharo/pull/11185 CleanUp ======= - ReleaseTest-Symbol-unused-classvar #11302 https://github.com/pharo-project/pharo/pull/11302 - remove methods that are the same in the superclass https://github.com/pharo-spec/Spec/pull/1276

June 3, 2022

[Issue Tracker] issues with proposed fixes

by Marcus Denker

Hi, Sometimes it happens that a *description* of a fix is added to the issue tracker entry. We try to tag them with âFix Proposed" https://github.com/pharo-project/pharo/labels/Fix%20proposed <https://github.com/pharo-project/pharo/labels/Fix%20proposed> In practice, it is either astonishingly hard to turn a description into a fix, or it is so trivial that it feels as if nothing has to be done. (It feels sometimes that finding the solution is the real contribution, while actually doing it feels like boring work). - add, if possible, the code of the whole changed method, too (I often wonder if it is all correct after doing the steps described by a textual description, especially if the method got changed in between) - Make it *really* clear in the issue tracker entry that a fix has been proposed. Sometimes it gets lost in the discussion. - Doing a PR from an issue that already has a description is very valuable This can a nice way to contribute for beginners. âIt donât understand if it is good, but I made a PR for the solution described in the issue" Marcus

June 2, 2022

[ Review needed] Some easy to review PRs

by Marcus Denker

Some easy to review small improvements (which in turn makes them easier to review): 1) use #isSelectorSymbol instead of #findInternedSelector: https://github.com/pharo-project/pharo/pull/11313 2) Improve: Symbol class>>#internSelector: can just record the symbol https://github.com/pharo-project/pharo/pull/11315 3) optimize MorphExtension>>#valueOfProperty: a little https://github.com/pharo-project/pharo/pull/11314 Marcus

June 1, 2022

Re: Question about MethodDictionary in TraitedClass (methodDict/localMethodDict)

by Andrei Chis

On Thu, May 12, 2022 at 3:19 PM Marcus Denker <marcus.denker(a)inria.fr> wrote: > > > > On 12 May 2022, at 11:27, Andrei Chis <chisvasileandrei(a)gmail.com> > wrote: > > > > Hi, > > > > We were profiling the loading of code that uses many Traits. We saw > something like this (this is a tally for > `TraitedMetaclass>>#addAndClassifySelector:withMethod:inProtocol:`): > > <Screenshot 2022-05-11 at 17.17.37.png> > > > > From what we saw in > `TraitedMetaclass>>#addAndClassifySelector:withMethod:inProtocol:` a method > is first added to `localMethodDict` and then it calls the super method that > adds the selector to the `methodDict` of the actual class in > `Behaviour>>addSelectorSilently:withMethod: `. Because of this we basically > get an overhead as a method is added to two method dictionaries and adding > a method to a MethodDictionary is slower, as it does operations like > `flushCache` and `cachePragmas`. > > > > Should the `localMethodDict` also be an instance of MethodDictionary or > could it be a simple dictionary? > > > > I think it can be normal Dictionary: MethodDictionary we just meed where > there VM executes code (for flushCache) and for lookup. > Ok. I can experiment with replacing it with a Dictionary and see how it goes. > > The only downside is that it then uses a little bit more memory > (MethodDictionary does not use associations), but that should be no problem. > Indeed that's an interesting aspect. Maybe more important for the Pharo image is memory rather than loading speed. Cheers, Andrei > > Marcus >

May 30, 2022

Re: [PR in need of Review] Redraw world on SDL_WINDOWEVENT_EXPOSED #11240

by Gabriel Cotelli

I've reviewed the changes and tried them in my image and seem to work fine. It fixes the glitch when switching windows on Linux and Pharo is not redrawn. On Mon, May 30, 2022 at 7:56 AM Marcus Denker <marcus.denker(a)inria.fr> wrote: > This PR needs a review: > > Redraw world on SDL_WINDOWEVENT_EXPOSED #11240 > https://github.com/pharo-project/pharo/pull/11240 >

May 30, 2022

[PR in need of Review] Redraw world on SDL_WINDOWEVENT_EXPOSED #11240

by Marcus Denker

This PR needs a review: Redraw world on SDL_WINDOWEVENT_EXPOSED #11240 https://github.com/pharo-project/pharo/pull/11240

May 30, 2022

This week (21/2022) on the Pharo Issue Tracker

by Marcus Denker

This week we merged 27 PRs and fixed 23 Issues. Symbol/SelectorTable Improvements ================================ - Symbol-simpler-findInternedSelector #11281 https://github.com/pharo-project/pharo/pull/11281 - 11243-Symbol-classrebuildSelectorTable-hould-not-use-allInstancesDo #11244 https://github.com/pharo-project/pharo/pull/11244 - remove OneCharacterSymbols from Symbol #11247 https://github.com/pharo-project/pharo/pull/11247 Features ======== - Add rules for simplification of isNotNil ifTrue:, notNil ifTrue:ifFalse #11261 https://github.com/pharo-project/pharo/pull/11261 Fixes ====== - Pharo11-UFT8-Monticello #11239 fixes: Commits with ... (introduced by GH) in summary cannot be loaded back in Pharo #11226 https://github.com/pharo-project/pharo/pull/11239 Debugger ======== - Adding debuggerLayout in TStDebuggerExtensionLayout` to be able to choose the layout the debugger extension should have #376 https://github.com/pharo-spec/NewTools/pull/376 - Fixing odd behaviour in debugger inspector filters #359 https://github.com/pharo-spec/NewTools/pull/359 - Fixes context menu commands of the StDebugger stack. #373 https://github.com/pharo-spec/NewTools/pull/373 - [cleanup] fix selector name: initalizeSelectedList #369 https://github.com/pharo-spec/NewTools/pull/369 Cleanups ======== - 11245-Trivial-FinderUI-classsearchedTextListMaxSize-can-be-removed #11249 https://github.com/pharo-project/pharo/pull/11249 - 11255-OrderedCollection-adding-methods-remove-a-poppushTemp #11256 https://github.com/pharo-project/pharo/pull/11256 - [clean up] isLiteral and TestRunner method never call #11257 https://github.com/pharo-project/pharo/pull/11257 - Fixed issue 11210: Do not skip NativeArrayTest>>#testDoubleArraysHaveCorrectLayout #11260 https://github.com/pharo-project/pharo/pull/11260 - 10185-merge-ReSmalllintChecker-into-SmalllintManifestChecker #11219 https://github.com/pharo-project/pharo/pull/11219 Trivial Cleanups ================ - Cleanup IdentityTransform class>>#initialize #11289 https://github.com/pharo-project/pharo/pull/11289 - Cleanup: categorize ProcessLocalSlotTest #11288 https://github.com/pharo-project/pharo/pull/11288 - Cleanup: align #isTransparent and other #isXXX methods #11280 https://github.com/pharo-project/pharo/pull/11280 - Cleanup: #isCharacter should be in same "testing" category (not "acceâ¦ #11278 https://github.com/pharo-project/pharo/pull/11278 - Cleanup: small improvements for Webbrowser package #11283 https://github.com/pharo-project/pharo/pull/11283 - Cleanup Delay>>#isExpired #11269 https://github.com/pharo-project/pharo/pull/11269 - Cleanup: lint for ComputedSlotTest>>#testReadComputedSlotCompiled #11285 https://github.com/pharo-project/pharo/pull/11285 - Cleanup: remove unnecessary class #initialize comment #11292 https://github.com/pharo-project/pharo/pull/11292 - Cleanup: #isBlock should be in same "testing" category (not "accessing") #11277 https://github.com/pharo-project/pharo/pull/11277 - Properly categorize "is..." methods #11267 https://github.com/pharo-project/pharo/pull/11267 - Cleanup: small category cleanup in Path #11274 https://github.com/pharo-project/pharo/pull/11274 - #isResumable shoud be in "testing" or "private - testing" #11271 https://github.com/pharo-project/pharo/pull/11271 - Cleanup-DeadCode-05-20 #11250 https://github.com/pharo-project/pharo/pull/11250

May 30, 2022

Re: About Image Size Pharo9->Pharo11

by Marcus Denker

> On 11 May 2022, at 09:00, Marcus Denker <marcus.denker(a)inria.fr> wrote: > > Pharo9: 65,8 MB > Pharo10: 57,4 MB > Pharo11: 56,5 MB Another small step: Pharo11: 55,4 MB Reset SelectorTable on #cleanUp #11235 https://github.com/pharo-project/pharo/pull/11235 - avoid interning selectors on compile, just do it on install (which means it will not be done for DoIts and FFI methods) - This allows to reset it on #cleanUp - Which is done on the CI as part of the build. - trivial: simplify #internSelector: a little (no temp needed) Not perfect, of course, as the SelectorTable is created on the first compile and then only removed if you call the reset method (e.g. via âSymbol cleanUpâ). But on the plus side, besides a smaller distributed image size, even e.g. a pre-build worker image should never allocate this and be a bit smaller during execution. Marcus

May 20, 2022