- Pharo-dev - lists.pharo.org

Progress Report -> Refactoring Project - ( March 8 - 12 )
by Evelyn Cusi Lopez March 15, 2021

March 15, 2021

Hello all, Last week I did these tasks: - Fix PR: RB - Migrate some class commands to commander2 <https://github.com/pharo-project/pharo/pull/8709> #8709 <https://github.com/pharo-project/pharo/pull/8709> - Introduce new refactoring: Copy package <https://github.com/pharo-project/pharo/pull/8781> , I took longer than expected with this issue, since I had to modify and add some functionalities to the RBNamespace model - Improve comments of some refactorings <https://github.com/pharo-project/pharo/pull/8788> - Start to introduce a new refactoring: Remove method senders <https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:RemoveSend…>, I still have some problems with this refactoring because it doesn't work as it should on cascading senders, I keep fixing it. Tasks for this week: - Continue working in: "Remove Senders Of Method" refactoring <https://github.com/pharo-project/pharo/issues/8710> - Continue improving instance variable refactoring with live objects. <https://github.com/pharo-project/pharo/issues/8665> <https://github.com/pharo-project/pharo/issues/6218> - Improve extract setUp method refactoring. <https://github.com/pharo-project/pharo/issues/2617> - Fix issue: <https://github.com/pharo-project/pharo/issues/8786>Extract Method does not search in the subclasses <https://github.com/pharo-project/pharo/issues/8786> - Fix issue: move to component refactoring issue. <https://github.com/pharo-project/pharo/issues/8499> - Fix issue: <https://github.com/pharo-project/pharo/issues/2617>Rewritting tool fails to place a cascade inside a cascade <https://github.com/pharo-project/pharo/issues/2617> Regards, Evelyn C.

1 0

This week (10/2021) on the Pharo Issue Tracker
by Marcus Denker March 12, 2021

March 12, 2021

We merged 34 Pull Requests and fixed 41 Issues this week. https://github.com/pharo-project/pharo/pulse Debugger ======== - 8659 expression leading to error is not highlighted anymore https://github.com/pharo-project/pharo/pull/8770 - 8502 cannot proceed in debugger when loading fuel and bytecodes have changed #8768 https://github.com/pharo-project/pharo/pull/8768 - Fixing packages for extension packages in the debugger stack #8764 https://github.com/pharo-project/pharo/pull/8764 Refactoring Tooks ================== - RB - Introduce new refactoring "Merge inst var x in y" #8708 https://github.com/pharo-project/pharo/pull/8708 - RB - Migrate some class commands to commander2 #8709 https://github.com/pharo-project/pharo/pull/8709 - RB - Improve messages of remove class command #8685 https://github.com/pharo-project/pharo/pull/8685 - RB - Add support for traits in RBNamespace #8660 https://github.com/pharo-project/pharo/pull/8660 Fixes ===== - 8641-valueWithExit-should-answer-the-result-of-value #8696 https://github.com/pharo-project/pharo/pull/8696 - Package level comments #8724 https://github.com/pharo-project/pharo/pull/8724 - In CompiledMethod, replace #definition by #definitionString #8760 https://github.com/pharo-project/pharo/pull/8760 - 7756 Broken class definitions combining traits and slots [new try] #8683 https://github.com/pharo-project/pharo/pull/8683 - 8698-Failing-on-CI-ReleaseTesttestThatAllMethodsArePackaged #8699 https://github.com/pharo-project/pharo/pull/8699 - 8766-Adding-new-accessors-to-the-VM-object-to-statistics-and-parameters #8767 https://github.com/pharo-project/pharo/pull/8767 Cleanup: Unused Variables ========================== The release tests for unused instance and class variables are now active! This means that we will never merge a PR that has an unused variable. - SptSpotterProcessor has an ivar "order" that is not used #8723 https://github.com/pharo-project/pharo/pull/8723 - 8720-RBTraitedMetaclass-has-unused-ivar #8721 https://github.com/pharo-project/pharo/pull/8721 - 8678-Clean-unused-Class-Vars-so-we-can-un-skip-testNoUnusedClassVariablesLeft #8707 https://github.com/pharo-project/pharo/pull/8707 - 8189-testNoUnusedInstanceVariablesLeft-is-skipped #8719 https://github.com/pharo-project/pharo/pull/8719 Cleanup: Abstract Classes ========================== - TaskIt: TKTRunner and TKTAbstractExecutor should be abstract #8739 https://github.com/pharo-project/pharo/pull/8739 - Abstract classes in Font packages #8748 https://github.com/pharo-project/pharo/pull/8748 - Cleanups in DependencyAnalyzer (abstract classes, class comments) #8756 https://github.com/pharo-project/pharo/pull/8756 - Abstract classes in VariablesLibrary package #8750 https://github.com/pharo-project/pharo/pull/8750 - Zodiac: ZdcAbstractSocketStream and ZdcAbstractSSLSession as abstract classes #8735 https://github.com/pharo-project/pharo/pull/8735 - T2AbstractTest should be abstract #8745 https://github.com/pharo-project/pharo/pull/8745 - TaAbstractComposition should implement #isAbstract #8737 https://github.com/pharo-project/pharo/pull/8737 - Is ReNodeRewriteRule an abstract class? #8731 https://github.com/pharo-project/pharo/pull/8731 - ReAbstractCritique should implement #isAbstract to return true #8727 https://github.com/pharo-project/pharo/pull/8727 - ReAbstractRuleTestCase should be abstract #8716 https://github.com/pharo-project/pharo/pull/8716 - AbstractSessionHandler should be abstract #8714 https://github.com/pharo-project/pharo/pull/8714 Cleanups ======== - Proper method category names in TKTRunner and subclasses #8743 https://github.com/pharo-project/pharo/pull/8743 - Deprecate JapaneseEnvironment leftover methods #8754 https://github.com/pharo-project/pharo/pull/8754 - Extract example in FontChooser #8752 https://github.com/pharo-project/pharo/pull/8752 - Cleanups in AST packages #8758 https://github.com/pharo-project/pharo/pull/8758 - Cleanup method categories in TKTAbstractExecutor #8741 https://github.com/pharo-project/pharo/pull/8741 - Typo in class comment of ReSystemAnnouncer #8730 https://github.com/pharo-project/pharo/pull/8730

