- Pharo-dev - lists.pharo.org

Re: Progress Report -> Refactoring Project - ( February 1 - 5 )

by Guillermo Polito

Hi Tim, you may want to have a look at Preserving Instance State during Refactorings in Live Environments https://hal.archives-ouvertes.fr/hal-02541754/document G > El 13 feb 2021, a las 9:46, Tim Mackinnon <tim(a)testit.works> escribiÃ³: > > I never thought of that implication - if ivars are proper objects (is that now applied?) would this issue go away (an ivar would just get a new parent)? > > This is a very interesting problem, as its a type of transaction, and I never though of it that way - its easy to think of it as just dumb code - but in effect it isnât. > > Tim > > On Fri, 12 Feb 2021, at 4:30 PM, Sean P. DeNigris wrote: >> Wonderful to have progress on this important topic - thank you! >> >> Sorry I haven't been following closely (maybe you addressed it already), but >> pushing up instance variables has a dangerous limitation - instances lose >> any data held in that var. I guess it's because it's deleted from the >> subclass prior to adding to the superclass to avoid duplicating. One >> solution would be to add a var to the superclass with a mangled name, copy >> the data for all instances, remove the var from the subclass, and then >> rename the mangled var. >> >> >> >> ----- >> Cheers, >> Sean >> -- >> Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html >>

Feb. 15, 2021

ARM64 VM for Manjaro/Arch

by Patrick Henning

Hello, I am writing to ask how I can access the beta ARM64 VM for Arch-type Linux (I am using Manjaro in particular). Thank you! Patrick

Feb. 15, 2021

Re: Progress Report -> Refactoring Project - ( February 1 - 5 )

by Tim Mackinnon

I never thought of that implication - if ivars are proper objects (is that now applied?) would this issue go away (an ivar would just get a new parent)? This is a very interesting problem, as its a type of transaction, and I never though of it that way - its easy to think of it as just dumb code - but in effect it isnât. Tim On Fri, 12 Feb 2021, at 4:30 PM, Sean P. DeNigris wrote: > Wonderful to have progress on this important topic - thank you! > > Sorry I haven't been following closely (maybe you addressed it already), but > pushing up instance variables has a dangerous limitation - instances lose > any data held in that var. I guess it's because it's deleted from the > subclass prior to adding to the superclass to avoid duplicating. One > solution would be to add a var to the superclass with a mangled name, copy > the data for all instances, remove the var from the subclass, and then > rename the mangled var. > > > > ----- > Cheers, > Sean > -- > Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html >

Feb. 13, 2021

Re: Progress Report -> Refactoring Project - ( February 1 - 5 )

by Sean P. DeNigris

Wonderful to have progress on this important topic - thank you! Sorry I haven't been following closely (maybe you addressed it already), but pushing up instance variables has a dangerous limitation - instances lose any data held in that var. I guess it's because it's deleted from the subclass prior to adding to the superclass to avoid duplicating. One solution would be to add a var to the superclass with a mangled name, copy the data for all instances, remove the var from the subclass, and then rename the mangled var. ----- Cheers, Sean -- Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html

Feb. 12, 2021

This week (06/2021) on the Issue Tracker

by Marcus Denker

