June 2015 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] [pharo-project/pharo-core] da585a: 50145

by Marcus Denker

This update lists mistakenly 15396 Refactor Duplicated #itemFromPoint: (3 slightly-differing implementations) as being part of this update, it is not because it has been already integrated earlier, but the report was not closed, leading it to be integrated again. Marcus > On 30 Jun 2015, at 15:24, GitHub <noreply(a)github.com> wrote: > > Branch: refs/heads/5.0 > Home: https://github.com/pharo-project/pharo-core > Commit: da585ad0fa675fff5ef4044db436387d86faf2c1 > https://github.com/pharo-project/pharo-core/commit/da585ad0fa675fff5ef4044d… > Author: Jenkins Build Server <board(a)pharo-project.org> > Date: 2015-06-30 (Tue, 30 Jun 2015) > > Changed paths: > A Kernel.package/ProcessList.class/README.md > A Kernel.package/ProcessList.class/class/accessing/streamSpecies.st > A Kernel.package/ProcessList.class/class/instance creation/newFrom_.st > A Kernel.package/ProcessList.class/class/instance creation/new_.st > A Kernel.package/ProcessList.class/class/stream creation/new_streamContents_.st > A Kernel.package/ProcessList.class/definition.st > A Kernel.package/ProcessList.class/instance/accessing/at_.st > A Kernel.package/ProcessList.class/instance/accessing/at_putLink_.st > A Kernel.package/ProcessList.class/instance/accessing/at_put_.st > A Kernel.package/ProcessList.class/instance/accessing/first.st > A Kernel.package/ProcessList.class/instance/accessing/firstLink.st > A Kernel.package/ProcessList.class/instance/accessing/last.st > A Kernel.package/ProcessList.class/instance/accessing/lastLink.st > A Kernel.package/ProcessList.class/instance/accessing/size.st > A Kernel.package/ProcessList.class/instance/accessing/swap_with_.st > A Kernel.package/ProcessList.class/instance/adding/addFirst_.st > A Kernel.package/ProcessList.class/instance/adding/addLast_.st > A Kernel.package/ProcessList.class/instance/adding/add_.st > A Kernel.package/ProcessList.class/instance/adding/add_afterLink_.st > A Kernel.package/ProcessList.class/instance/adding/add_after_.st > A Kernel.package/ProcessList.class/instance/adding/add_beforeLink_.st > A Kernel.package/ProcessList.class/instance/adding/add_before_.st > A Kernel.package/ProcessList.class/instance/copying/copyWith_.st > A Kernel.package/ProcessList.class/instance/copying/copyWithout_.st > A Kernel.package/ProcessList.class/instance/copying/postCopy.st > A Kernel.package/ProcessList.class/instance/enumerating/collect_.st > A Kernel.package/ProcessList.class/instance/enumerating/collect_thenSelect_.st > A Kernel.package/ProcessList.class/instance/enumerating/do_.st > A Kernel.package/ProcessList.class/instance/enumerating/linksDo_.st > A Kernel.package/ProcessList.class/instance/enumerating/select_.st > A Kernel.package/ProcessList.class/instance/enumerating/select_thenCollect_.st > A Kernel.package/ProcessList.class/instance/enumerating/species.st > A Kernel.package/ProcessList.class/instance/private/indexOf_startingAt_ifAbsent_.st > A Kernel.package/ProcessList.class/instance/private/linkAt_.st > A Kernel.package/ProcessList.class/instance/private/linkAt_ifAbsent_.st > A Kernel.package/ProcessList.class/instance/private/linkOf_.st > A Kernel.package/ProcessList.class/instance/private/linkOf_ifAbsent_.st > A Kernel.package/ProcessList.class/instance/private/validIndex_.st > A Kernel.package/ProcessList.class/instance/removing/removeAll.st > A Kernel.package/ProcessList.class/instance/removing/removeAllSuchThat_.st > A Kernel.package/ProcessList.class/instance/removing/removeFirst.st > A Kernel.package/ProcessList.class/instance/removing/removeLast.st > A Kernel.package/ProcessList.class/instance/removing/removeLink_.st > A Kernel.package/ProcessList.class/instance/removing/removeLink_ifAbsent_.st > A Kernel.package/ProcessList.class/instance/removing/remove_ifAbsent_.st > A Kernel.package/ProcessList.class/instance/testing/isEmpty.st > M Kernel.package/ProcessorScheduler.class/instance/accessing/highestPriority_.st > R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50144.st > A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50145.st > R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50144.st > A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50145.st > M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st > > Log Message: > ----------- > 50145 > 15396 Refactor Duplicated #itemFromPoint: (3 slightly-differing implementations) > https://pharo.fogbugz.com/f/cases/15396 > > 15868 Differentiate ProcessList and LinkedList > https://pharo.fogbugz.com/f/cases/15868 > > http://files.pharo.org/image/50/50145.zip > >