1 0

Re: Cmd-line (headless) Pharo Launcher - structure of commands
by Guillermo Polito March 11, 2021

March 11, 2021

Hi David, I think this is cool :) The things you describe are what I was having in mind too. The commands I use the most from the UI are: - create/delete an image - launch it :) About the design of the command line, have you checked Clap? It should provide support for parsing command line arguments and printing all the helps for you. Also, If I can dream, Iâd like that the launcher remembers the open images, so we could also introduce something like launcher kill imageXXX Thanks!!! G. > El 10 mar 2021, a las 14:00, bajger(a)gmail.com escribiÃ³: > > Hi all! > I had a chat with Stephane regarding his inquiry about Pharo Launcher controlled from command-line interface (headless mode). > I'd like to ask core Pharo developers to briefly review, if current list of commands seems to be ok. Document here: https://github.com/Bajger/Pharo-snippets/blob/master/pharoLauncherCmdLine-d… <https://github.com/Bajger/Pharo-snippets/blob/master/pharoLauncherCmdLine-d…> > Some notes: > - List of commands is based on what can be executed on UI of Pharo Launcher > - there are some open questions, that you can try to comment on. > > Also, please suggest, if I should move .md page to some more appropriate project (e.g. Pharo wiki?) > Thanks for feedback! > David >

2 1

Pharo on Emscripten (WebAssembly)
by Pavel Krivanek March 8, 2021

March 8, 2021

Hi, I just want to note that in 2019, I tried to build a Pharo VM to WebAssembly using Emscripten. It was not really working but I was able to, at least, build and run the VM. [image: EmPharo.png] The code is here: https://github.com/pavel-krivanek/EmPharo Maybe someone will be interested enough to try to push it :-) The main issues I found: - the VM seems to use the long jumps and in very ugly way - it specifies assembler labels and then it jumps to them - heartbeat - atomic operations (probably not so critical) Cheers, -- Pavel

1 0

Progress Report -> Refactoring Project - ( March 1 - 5 )
by Evelyn Cusi Lopez March 8, 2021

March 8, 2021

