http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip
The zip contains a new sources file. Marcus _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
MCMergeBrowser>>merger:
conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. OrderedCollection dnu sort: introduced I guess http://code.google.com/p/pharo/issues/detail?id=1320 how was this change tested? thanks, Mike On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: > http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip > > The zip contains a new sources file. > > Marcus > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
That was me, it comes from Squeak trunk. I didn't test it, because it
looked so harmless and works in Squeak … I'm sorry. It should be "sortBy: rather than sort:" niko On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: > MCMergeBrowser>>merger: > conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. > > OrderedCollection dnu sort: > > introduced I guess > > http://code.google.com/p/pharo/issues/detail?id=1320 > > how was this change tested? > > thanks, > Mike > > On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >> >> The zip contains a new sources file. >> >> Marcus >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
I clicked around for some time now but I can't trigger that method to
test it properly this time. It isn't called on conflicts, it isn't called on merges. What do you need to do to get that method executed artificially? Niko On Mon, Oct 19, 2009 at 8:48 PM, Niko Schwarz <[hidden email]> wrote: > That was me, it comes from Squeak trunk. I didn't test it, because it > looked so harmless and works in Squeak … I'm sorry. It should be > "sortBy: rather than sort:" > > niko > > On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >> MCMergeBrowser>>merger: >> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >> >> OrderedCollection dnu sort: >> >> introduced I guess >> >> http://code.google.com/p/pharo/issues/detail?id=1320 >> >> how was this change tested? >> >> thanks, >> Mike >> >> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>> >>> The zip contains a new sources file. >>> >>> Marcus >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
It has been superseded by Polymorph.
Maybe there was a way to use old interface ? Preferences ? Nicolas 2009/10/19 Niko Schwarz <[hidden email]>: > I clicked around for some time now but I can't trigger that method to > test it properly this time. It isn't called on conflicts, it isn't > called on merges. What do you need to do to get that method executed > artificially? > > Niko > > On Mon, Oct 19, 2009 at 8:48 PM, Niko Schwarz > <[hidden email]> wrote: >> That was me, it comes from Squeak trunk. I didn't test it, because it >> looked so harmless and works in Squeak … I'm sorry. It should be >> "sortBy: rather than sort:" >> >> niko >> >> On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >>> MCMergeBrowser>>merger: >>> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >>> >>> OrderedCollection dnu sort: >>> >>> introduced I guess >>> >>> http://code.google.com/p/pharo/issues/detail?id=1320 >>> >>> how was this change tested? >>> >>> thanks, >>> Mike >>> >>> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>>> >>>> The zip contains a new sources file. >>>> >>>> Marcus >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
In reply to this post by niko.schwarz
Implementation details: #sortBy: is not exactly #sort:
#sort: does a fast inplace sort, when #sortBy: performs useless copy operations... Nicolas 2009/10/19 Niko Schwarz <[hidden email]>: > That was me, it comes from Squeak trunk. I didn't test it, because it > looked so harmless and works in Squeak … I'm sorry. It should be > "sortBy: rather than sort:" > > niko > > On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >> MCMergeBrowser>>merger: >> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >> >> OrderedCollection dnu sort: >> >> introduced I guess >> >> http://code.google.com/p/pharo/issues/detail?id=1320 >> >> how was this change tested? >> >> thanks, >> Mike >> >> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>> >>> The zip contains a new sources file. >>> >>> Marcus >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Then, why was sort: yanked from OrderedCollection? I'm confused.
On Mon, Oct 19, 2009 at 9:17 PM, Nicolas Cellier <[hidden email]> wrote: > Implementation details: #sortBy: is not exactly #sort: > #sort: does a fast inplace sort, when #sortBy: performs useless copy > operations... > > Nicolas > > 2009/10/19 Niko Schwarz <[hidden email]>: >> That was me, it comes from Squeak trunk. I didn't test it, because it >> looked so harmless and works in Squeak … I'm sorry. It should be >> "sortBy: rather than sort:" >> >> niko >> >> On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >>> MCMergeBrowser>>merger: >>> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >>> >>> OrderedCollection dnu sort: >>> >>> introduced I guess >>> >>> http://code.google.com/p/pharo/issues/detail?id=1320 >>> >>> how was this change tested? >>> >>> thanks, >>> Mike >>> >>> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>>> >>>> The zip contains a new sources file. >>>> >>>> Marcus >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Don't know...
Maybe the plan was to move it up in SequenceableCollection, like http://bugs.squeak.org/view.php?id=7242 ? Nicolas 2009/10/19 Niko Schwarz <[hidden email]>: > Then, why was sort: yanked from OrderedCollection? I'm confused. > > On Mon, Oct 19, 2009 at 9:17 PM, Nicolas Cellier > <[hidden email]> wrote: >> Implementation details: #sortBy: is not exactly #sort: >> #sort: does a fast inplace sort, when #sortBy: performs useless copy >> operations... >> >> Nicolas >> >> 2009/10/19 Niko Schwarz <[hidden email]>: >>> That was me, it comes from Squeak trunk. I didn't test it, because it >>> looked so harmless and works in Squeak … I'm sorry. It should be >>> "sortBy: rather than sort:" >>> >>> niko >>> >>> On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >>>> MCMergeBrowser>>merger: >>>> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >>>> >>>> OrderedCollection dnu sort: >>>> >>>> introduced I guess >>>> >>>> http://code.google.com/p/pharo/issues/detail?id=1320 >>>> >>>> how was this change tested? >>>> >>>> thanks, >>>> Mike >>>> >>>> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>>>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>>>> >>>>> The zip contains a new sources file. >>>>> >>>>> Marcus >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [hidden email] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
See also http://code.google.com/p/pharo/issues/detail?id=1214
2009/10/19 Nicolas Cellier <[hidden email]>: > Don't know... > Maybe the plan was to move it up in SequenceableCollection, like > http://bugs.squeak.org/view.php?id=7242 ? > > Nicolas > > 2009/10/19 Niko Schwarz <[hidden email]>: >> Then, why was sort: yanked from OrderedCollection? I'm confused. >> >> On Mon, Oct 19, 2009 at 9:17 PM, Nicolas Cellier >> <[hidden email]> wrote: >>> Implementation details: #sortBy: is not exactly #sort: >>> #sort: does a fast inplace sort, when #sortBy: performs useless copy >>> operations... >>> >>> Nicolas >>> >>> 2009/10/19 Niko Schwarz <[hidden email]>: >>>> That was me, it comes from Squeak trunk. I didn't test it, because it >>>> looked so harmless and works in Squeak … I'm sorry. It should be >>>> "sortBy: rather than sort:" >>>> >>>> niko >>>> >>>> On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >>>>> MCMergeBrowser>>merger: >>>>> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >>>>> >>>>> OrderedCollection dnu sort: >>>>> >>>>> introduced I guess >>>>> >>>>> http://code.google.com/p/pharo/issues/detail?id=1320 >>>>> >>>>> how was this change tested? >>>>> >>>>> thanks, >>>>> Mike >>>>> >>>>> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>>>>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>>>>> >>>>>> The zip contains a new sources file. >>>>>> >>>>>> Marcus >>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> [hidden email] >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [hidden email] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> _______________________________________________ >> Pharo-project mailing list >> [hidden email] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
I don't use the polymorph merge tool. As discussed in a previous
thread it's very limited for doing serious merging. It's the first thing I turn off in an update. Just hit the merge button. thanks, Mike On Mon, Oct 19, 2009 at 8:30 PM, Nicolas Cellier <[hidden email]> wrote: > See also http://code.google.com/p/pharo/issues/detail?id=1214 > > 2009/10/19 Nicolas Cellier <[hidden email]>: >> Don't know... >> Maybe the plan was to move it up in SequenceableCollection, like >> http://bugs.squeak.org/view.php?id=7242 ? >> >> Nicolas >> >> 2009/10/19 Niko Schwarz <[hidden email]>: >>> Then, why was sort: yanked from OrderedCollection? I'm confused. >>> >>> On Mon, Oct 19, 2009 at 9:17 PM, Nicolas Cellier >>> <[hidden email]> wrote: >>>> Implementation details: #sortBy: is not exactly #sort: >>>> #sort: does a fast inplace sort, when #sortBy: performs useless copy >>>> operations... >>>> >>>> Nicolas >>>> >>>> 2009/10/19 Niko Schwarz <[hidden email]>: >>>>> That was me, it comes from Squeak trunk. I didn't test it, because it >>>>> looked so harmless and works in Squeak … I'm sorry. It should be >>>>> "sortBy: rather than sort:" >>>>> >>>>> niko >>>>> >>>>> On Mon, Oct 19, 2009 at 7:06 PM, Michael Roberts <[hidden email]> wrote: >>>>>> MCMergeBrowser>>merger: >>>>>> conflicts := aMerger conflicts sort: [:a :b | a operation <= b operation]. >>>>>> >>>>>> OrderedCollection dnu sort: >>>>>> >>>>>> introduced I guess >>>>>> >>>>>> http://code.google.com/p/pharo/issues/detail?id=1320 >>>>>> >>>>>> how was this change tested? >>>>>> >>>>>> thanks, >>>>>> Mike >>>>>> >>>>>> On Mon, Oct 19, 2009 at 4:25 PM, Marcus Denker <[hidden email]> wrote: >>>>>>> http://gforge.inria.fr/frs/download.php/24363/PharoCore-1.0-10491rc1.zip >>>>>>> >>>>>>> The zip contains a new sources file. >>>>>>> >>>>>>> Marcus >>>>>>> >>>>>>> _______________________________________________ >>>>>>> Pharo-project mailing list >>>>>>> [hidden email] >>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> [hidden email] >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [hidden email] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [hidden email] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [hidden email] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> > > _______________________________________________ > Pharo-project mailing list > [hidden email] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > _______________________________________________ Pharo-project mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project |
Free forum by Nabble | Edit this page |