June 30, 2015

Re: [Pharo-dev] Stack

by Thierry Goubier

2015-06-30 14:05 GMT+02:00 Sean P. DeNigris <sean(a)clipperadams.com>: > Thierry Goubier wrote > > I consider that subclassing should be used for implementation reuse and > > not for subtyping. > > That is the GoF position and it makes a lot of sense to me. In fact, I > think > we Smalltalkers suffer from McLuhan's "people become their tools" syndrome > in that, because the browser makes it easy to view inheritance trees, we > confuse inheritance with subtyping, creating unnecessary coupling. This > also > adds to the "Smalltalk has no APIs" problem; when only subclasses are > considered subtypes, one never has to define what is and is not the public > API; protocols could help here, but have never really been fleshed out for > this purpose and are a mess right now due to overloading with extension > method duties. > I'll wonder what could be done with Traits about that. Traits may be better at establishing an API. Tools could do more to hide the extension thing; I still can't understand why, more than two years after I did it, all Pharo users are still writing their * a-long-package-name by hand. I would hate having to explain to Pharo beginners how to make a method an extension. (by the way, dropping a method on a package is a nice way to get a DNU in Nautilus on Pharo 4...) Thierry

June 30, 2015

[pharo-project/pharo-core]

by GitHub

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

June 30, 2015

[pharo-project/pharo-core] da585a: 50145

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: da585ad0fa675fff5ef4044db436387d86faf2c1 https://github.com/pharo-project/pharo-core/commit/da585ad0fa675fff5ef4044d… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-06-30 (Tue, 30 Jun 2015) Changed paths: A Kernel.package/ProcessList.class/README.md A Kernel.package/ProcessList.class/class/accessing/streamSpecies.st A Kernel.package/ProcessList.class/class/instance creation/newFrom_.st A Kernel.package/ProcessList.class/class/instance creation/new_.st A Kernel.package/ProcessList.class/class/stream creation/new_streamContents_.st A Kernel.package/ProcessList.class/definition.st A Kernel.package/ProcessList.class/instance/accessing/at_.st A Kernel.package/ProcessList.class/instance/accessing/at_putLink_.st A Kernel.package/ProcessList.class/instance/accessing/at_put_.st A Kernel.package/ProcessList.class/instance/accessing/first.st A Kernel.package/ProcessList.class/instance/accessing/firstLink.st A Kernel.package/ProcessList.class/instance/accessing/last.st A Kernel.package/ProcessList.class/instance/accessing/lastLink.st A Kernel.package/ProcessList.class/instance/accessing/size.st A Kernel.package/ProcessList.class/instance/accessing/swap_with_.st A Kernel.package/ProcessList.class/instance/adding/addFirst_.st A Kernel.package/ProcessList.class/instance/adding/addLast_.st A Kernel.package/ProcessList.class/instance/adding/add_.st A Kernel.package/ProcessList.class/instance/adding/add_afterLink_.st A Kernel.package/ProcessList.class/instance/adding/add_after_.st A Kernel.package/ProcessList.class/instance/adding/add_beforeLink_.st A Kernel.package/ProcessList.class/instance/adding/add_before_.st A Kernel.package/ProcessList.class/instance/copying/copyWith_.st A Kernel.package/ProcessList.class/instance/copying/copyWithout_.st A Kernel.package/ProcessList.class/instance/copying/postCopy.st A Kernel.package/ProcessList.class/instance/enumerating/collect_.st A Kernel.package/ProcessList.class/instance/enumerating/collect_thenSelect_.st A Kernel.package/ProcessList.class/instance/enumerating/do_.st A Kernel.package/ProcessList.class/instance/enumerating/linksDo_.st A Kernel.package/ProcessList.class/instance/enumerating/select_.st A Kernel.package/ProcessList.class/instance/enumerating/select_thenCollect_.st A Kernel.package/ProcessList.class/instance/enumerating/species.st A Kernel.package/ProcessList.class/instance/private/indexOf_startingAt_ifAbsent_.st A Kernel.package/ProcessList.class/instance/private/linkAt_.st A Kernel.package/ProcessList.class/instance/private/linkAt_ifAbsent_.st A Kernel.package/ProcessList.class/instance/private/linkOf_.st A Kernel.package/ProcessList.class/instance/private/linkOf_ifAbsent_.st A Kernel.package/ProcessList.class/instance/private/validIndex_.st A Kernel.package/ProcessList.class/instance/removing/removeAll.st A Kernel.package/ProcessList.class/instance/removing/removeAllSuchThat_.st A Kernel.package/ProcessList.class/instance/removing/removeFirst.st A Kernel.package/ProcessList.class/instance/removing/removeLast.st A Kernel.package/ProcessList.class/instance/removing/removeLink_.st A Kernel.package/ProcessList.class/instance/removing/removeLink_ifAbsent_.st A Kernel.package/ProcessList.class/instance/removing/remove_ifAbsent_.st A Kernel.package/ProcessList.class/instance/testing/isEmpty.st M Kernel.package/ProcessorScheduler.class/instance/accessing/highestPriority_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50144.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50145.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50144.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50145.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50145 15396 Refactor Duplicated #itemFromPoint: (3 slightly-differing implementations) https://pharo.fogbugz.com/f/cases/15396 15868 Differentiate ProcessList and LinkedList https://pharo.fogbugz.com/f/cases/15868 http://files.pharo.org/image/50/50145.zip