merge spec2 0.8.2 (GUI) ================= - all presenters can now be styled (not just widgets as before). - lists/trees/tables: activation happens also on enter (not just double click). - applications now have icon providers to answer icons by application - start documenting layouts (box layout) - bugfixes - https://github.com/pharo-spec/Spec/releases/tag/v0.8.2 Move to Iceberg v2 (git) =================== In short, this version is a UI update that was required for Pharo 9.0. The functionality should be the same as in previous version. - Migration from Spec1 to Spec2 - Migration from Commander1 to Commander2 - General pass over source code, especially in TipUI package. - Pass on README and Wiki documentation - GitHub Actions workflows instead of Travis and AppVeyor - Tests don't depend on OSSubProcess or ProcessWrapper - https://github.com/pharo-vcs/iceberg/releases/tag/v2.0.0 Features ======== - New icon styler for disabled breakpoints #8530 https://github.com/pharo-project/pharo/pull/8530 Fixes ====== - 8503-Slot-remove-in-Stateful-Traits #8539 https://github.com/pharo-project/pharo/pull/8539 - 8540-TFFI-should-support-passing-a-ByteArray-in-a-char-parameter-a-TFStringType #8541 https://github.com/pharo-project/pharo/pull/8541 - Fixes #4011 Character nextObject crashes Pharo too #8545 https://github.com/pharo-project/pharo/pull/8545 - Fixes issue #7616 implementors and senders browsing broken in the debugger #8462 https://github.com/pharo-project/pharo/pull/8462 - 8551-Error-when-doing-implementors-of-of-non-string-literals #8552 https://github.com/pharo-project/pharo/pull/8552 - 8520-TwoWay-Become-should-not-loop-infinite-when-out-of-space https://github.com/pharo-project/pharo/pull/8523 - 7662-Must-be-boolean-deoptimization-targeting-wrong-or-block #8519 https://github.com/pharo-project/pharo/pull/8519 - ReflectivityTest>>#setUp copies the ASTCache #8527 https://github.com/pharo-project/pharo/pull/8527 - Testing and adding debugger inspector maximize assertion failure diff pane #8501 https://github.com/pharo-project/pharo/pull/8501 Cleanups ======== - AST-unify-children #8555 https://github.com/pharo-project/pharo/pull/8555 - 8042 deprecate and remove pointdist because there is pointdistance to with exactly the same implementation #8543 https://github.com/pharo-project/pharo/pull/8543 - 8353-startStepping-is-calling-an-undefined-method #8535 https://github.com/pharo-project/pharo/pull/8535 - fixes #4856 - print on methods should avoid calling blindly print string on their elements #8497 https://github.com/pharo-project/pharo/pull/8497 - Adding deprecations that were recommended by DepMiner (accepted by 2 reviewers) #8496 https://github.com/pharo-project/pharo/pull/8496 - Adding deprecations for Opal Compiler that were recommended by DepMiner and accepted by Pablo #8507 https://github.com/pharo-project/pharo/pull/8507 - Cleanup-Compiler-Backend-DeadCode #8517 https://github.com/pharo-project/pharo/pull/8517 - 6757-Remove-references-to-the-class-Interpreter #8518 https://github.com/pharo-project/pharo/pull/8518 - 8473-whenGotForcus-method-in-RubScrolledTextMorph-class #8516 https://github.com/pharo-project/pharo/pull/8516

Feb. 12, 2021

Progress Report -> Refactoring Project - ( February 1 - 5 )

by Evelyn Cusi Lopez

