- Pharo-dev - lists.pharo.org

Code quality: tagging unsent selectors (e.g. in tests)
by Marcus Denker Jan. 30, 2023

Jan. 30, 2023

Hi, If you have in your code a message send with a selector that is not implemented anywhere, it likely will result in a DNU - but not if you compile a method on the fly (used in tests often) - if the DNU is catched (e.g. for a proxy pattern) - if the code is compatibility code, never executed in this version of the system We can ban the rule, but that just stopps checking for the whole method for all selectors, what we want is to communicate that *this selector* is ok in this special use (per method granularity is ok). SystemNaviation>>#allUnimplementedCalls did have the feature that one can tag selectors that should be ignored. This uses the pragma #ignoreUnimplementedCalls: see Context class >> initializePrimitiveSimulators <ignoreUnimplementedCalls: #(#registerPrimitiveSimulators)> This was used in just one method, I think because a) the rule did not take it into account and b) the terminology is just wrong (âCallâ). I think - ReSentNotImplementedRule should takes this pragma into account - change the bad terminology âallUnimplementedCalls" - use pragma: <ignoreNotImplementedSelectors: > PR is here: https://github.com/pharo-project/pharo/pull/12295 (what the PR does not yet do is to change syntax highlight to not color the selector red, this could be done later, too)

3 5

This week (4/2023) on the Pharo Issue Tracker
by Marcus Denker Jan. 27, 2023

Jan. 27, 2023

This week, we merged 28 Pull requests, fixing 31 issues Fixes ===== - Update BaselineOfPharo with Roassal3 version v0.9.9L #12430 https://github.com/pharo-project/pharo/pull/12430 - checkIfMustRefresh in OmBlockFileStore>>blockForLocalNameAsInteger: #12197 https://github.com/pharo-project/pharo/pull/12197 - Fix for multiple Epicea issues #12375 https://github.com/pharo-project/pharo/pull/12375 - FIX CI issues - FFIExternalString>>#basicHandle:at: calls deprecated message #12405 https://github.com/pharo-project/pharo/pull/12405 - Fix for "Browse this version in Calypso" Error after clicking an unloaded package #12423 https://github.com/pharo-project/pharo/pull/12423 - 12410-Minimal-Image-does-not-open-ChangesFile-correctly #12411 https://github.com/pharo-project/pharo/pull/12411 - fix testTallyInstructions #12393 https://github.com/pharo-project/pharo/pull/12393 - Spotter is broken> DNU #tools #12397 https://github.com/pharo-project/pharo/pull/12397 Cleanups ======== - RubScrolledTextMorph>>#mouseWheel: sends unimplemented message #scrolâ¦ #12402 https://github.com/pharo-project/pharo/pull/12402 - 12416-Fixing-Classes-Reported-by-NoUnusedVariablesLeftTest #12417 https://github.com/pharo-project/pharo/pull/12417 - 12413-Improving-the-dependency-of-Formatter-Settings-with-EFFormatter #12414 https://github.com/pharo-project/pharo/pull/12414 - Move #asNumber from AST-Core to Kernel (where it is also already tested) #12388 https://github.com/pharo-project/pharo/pull/12388 - extract return from ifTrue:ifFalse: in Integer>>#printStringRadix: #12433 https://github.com/pharo-project/pharo/pull/12433 - 11x Cleanups of the kind Cleanup: Condense Calypso packages #12284 https://github.com/pharo-project/pharo/pull/12284 this Changelog is archived at https://github.com/pharo-project/pharo-changelogs/blob/master/weekly/2023/4…

1 0

[Issue Tracker] reviews needed
by Marcus Denker Jan. 26, 2023

Jan. 26, 2023