June 30, 2015

Re: [Pharo-dev] [pharo-project/pharo-core] 05f052: 50144

by Marcus Denker

This update lists issue 15867 as integrated as part of this update, but this was already integrated in some other earlier update but the issue was not closed back them, leading to me integrating again, which leads to an update that lists this issue yet does not contain any changes related to the issue. On Tue, Jun 30, 2015 at 2:58 PM, GitHub <noreply(a)github.com> wrote: > Branch: refs/heads/5.0 > Home: https://github.com/pharo-project/pharo-core > Commit: 05f05284ec286ed63c0e0ff2d8fdc19dfe9f1481 > > https://github.com/pharo-project/pharo-core/commit/05f05284ec286ed63c0e0ff2… > Author: Jenkins Build Server <board(a)pharo-project.org> > Date: 2015-06-30 (Tue, 30 Jun 2015) > > Changed paths: > M AST-Core.package/RBMessageNode.class/instance/testing/lastIsReturn.st > M > OpalCompiler-Core.package/extension/RBMessageNode/instance/isInlineIf.st > M > OpalCompiler-Core.package/extension/RBMessageNode/instance/isInlineIfNil.st > M > OpalCompiler-Tests.package/OCOpalExamples.class/instance/examples-blocks-optimized/exampleIfTrue.st > R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/ > script50143.st > A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/ > script50144.st > R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/ > update50143.st > A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/ > update50144.st > M > ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st > > Log Message: > ----------- > 50144 > 15450 VM crash with disabled optionInlineIfNil > https://pharo.fogbugz.com/f/cases/15450 > > 15867 Fix shortcut for smart suggestions in Nautilus > https://pharo.fogbugz.com/f/cases/15867 > > http://files.pharo.org/image/50/50144.zip > > > -- -- Marcus Denker -- denker(a)acm.org http://www.marcusdenker.de

June 30, 2015

[pharo-project/pharo-core]

by GitHub

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

June 30, 2015

[pharo-project/pharo-core] 05f052: 50144

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 05f05284ec286ed63c0e0ff2d8fdc19dfe9f1481 https://github.com/pharo-project/pharo-core/commit/05f05284ec286ed63c0e0ff2… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-06-30 (Tue, 30 Jun 2015) Changed paths: M AST-Core.package/RBMessageNode.class/instance/testing/lastIsReturn.st M OpalCompiler-Core.package/extension/RBMessageNode/instance/isInlineIf.st M OpalCompiler-Core.package/extension/RBMessageNode/instance/isInlineIfNil.st M OpalCompiler-Tests.package/OCOpalExamples.class/instance/examples-blocks-optimized/exampleIfTrue.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50143.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50144.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50143.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50144.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50144 15450 VM crash with disabled optionInlineIfNil https://pharo.fogbugz.com/f/cases/15450 15867 Fix shortcut for smart suggestions in Nautilus https://pharo.fogbugz.com/f/cases/15867 http://files.pharo.org/image/50/50144.zip

June 30, 2015

Re: [Pharo-dev] Stack

by Sean P. DeNigris