Hello all, Last week I did these tasks: - Add commands to use new refactorings with commander2 in class context. <https://github.com/pharo-project/pharo/pull/8663> - Migrate some class commands to commander2 <https://github.com/pharo-project/pharo/pull/8709> (SycAddSubclassCommand, SycInsertSubclassCommand and SycInsertSuperclassCommand) - Improve messages of remove class command. <https://github.com/pharo-project/pharo/pull/8685> - Close issue: <https://github.com/pharo-project/pharo/pull/8686>RB*Change should work on environment and not Smalltalk globals. <https://github.com/pharo-project/pharo/pull/8686> - Introduce new refactoring: "Merge instance variable into another". <https://github.com/pharo-project/pharo/pull/8708> - Start to improve inst variables refactorings with live objects. Tasks for this week: - Continue improving instance variable refactoring with live objects. <https://github.com/pharo-project/pharo/issues/8665> <https://github.com/pharo-project/pharo/issues/6218> - Continue improving extract setUp method refactoring. - Fix issue: Copy (updating scoped references) <https://github.com/pharo-project/pharo/issues/8568> - Fix issue: move to component refactoring issue. <https://github.com/pharo-project/pharo/issues/8499> - Fix <https://github.com/pharo-project/pharo/issues/6218>Inline senders refactoring issue <https://github.com/pharo-project/pharo/issues/6218> - Fix issue: <https://github.com/pharo-project/pharo/issues/2617>Rewritting tool fails to place a cascade inside a cascade <https://github.com/pharo-project/pharo/issues/2617> - Introduce: Refactoring idea "Remove User Of Method" <https://github.com/pharo-project/pharo/issues/8710> Regards, Evelyn C.

1 0

This week (09/2021) on the Issue Tracker
by Marcus Denker March 5, 2021

March 5, 2021

merge spec 0.8.4 #8679 =================== - add tree drag&drop support - improve merging of morphic stylesheets - fixed a bug on tables that was preventing users to listen keyUp/keyDown events - fixed a bug where styles were not applied to component list elements. - other minor fixes merge newtools 0.5.3 #8684 ====================== - introduce StPharoStyleContributor to allow tools to make explicit its style requirements - fixes Merge Iceberg/LibGit ===================== - fixing-initialization-of-iceberg #8693 https://github.com/pharo-project/pharo/pull/8693 - #call:options: send in LGitDiff>>#diff_find_similar:diff_find_options: #8128 https://github.com/pharo-project/pharo/issues/8128 - #postLoadActions of LibtGit needs to initialize the FFICompilerPlugin #5129 https://github.com/pharo-project/pharo/issues/5129 - Unused ivars in Iceberg #8606 https://github.com/pharo-project/pharo/issues/8606 - Unused class vars in Iceberg #8612 https://github.com/pharo-project/pharo/issues/8612 Refactoring Improvements ====================== - RB - Fix rename shortcut #8664 https://github.com/pharo-project/pharo/pull/8664 - RB - Add commands to use new refactorings with commander2 in class and method context #8663 https://github.com/pharo-project/pharo/pull/8663 - RB - Remove unnecessary initialization #8686 https://github.com/pharo-project/pharo/pull/8686 Debugger ======= - 8595 enhance debugger to show the dnu call in the stack #8651 https://github.com/pharo-project/pharo/pull/8651 - Fixing how undefined objects test for undefined variable #8653 https://github.com/pharo-project/pharo/pull/8653 - 8544 new debugger impossible to step into debugger complete to sender methods #8675 https://github.com/pharo-project/pharo/pull/8675 Fixes ===== - Fix issue 8642: ClassTrait instance variable vanishing when defining new instance variable #8650 https://github.com/pharo-project/pharo/pull/8650 - 8644-Rubric-Field-should-not-lose-the-password-state-when-a-font-is-applied #8648 https://github.com/pharo-project/pharo/pull/8648 - Decompiler-test-and-fix-super-inBlock-classSide #8629 https://github.com/pharo-project/pharo/pull/8629 - increasing-number-of-initial-stack-pages #8614 https://github.com/pharo-project/pharo/pull/8614 - Uncommenting to lines in ClyTextEditor so that Cmd-n and Cmd-m in theâ¦ #8640 https://github.com/pharo-project/pharo/pull/8640 - 8680-Calypso-is-showing-bad-the-shortcuts #8681 https://github.com/pharo-project/pharo/pull/8681 - Move SDL_HINT_RENDER_SCALE_QUALITY setting to non-OSX platforms #8691 https://github.com/pharo-project/pharo/pull/8691 - Activate smart characters even when in completion position #8692 https://github.com/pharo-project/pharo/pull/8692 - 8689-DropListBox-is-rendering-the-text-on-top-of-the-button #8690 https://github.com/pharo-project/pharo/pull/8690 Cleanup ======= - Adding a deprecation that was recommended by DepMiner and rewritten by Guille #8672 https://github.com/pharo-project/pharo/pull/8672 - Cleanup-unused-classvars-FT2Handle #8600 https://github.com/pharo-project/pharo/pull/8600 - cleanup-ununsed-classvar-LiteralVariableTest #8647 https://github.com/pharo-project/pharo/pull/8647 - cleanup-unused-classVar-ClySubclassLOfClass2FromP1 #8649 https://github.com/pharo-project/pharo/pull/8649 - cleanup-unused-classvar-RBTransformationRuleTestData1 #8645 https://github.com/pharo-project/pharo/pull/8645 - cleanup-unused-classvar-ClyClass2FromP1 #8646 https://github.com/pharo-project/pharo/pull/8646