Hi, I wanted again to make some noise about the issue tracker. Yes, we merge 20-40 PRs per week, but even with that, there are many PRs in the queueâ¦ e.g. - Complete symbols #11918 https://github.com/pharo-project/pharo/pull/11918 - 12128-Poor-performance-with-large-WriteStreams #12205 https://github.com/pharo-project/pharo/pull/12205 - Fix Calypso browsing unloaded package artifacts from Iceberg #12425 https://github.com/pharo-project/pharo/pull/12425 - Introducing a Color Palette to cache the allocation of Color objects by tht UITheme #12419 https://github.com/pharo-project/pharo/pull/12419 We have open 35 PRs, the complete list is here: https://github.com/pharo-project/pharo/pulls some random notes: - reviews are always helpful, and even if there is already a positive review, a second one helps. - one part of a review is to check that the code is not doing obvious bad things (like un-doing older changes, dumb mistakesâ¦.) if you feel unsure what you can do as you do not really understand the change, that one you can check (and just say that in the review). - if you do PRs e.g with a pair, ask the person that did not commit to add a real review so it is clear that there where already two eyes on the code - if you work in a team, ask your team mates to review your code especially if you use that fix already internally - if you use the fix in your project already, add a review to just state that - if you see a fix where you think âI hope this will be merged soonâ, if you review, you will make it much easierâ¦ - if you see a fix that is reviewed, green and not merged and you do not understand why: do a review - consider just reviewing / doing *something* once every week. or even once every day, instead of letting it all pile up. Marcus

1 0

Pharo11 32 bits images missing?
by Pierre Misse Chanabier Jan. 23, 2023

Jan. 23, 2023

Hello, Apparently Pharo 11 does not build 32 bit images [1]. Did I miss an announcement? Pierre http://files.pharo.org/image/110/

2 1

[ANN] Pharo Mastodon
by Marcus Denker Jan. 20, 2023

Jan. 20, 2023

Yes, not really news: Since Jan 4, 2018 we have a mastodon account. Esteban created that very early on, for now it re-shares the @pharoproject twitter account. https://mastodon.social/@pharoproject <https://mastodon.social/@pharoproject> I have finally updated https://www.pharo.org/community to link to it. Marcus

1 0

This week (3/2023) on the Pharo Issue Tracker
by Marcus Denker Jan. 20, 2023

Jan. 20, 2023

Features ======== - Add ifCutOn:doWithCutAndUncuts: #12374 https://github.com/pharo-project/pharo/pull/12374 - Lint rule warning for Calypso to warn unconditional recursion #12389 https://github.com/pharo-project/pharo/pull/12389 - Add #normalizeCamelCase and test #12373 https://github.com/pharo-project/pharo/pull/12373 - Add Generator class>>onDo: #12395 https://github.com/pharo-project/pharo/pull/12395 Fixes ===== - Variables: Fix Prrinting and Fluid Class definition that uses ClassVariable #12333 https://github.com/pharo-project/pharo/pull/12333 - Spotter is broken> DNU #tools #12397 https://github.com/pharo-project/pharo/pull/12397 - fixes #1659 IceCredentialStore did not understand #repositoryModel #1661 https://github.com/pharo-vcs/iceberg/pull/1661 - Issue 1275: Notify scroll changes #1344 https://github.com/pharo-spec/Spec/pull/1344 - fix tags access #1660 https://github.com/pharo-vcs/iceberg/pull/1660 Cleanup ======= - Move SmallInteger class>>#guideToDivision comment to class comment. #12391 https://github.com/pharo-project/pharo/pull/12391 - avoid hardcoding smalltalk everywhere and just factor out some use foâ¦ #449 https://github.com/pharo-spec/NewTools/pull/449 - Refactoring/categorizing unexposed refactorings #12370 https://github.com/pharo-project/pharo/pull/12370 - 20x cleanups of the kind Cleanup: Condense FFI-Kernel package #12328 https://github.com/pharo-project/pharo/pull/12328 - extract return from ifTrue:ifFalse: expression and whitespaces at the eol #12332 https://github.com/pharo-project/pharo/pull/12332 - extract return from ifTrue:ifFalse: expression #12177 https://github.com/pharo-project/pharo/pull/12177

1 0

Fwd: Iceberg in github action
by Pierre Misse Chanabier Jan. 14, 2023

Jan. 14, 2023

Hi, After quite a bit of work and investigating, we are finally able to use iceberg scripting in smalltalkci :) https://github.com/hpi-swa/smalltalkCI/pull/582 I'm using it in my Polyphemus repository if you want an example (https://github.com/hogoww/Polyphemus/actions) Cheers, Pierre