Thierry Goubier wrote > I consider that subclassing should be used for implementation reuse and > not for subtyping. That is the GoF position and it makes a lot of sense to me. In fact, I think we Smalltalkers suffer from McLuhan's "people become their tools" syndrome in that, because the browser makes it easy to view inheritance trees, we confuse inheritance with subtyping, creating unnecessary coupling. This also adds to the "Smalltalk has no APIs" problem; when only subclasses are considered subtypes, one never has to define what is and is not the public API; protocols could help here, but have never really been fleshed out for this purpose and are a mess right now due to overloading with extension method duties. ----- Cheers, Sean -- View this message in context: http://forum.world.st/Stack-tp4834494p4834869.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

June 30, 2015

Re: [Pharo-dev] Stack

by stepharo

:) my question was to check if we can change it :) Le 30/6/15 11:10, Max Leske a Ã©crit : >> On 30 Jun 2015, at 11:00, stepharo <stepharo(a)free.fr> wrote: >> >> >> >> Le 28/6/15 19:00, Sven Van Caekenberghe a Ã©crit : >>>> On 28 Jun 2015, at 17:59, Max Leske <maxleske(a)gmail.com> wrote: >>>> >>>> >>>>> On 28 Jun 2015, at 17:22, stepharo <stepharo(a)free.fr> wrote: >>>>> >>>>> Hi guys >>>>> >>>>> is there a real reason for Stack being a subclass of LinkedLink? >>>> 1. You donât need to adjust an internal data structure (grow, shrink, move, sort) to update the stack but only move references. >> But it is used in practice? >> Because Stack could use a LinkedList and you get the same benefits and not stupid methods that do not make sense for Stack. > agreed. > >>>> 2. The interface is pretty similar >> No there are not :) >> Stack is push pop top not insert after:..... >> or this is a strange stack. > What I meant was: Linked list provides an interface that allows for stack operations. Not that the naming conventions are the same. > >>> 3. It is an implementation detail/technique, it is indeed not as if Stack is-a LinkedList (from that point of view it is confusing) >> What I mean is that this is not good to have subclassing when we can use subclassing. >> We remove the fact that Dictionary is a subclass of Set for this reason. >> > You asked for reasons. I didnât say they were *good* reasons :) > >>>>> Stef >>>>> >>> >> > >

June 30, 2015

[pharo-project/pharo-core] 6f0eb1: 50143

by GitHub

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 6f0eb1c292ae34764b5e0ba4809355818878f247 https://github.com/pharo-project/pharo-core/commit/6f0eb1c292ae34764b5e0ba4… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-06-30 (Tue, 30 Jun 2015) Changed paths: M ConfigurationOfGlamourCore.package/ConfigurationOfGlamourCore.class/instance/symbolic versions/stable_.st A ConfigurationOfGlamourCore.package/ConfigurationOfGlamourCore.class/instance/versions/version315_.st A ConfigurationOfGlamourCore.package/ConfigurationOfGlamourCore.class/instance/versions/version316_.st R Morphic-Widgets-List.package/SimpleHierarchicalListMorph.class/instance/event handling/itemFromPoint_.st R Morphic-Widgets-Pluggable.package/PluggableListMorph.class/instance/accessing/itemFromPoint_.st A Morphic-Widgets-Pluggable.package/PluggableMultiColumnListMorph.class/instance/accessing/findSubmorphFor_.st R Morphic-Widgets-Pluggable.package/PluggableMultiColumnListMorph.class/instance/accessing/itemFromPoint_.st A Morphic-Widgets-Scrolling.package/ScrollPane.class/instance/event handling/findSubmorphFor_.st A Morphic-Widgets-Scrolling.package/ScrollPane.class/instance/event handling/itemFromPoint_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50142.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50143.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50142.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50143.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M SmartSuggestions.package/SugsMenuBuilder.class/class/builder/buildSmartActionsShortcuts_.st M SmartSuggestions.package/SugsMenuBuilder.class/class/private/findBestNodeFor_.st Log Message: ----------- 50143 15396 Refactor Duplicated #itemFromPoint: (3 slightly-differing implementations) https://pharo.fogbugz.com/f/cases/15396 15863 Update Glamour to load last version of Rubric https://pharo.fogbugz.com/f/cases/15863 15867 Fix shortcut for smart suggestions in Nautilus https://pharo.fogbugz.com/f/cases/15867 http://files.pharo.org/image/50/50143.zip

June 30, 2015