Hi all Last week I did these tasks: - Expose variable refactoring commands in source context ( https://github.com/pharo-project/pharo/pull/8472) - Improve messages of push up method refactoring, also add the option to push up referenced instance variables ( https://github.com/pharo-project/pharo/pull/8531) - Introduce a new way to move the method to class side refactoring ( https://github.com/pharo-project/pharo/pull/8532) - Add rename package refactoring ( https://github.com/pharo-project/pharo/pull/8533) Tasks for this week: - Improve extract setUp refactoring. - Fix previews of extract method with arguments ( https://github.com/pharo-project/pharo/issues/5855) - Improve renaming method refactoring, to only rename senders in specific packages (https://github.com/pharo-project/pharo/issues/8513) - Fix move to component issue ( https://github.com/pharo-project/pharo/issues/8499) Cheers, Evelyn

Feb. 8, 2021

This week (05/2021) on the Issue Tracker

by Marcus Denker

We merged 34 Pull Requests and closed 64 issues this week. Often one focuses on the pull requests. But even closed issue tracker entries that do not result in one PR did require work... even if it might be just spending 30 minutes trying to recreate the problem only to find that it has indeed been fixed already a long time ago. A complete list of the closed issues is available at https://github.com/pharo-project/pharo/pulse Here are the PRs: Debugger ======== - 2681 debugger step into does not go into accessors or methods with primitives #8456 https://github.com/pharo-project/pharo/pull/8456 - 8379-Debugger-Contextat-infinite-loop-when-pressing-Through-in-a-test-case-condition #8426 https://github.com/pharo-project/pharo/pull/8426 - Removing suspicious update code in the debugger inspector, and fixing variable update problem with recursive call #8430 https://github.com/pharo-project/pharo/pull/8430 - 8370 metalinks cannot be reinstalled if the image was saved while they were uninstalled #8448 https://github.com/pharo-project/pharo/pull/8448 Refactoring =========== - RB - Expose variable refactoring commands in source context #8472 https://github.com/pharo-project/pharo/pull/8472 - RB - Create new refactoring, to create accessors with lazy initialization #8464 https://github.com/pharo-project/pharo/pull/8464 - RB - Fix ExtractMethodTransformation #8465 https://github.com/pharo-project/pharo/pull/8465 - RB - Introduce Extract setUp refactoring #8466 https://github.com/pharo-project/pharo/pull/8466 Features ======== - Reorganize world menu #8414 https://github.com/pharo-project/pharo/pull/8414 Fixes ===== - Fix smart characters and add missing tests #8504 https://github.com/pharo-project/pharo/pull/8504 - TaRenameSlot-TestAndFix #8484 https://github.com/pharo-project/pharo/pull/8484 - Changeset-fileout-DNU #8486 https://github.com/pharo-project/pharo/pull/8486 - 692-Deprecation-exception-when-use-Open-Directory-Chooser-button-dialog-of-Iceberg-Settings-UI #8492 https://github.com/pharo-project/pharo/pull/8492 - 8458-Filelist-does-not-understand-readStream #8480 https://github.com/pharo-project/pharo/pull/8480 - Introduce asOpenInterval in Interval #8477 https://github.com/pharo-project/pharo/pull/8477 - 8450-ChangeSorter-should-open-and-work-with-new-Spec2 #8453 https://github.com/pharo-project/pharo/pull/8453 - 7987-Class-definition-should-not-be-evaluated-if-it-contains-a-syntax-error #8452 https://github.com/pharo-project/pharo/pull/8452 - Fix copy-paste to take into account real AST node #8449 https://github.com/pharo-project/pharo/pull/8449 Cleanups - Deprecations =================== Work related to deprecations.. as you might know, we can tag methods not only as deprecated, but add code to them which on-the-fly, at runtime, fixes the caller. This makes it much easier to evolve APIs. - Adding deprecations that were recommended by DepMiner (accepted by 3 reviewers) #8495 https://github.com/pharo-project/pharo/pull/8495 - Adding deprecations that were recommended by DepMiner (accepted by all 4 reviewers) #8494 https://github.com/pharo-project/pharo/pull/8494 - Adding transformation rules for two deprecated methods of Context. One of them was recommended by DepMiner but had to be modified #8510 https://github.com/pharo-project/pharo/pull/8510 - 7638-withSqueakLineEnding-should-be-moved-to-deprecated-package #8470 https://github.com/pharo-project/pharo/pull/8470 - 8471-Deprecate-OSSDL2WindowHandle #847 https://github.com/pharo-project/pharo/pull/8475 - 8467-Fuel-is-using-an-old-API-from-MultiByteFileStream #8468 https://github.com/pharo-project/pharo/pull/8468 Cleanups ======== And more cleanupsâ¦ Among other cleanups, we are still slowly fixing all the cases where a variable is defined but not used. Nearly there! when it is done, we add a release test and thus will never be able to merge code like that ever again. - DiskDirectoryEntry-remove-unused-classVar #8488 https://github.com/pharo-project/pharo/pull/8488 - Cleanup-unused-vars-JobProgressBarMorph #8498 https://github.com/pharo-project/pharo/pull/8498 - remove some unused class variables 1 #8433 https://github.com/pharo-project/pharo/pull/8433 - Classify one unclassified method in Trait #8487 https://github.com/pharo-project/pharo/pull/8487 - Clean the log file created on the disk by the test #8457 https://github.com/pharo-project/pharo/pull/8457 - 7889-GZipReadStream--fileIntoNewChangeSet-fullFileName #8447 https://github.com/pharo-project/pharo/pull/8447 - 7879-Clunky-interactive-but-bad-flow-for-fileIntoNewChangeSet #8493 https://github.com/pharo-project/pharo/pull/8493 - Move-ArgumentNamesTest-to-Debugger-Model-Tests-Core #8438 https://github.com/pharo-project/pharo/pull/8438 - 8454-Removing-more-users-of-oldDefinition #8455 https://github.com/pharo-project/pharo/pull/8455 Documentation ============== - 4372-Improve-ifNotNil-and-friends-comment-methods-to-indicate-that-they-are-optimized #8483 https://github.com/pharo-project/pharo/pull/8483 - Comment Interval>>#rangeIncludes: and more complete test #8446 https://github.com/pharo-project/pharo/pull/8446

Feb. 5, 2021

Progress Report -> Refactoring Project - ( January 24 - 28 )

by Evelyn Cusi Lopez

Hi all Last week I did these tasks: - Review PRs of clean unused variables ( https://github.com/pharo-project/pharo/pull/8425 , https://github.com/pharo-project/pharo/pull/8422) - Fix issue to reuse existing method in RBExtractMethodTransformation ( https://github.com/pharo-project/pharo/pull/8465 ). - Add new refactoring to generate accessors with lazy initialization ( https://github.com/pharo-project/pharo/pull/8464) - Review some issues and I help to close some were already fixed ( https://github.com/pharo-project/pharo/issues/6002 , https://github.com/pharo-project/pharo/issues/7108) - Fix issue to rename manifest class when rename a package, but I have problems with dependencies ( https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:RenameMani… ). - Add the first version of extract setUp refactoring ( https://github.com/pharo-project/pharo/pull/8466) Tasks for this week: - Fix dependencies problems of rename manifest issue. - Improve extract setUp refactoring. - Fix previews of extract method with arguments ( https://github.com/pharo-project/pharo/issues/5855) - Fix rename refactoring, this should rename only local methods ( https://github.com/pharo-project/pharo/issues/6366) - Fix protect instance variable issue Cheers, Evelyn

Feb. 1, 2021

This week (04/2021) on the Issue Tracker

by Marcus Denker

If you want to help, a nice first step to get familiar with improving the system is to review Pull Requests. For example, there are always very simple ones (like the ones listed here in "Cleanup Unused Variablesâ) that are fairly easy to understand and review. Open Pull requests are here: https://github.com/pharo-project/pharo/pulls Debugger improvements and Fixes =========================== - Added empty selection interval to force SpCodePresenter to auto-scroll. #8428 https://github.com/pharo-project/pharo/pull/8428 - OCDoitTest-testDoItContextReadIvar #8437 https://github.com/pharo-project/pharo/pull/8437 - 7763-Does-BlockClosurecull-need-the-pragma-debuggerCompleteToSender #8427 https://github.com/pharo-project/pharo/pull/8427 - Inspecting a Text no longer shows the formatted text #8352 https://github.com/pharo-project/pharo/issues/8352 Refactoring Improvements ======================== - Fix broken tests - extract method and ocurrences #8420 https://github.com/pharo-project/pharo/pull/8420 - RB - Add message to kwon how many occurrences were replaced #8380 https://github.com/pharo-project/pharo/pull/8380 Fixes ===== - 8168-CompiledMethodreferencedClasses-should-not-take-class-variables-into-account #8240 https://github.com/pharo-project/pharo/pull/8240 - Mouse events were sent twice when using an event handler #8404 https://github.com/pharo-project/pharo/pull/8404 - Mouse over event is sent multiple times #8401 https://github.com/pharo-project/pharo/pull/8401 - Improve the Help for Spec2 #8410 https://github.com/pharo-project/pharo/pull/8410 - Fix a regression from earlier pharo that used to perserve order in sequences #8402 https://github.com/pharo-project/pharo/pull/8402 Cleanup Unused Variables ========================= - spec1-clean-ivars #8431 https://github.com/pharo-project/pharo/pull/8431 - remove two unused ivars in GT-Tests #8432 https://github.com/pharo-project/pharo/pull/8432 - Remove-unused-vars-RB #8422 https://github.com/pharo-project/pharo/pull/8422 -RBExtractMethodAndOccurrences-unused-ivar #8425 https://github.com/pharo-project/pharo/pull/8425 - 8354-SycRemoveMessageArgumentCommand-with-unused-ivar #8419 https://github.com/pharo-project/pharo/pull/8419

Jan. 29, 2021

Re: Look for software for virtual outing

by askoh

Thanks. I think my employer, MathWorks, can find VR capabilities very interesting. Its Mission is: "Our goal is to change the world by accelerating the pace of discovery, innovation, development, and learning in engineering and science. We work to provide the ultimate computing environment for technical computation, visualization, design, simulation, and implementation. We use this environment to provide innovative solutions in a wide range of application areas." I hope we can try Terf. All the best, Aik-Siong Koh -- Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html

Jan. 27, 2021