1 0

This week (2/2023) on the Pharo Issue Tracker
by Marcus Denker Jan. 13, 2023

Jan. 13, 2023

42 PRs merged, 40 Issues closed Features ======= - RubTextEditor/TextEditor: Make ctrl+backspace delete the previous word #12096 https://github.com/pharo-project/pharo/pull/12096 - Add source and target labels to SpChooserPresenter ( #1341) #1342 https://github.com/pharo-spec/Spec/pull/1342 Git/Iceberg ============ - add a manual way to add/remove tags #1657 https://github.com/pharo-vcs/iceberg/pull/1657 - add default tag "system" to bootstrap registered projects #12303 https://github.com/pharo-project/pharo/pull/12303 - fix bad categorizations #1655 https://github.com/pharo-vcs/iceberg/pull/1655 - Adapt to latest spec2 pass2 #1632 https://github.com/pharo-vcs/iceberg/pull/1632 Refactoring Engine ================== - Cleaning RB issues #12151 https://github.com/pharo-project/pharo/pull/12151 - Refactorings cleanup #12256 https://github.com/pharo-project/pharo/pull/12256 Fixes ===== - Fix refactoring using old class def #12195 https://github.com/pharo-project/pharo/pull/12195 - remove ByteLayout from FFIBackend and children #12225 https://github.com/pharo-project/pharo/pull/12225 - 12289-Refactoring-Tests-are-failling-when-using-Fluid-syntax #12290 https://github.com/pharo-project/pharo/pull/12290 - Fix hasSelector:specialSelectorIndex: on FFI methods that are compiled #12305 https://github.com/pharo-project/pharo/pull/12305 - 12300-Compiler-is-slow-with-string-literals-containing-exclamation-marks #12306 https://github.com/pharo-project/pharo/pull/12306 Cleanups ======== - 18 whitespace cleanups like Cleanup: Condense CodeImport packages #12229 https://github.com/pharo-project/pharo/pull/12229 - 5 remove return out of isTrue:ifFalse: lke extract return from ifTrue:ifFalse: expression in Number floor #12292 https://github.com/pharo-project/pharo/pull/12292 - remove #Numeric from source code. #12265 https://github.com/pharo-project/pharo/pull/12265 - 12307 remove gt pragma #12308 https://github.com/pharo-project/pharo/pull/12308 - Unncessary class RBMissingOpenerErrorNode #12313 https://github.com/pharo-project/pharo/pull/12313 - Class RBShortAssignmentToken is not needed anymore #12315 https://github.com/pharo-project/pharo/pull/12315 Comments ======== - Improving layoutFrame comments and add examples (fixes #12259) #12260 https://github.com/pharo-project/pharo/pull/12260 - extract comments as runnable example in Number>>#// #12263 https://github.com/pharo-project/pharo/pull/12263 - use isInstanceSide message in example #12270 https://github.com/pharo-project/pharo/pull/12270

1 0

[Website] some fixes: links to mailinglist and newsletter
by Marcus Denker Jan. 12, 2023

Jan. 12, 2023

Hi, A small update to https://www.pharo.org/community I fixed the links to the mailinglist and the newsletter subcription. (there are real âaboutâ pages.. I guess I was not the only one complaining that this is neededâ¦) https://lists.pharo.org/list/pharo-dev.lists.pharo.org https://lists.pharo.org/list/pharo-users.lists.pharo.org <https://lists.pharo.org/list/pharo-users.lists.pharo.org> (this closes https://github.com/pharo-project/pharo-site/issues/13) Marcus

1 0

IWST proceedings online
by stephane ducasse Jan. 11, 2023

Jan. 11, 2023

Hi guys Iâm happy to sahre with you this EXCELLENT news. The proceedings of IWST 22 are now online https://ceur-ws.org/Vol-3325/ï¿¼ in a nice collection. I would like to warmly thank all the authors, loic and vincent for their great job. Soon we will send around the call for paper for 2023! S

1 0