1 0

Progress Report -> Refactoring Project - ( February 22 - 26 )
by Evelyn Cusi Lopez March 1, 2021

March 1, 2021

Hi all, Last week I did these tasks: - Improve convert temporary to instance variable refactoring <https://github.com/pharo-project/pharo/pull/8609> - Refactor methods to use Factory class instead of RB*Change classes directly <https://github.com/pharo-project/pharo/pull/8619> - Add support for traits in RBNamespace <https://github.com/pharo-project/pharo/pull/8660> - I fixed the issue of renaming classes and methods referenced in the traits' methods, so that it doesn't duplicate the method in classes that use the trait. - Add commands to use new refactorings with commander2 in Method context <https://github.com/pharo-project/pharo/pull/8663> - Fix rename shortcut <https://github.com/pharo-project/pharo/pull/8664> Tasks for this week: - Improve messages of issues: Incorrect message in Refactor push up with super sends #2656 <https://github.com/pharo-project/pharo/issues/2656> , Deleting a class with references gives extra "Do you want to delete anyway" msg #2586 - Fix move to component refactoring issue. - Fix Inline senders refactoring issue ( https://github.com/pharo-project/pharo/issues/6218) - Continue improving extract setUp method refactoring. - Improve instance variable refactoring <https://github.com/pharo-project/pharo/issues/8665> - Add ReplaceUsersOfInstanceVarXByY Refactoring <https://github.com/pharo-project/pharo/issues/8666> Regards, Evelyn C. <https://github.com/pharo-project/pharo/pull/8609#partial-pull-merging>

1 0

This week (08/2021) on the Issue Tracker
by Marcus Denker Feb. 26, 2021

Feb. 26, 2021

Merge NewTools 0.5.2 #8589 ========================== - VersionBrowser: fix DNU when browsing source code. - ExternalBrowser: Migrated (not activated yet) - Inspector: now it can inspect ProtoObjects - Inspector: add several object inspections (text, blocks) - Debugger: add segment selections to highlight position - Debugger: several bug fixes - a lot of small bugfixes - https://github.com/pharo-spec/NewTools/releases/tag/v0.5.2 Spec 0.8.3: =========== - add limited support for drag&drop (drop just to tables/lists, for the moment) - fixes a problem of sizing lists/trees within paned layouts. - fixes a problem on default SpIconProvider, which should answer a broken icon instead throwing an error if icon is not there - fixes selected icons in trees and sorted columns - minor bugfixes - https://github.com/pharo-spec/Spec/releases/tag/v0.8.3 upgrading Pharo 9 with Roassal 0.9.6 ==================================== - https://github.com/ObjectProfile/Roassal3/releases/tag/v0.9.6 Refactoring Browser enhancements =========================== RB - Improve convert temporary to instance variable refactoring #8609 https://github.com/pharo-project/pharo/pull/8609 RB - Improve some warning messages #8598 https://github.com/pharo-project/pharo/pull/8598 RB - Rename method only in specific packages #8597 https://github.com/pharo-project/pharo/pull/8597 RB - Improve extract set up method refactoring #8596 https://github.com/pharo-project/pharo/pull/8596 RB - Refactor methods to use Factory class instead of RB*Change classes directly #8619 https://github.com/pharo-project/pharo/pull/8619 Features ======== - Icon for abstract classes #8618 https://github.com/pharo-project/pharo/pull/8618 Fixes/Cleaning =============== - fix column width calculous #8628 https://github.com/pharo-project/pharo/pull/8628 - Rule for setUp and tearDown should be applied for TestResource too #8616 https://github.com/pharo-project/pharo/pull/8616 - improve browse history when a method belongs to multiple packages #8633 https://github.com/pharo-project/pharo/pull/8633 - 8564-Running-tests-from-the-command-line-does-not-execute-parametrized-cases #8565 https://github.com/pharo-project/pharo/pull/8565 - Fix #8621: Improve the help of the new spec-based inspector #8622 https://github.com/pharo-project/pharo/pull/8622 - Inconsistent casing in Debug menu item #8637 https://github.com/pharo-project/pharo/pull/8637 - 6438-TextEditorpluggableYellowButtonActivity-call-dead-code #8626 https://github.com/pharo-project/pharo/pull/8626 Cleanups unused variables ========================= - testNoUnusedClassVariablesLeft-improve #8624 https://github.com/pharo-project/pharo/pull/8624 - Cleanup-unused-classvars-WindowsStore #8603 https://github.com/pharo-project/pharo/pull/8603 - 8613-Unused-class-Variable-in-ZnConstants #8620 https://github.com/pharo-project/pharo/pull/8620 - cleanup-unused-ClassVar-ISO88597TextConverter #8611 https://github.com/pharo-project/pharo/pull/8611 - Fix-unused-classvars-CDClassWithFullDefinitionExample #8607 https://github.com/pharo-project/pharo/pull/8607 - Cleanup-unused-ivar-RBItemsSelectionPresenter #8605 https://github.com/pharo-project/pharo/pull/8605 - Cleanup-unused-classvars-Locale #8604 https://github.com/pharo-project/pharo/pull/8604 - Cleanup-unused-classVar-FFIStringCalloutTest #8590 https://github.com/pharo-project/pharo/pull/8590 - Unused-ClassVar-CleanUp-Deprecated #8591 https://github.com/pharo-project/pharo/pull/8591 - Cleanup-unused-classvars-FTSizeRec #8601 https://github.com/pharo-project/pharo/pull/8601

1 0

Progress Report -> Refactoring Project - ( February 8 - 12 )
by Evelyn Cusi Lopez Feb. 24, 2021

Feb. 24, 2021

Hi all Last week I did these tasks: - Add preview to select variables when running the generate accessors command (Fix issue #5270 <https://github.com/pharo-project/pharo/issues/5270>). - Separate the link between views and refactorings (setUp and pushDown method refactoring). - Add generic previews and remove specific previews of PushUp and PushDown refactoring (https://github.com/pharo-project/pharo/pull/8553) - I started the modification to make refactorings only in some packages, add a searcher for classes and methods in packages (these have reference to the package internally), I still have to finish implementing this part so that it is updated correctly when there are changes, deletions and additions. Fix issues #8513 <https://github.com/pharo-project/pharo/issues/8513>, #6366 <https://github.com/pharo-project/pharo/issues/6366>. - Improve "Move method to class side" Refactoring, the new version catch broken references (method senders and direct access to instVar) and fix them ( https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:NewMoveToC…) for instance: Before refactoring: *RBTransformationRuleTestData* >> rewriteUsing: searchReplacer rewriteRule := searchReplacer. self resetResult. After refactoring: *RBTransformationRuleTestData* >> rewriteUsing: searchReplacer ^ self class rewriteUsing: searchReplace. *RBTransformationRuleTestData* *class* >> rewriteUsing: searchReplacer | aRBTransformationRuleTestData | aRBTransformationRuleTestData := self new. aRBTransformationRuleTestData rewriteRule: searchReplacer. aRBTransformationRuleTestData resetResult'). ----------------------------------------------------------------------- Tasks for this week: - Improve extract setUp refactoring. - Finish to rename methods in specific packages (Fix issues #8513 <https://github.com/pharo-project/pharo/issues/8513>, #6366 <https://github.com/pharo-project/pharo/issues/6366>.) - Improve messages when exists a Warnings or Expections in refactorings. - Fix move to component refactoring issue ( #8499 <https://github.com/pharo-project/pharo/issues/8499>) - Fix extract method issue (#5855 <https://github.com/pharo-project/pharo/issues/5855>) - Check if it is better to change the behavior of the "Remove method" command, using a refactoring instead of a strategy Cheers, Evelyn

2 3

[ANN] Next Pharo Sprint: Feb 26
by Marcus Denker Feb. 24, 2021

Feb. 24, 2021

We will organize a Pharo sprint Feb 26 (this Friday) Goals of this sprint: - Fix issues from tracker https://github.com/pharo-project/pharo/issues This is a Remote Sprint, it happens on Discord. http://pharo.org/contribute-events We have added a Board to coordinate the work: https://github.com/orgs/pharo-project/projects/9

1 0