- Pharo-dev - lists.pharo.org

Re: [Pharo-project] [ANN] 10095

by StÃ©phane Ducasse

tx > - Reorganization of the world menu and its sub-menus #185 > - Kernel-Number fixes #245 > - Replaced (Symbol allInstances) with (Symbol allSymbol) #227 > - Duplicated methods in ClassTrait #140 > - Remove #hash from Association (fix #testHash) #196 > - Fix MC initialization broken by previous update what was it?

Oct. 13, 2008

[Pharo-project] [ANN] 10095

by Adrian Lienhard

- Reorganization of the world menu and its sub-menus #185 - Kernel-Number fixes #245 - Replaced (Symbol allInstances) with (Symbol allSymbol) #227 - Duplicated methods in ClassTrait #140 - Remove #hash from Association (fix #testHash) #196 - Fix MC initialization broken by previous update Cheers, Adrian ___________________ http://www.adrian-lienhard.ch/

Oct. 13, 2008

Re: [Pharo-project] SLICE-Mantis-Collection-Fix-mining-nice-2

by Adrian Lienhard

Thanks, Nicolas! These fixes will appear in update 10095. It would be helpful if you could also create a report in the issue tracker and post the comments there too. This helps us to better track changes. I created the following issue for this mail: http://code.google.com/p/pharo/issues/detail?id=245 Cheers, Adrian On Oct 9, 2008, at 00:54 , nicolas cellier wrote: > I have uploaded a SLICE in PharoInbox gathering some more or less > important collection fixes from mantis. > > > GATHER SEVERAL COLLECTION RELATED MANTIS TESTS AND FIXES > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=7172 > > CharacterSetComplement has spoiled previous byteArrayMap optimization. > Using #byteArrayMap is an optimization per se, possibly tailored for > tight loops. > Taking too much time to compute this map is spoiling efficiency. > It is necessary to cache this map for restoring efficiency to the > level preceding introduction of CharacterSetComplement. > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=6998 > > ((String new: 1) at: 1 put: $a) = $a -> true. > ((WideString new: 1) at: 1 put: $a) = $a -> false. > > WideString at:put: should return the put object like any other > collection > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=5331 > http://bugs.squeak.org/view.php?id=6366 > > Lots of tests and few patches for WideString not behaving correctly. > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=7180 +6455 +1603 +1602 > > Patches for Interval indexOf: and includes: > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=6994 > > Authorize this: > (#(1 2 3) as: ByteArray) as: Interval > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=7121 (ar) > > Patches for OrderedCollection add:after: add:before: > Try this one to be convinced for the necessity: > | oc | > oc := #(1 2 3 4) asOrderedCollection. > (oc first:3) do: [:e | oc remove: e]. > oc add: 0 beforeIndex: -2. > ^oc > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=6977 > incorrect use of super > (super basicNew in #new: dating from some 3.9 changes) > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=6778 > SkipList copy is shallow > > ---------------------------------------- > > http://bugs.squeak.org/view.php?id=6994 > deprecate the Service related #startsWith: because single > implementor completely redundant with beginsWith: > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Oct. 13, 2008

[Pharo-project] SLICE-Mantis-Collection-Fix-mining-nice-2

by nicolas cellier

Please note that original Collections-nice.111 had the deprecated method (at least my local copy of it) SequenceableCollection>>startsWith: aCollection self deprecated: 'Use beginsWith:.'. ^self beginsWith: aCollection Maybe Stef was overzealous because I replaced two sole senders in base image (Service-Base-nice.38) ?

Oct. 13, 2008

Re: [Pharo-project] [ANN] 10093

by Simon Kirk

Sorry - sent this while offline, and didn't get online before having read the whole conversation that ensued - ignore my previous mail :) On 13 Oct 2008, at 18:12, Simon Kirk wrote: > Consistency and clarity? > > On 13 Oct 2008, at 17:24, Matthew Fulmer wrote: > >> On Mon, Oct 13, 2008 at 05:50:56PM +0200, Adrian Lienhard wrote: >>> method categories cleanup by Lukas Renggli #241: >>> - remove empty categories >>> - sort all categories >>> - make all categories lower case >>> - replace #initialize-release/#class-initialization with >>> #initialization >> >> Why? >> >> -- >> Matthew Fulmer -- http://mtfulmer.wordpress.com/ >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > > ****************************************************************************************************************************************** > This email is from Pinesoft Limited. Its contents are confidential > to the intended recipient(s) at the email address(es) to which it > has been addressed. It may not be disclosed to or used by anyone > other than the addressee(s), nor may it be copied in anyway. If > received in error, please contact the sender, then delete it from > your system. Although this email and attachments are believed to be > free of virus, or any other defect which might affect any computer > or IT system into which they are received and opened, it is the > responsibility of the recipient to ensure that they are virus free > and no responsibility is accepted by Pinesoft for any loss or damage > arising in any way from receipt or use thereof. > ******************************************************************************************************************************************* > > > Pinesoft Limited are registered in England, Registered number: > 2914825. Registered office: 266-268 High Street, Waltham Cross, > Herts, EN8 7EA > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project ****************************************************************************************************************************************** This email is from Pinesoft Limited. Its contents are confidential to the intended recipient(s) at the email address(es) to which it has been addressed. It may not be disclosed to or used by anyone other than the addressee(s), nor may it be copied in anyway. If received in error, please contact the sender, then delete it from your system. Although this email and attachments are believed to be free of virus, or any other defect which might affect any computer or IT system into which they are received and opened, it is the responsibility of the recipient to ensure that they are virus free and no responsibility is accepted by Pinesoft for any loss or damage arising in any way from receipt or use thereof. ******************************************************************************************************************************************* Pinesoft Limited are registered in England, Registered number: 2914825. Registered office: 266-268 High Street, Waltham Cross, Herts, EN8 7EA

Oct. 13, 2008

Re: [Pharo-project] SLICE-Mantis-Collection-Fix-mining-nice-2

by StÃ©phane Ducasse

Tx. We should use more often deprecate: because this is a cheap way to support evolution without impacting directly clients. Stef On Oct 13, 2008, at 9:01 PM, Damien Pollet wrote: > In my image the problems are that OB-Enhancements and OB-Standard each > have a sender of #startsWith: > OBEnrichedMercuryQuery class>>selectClassForQuery: > OBCodeBrowser class>>defaultBrowserClass > > Replacing them with #beginsWith: fixes the problem. > > On Mon, Oct 13, 2008 at 4:44 PM, StÃ©phane Ducasse > <stephane.ducasse(a)inria.fr> wrote: >> Sequenceable>>startsWith: start >> | comp | >> self size < start size ifTrue: [^ false]. >> comp := true. >> (self first: start size) with: start >> do: [:ea :ea2 | ea = ea2 ifFalse: [comp := false]]. >> ^ comp >> >> And indeed with should use deprecate:... >> >> Stef >> >> >> On Oct 13, 2008, at 4:34 PM, Damien Pollet wrote: >> >>> On Thu, Oct 9, 2008 at 12:54 AM, nicolas cellier <ncellier(a)ifrance.com >>> > >>> wrote: >>>> >>>> http://bugs.squeak.org/view.php?id=6994 >>>> deprecate the Service related #startsWith: because single >>>> implementor >>>> completely redundant with beginsWith: >>> >>> #startsWith: is not there anymore but it's still used (e.g. by the >>> mercury field in OB) >>> >>> -- >>> Damien Pollet >>> type less, do more [ | ] http://people.untyped.org/damien.pollet >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> Pharo-project(a)lists.gforge.inria.fr >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > > -- > Damien Pollet > type less, do more [ | ] http://people.untyped.org/damien.pollet > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Oct. 13, 2008

Re: [Pharo-project] SLICE-Mantis-Collection-Fix-mining-nice-2

by Damien Pollet

In my image the problems are that OB-Enhancements and OB-Standard each have a sender of #startsWith: OBEnrichedMercuryQuery class>>selectClassForQuery: OBCodeBrowser class>>defaultBrowserClass Replacing them with #beginsWith: fixes the problem. On Mon, Oct 13, 2008 at 4:44 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > Sequenceable>>startsWith: start > | comp | > self size < start size ifTrue: [^ false]. > comp := true. > (self first: start size) with: start > do: [:ea :ea2 | ea = ea2 ifFalse: [comp := false]]. > ^ comp > > And indeed with should use deprecate:... > > Stef > > > On Oct 13, 2008, at 4:34 PM, Damien Pollet wrote: > >> On Thu, Oct 9, 2008 at 12:54 AM, nicolas cellier <ncellier(a)ifrance.com> >> wrote: >>> >>> http://bugs.squeak.org/view.php?id=6994 >>> deprecate the Service related #startsWith: because single implementor >>> completely redundant with beginsWith: >> >> #startsWith: is not there anymore but it's still used (e.g. by the >> mercury field in OB) >> >> -- >> Damien Pollet >> type less, do more [ | ] http://people.untyped.org/damien.pollet >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Damien Pollet type less, do more [ | ] http://people.untyped.org/damien.pollet

Oct. 13, 2008

Re: [Pharo-project] [ANN] 10093

by StÃ©phane Ducasse

I love the feeling to see the system improving :) and me just pressing update. :) Stef > Its a typo, sorry. > > Correct: 10094 > > Adrian > > On Oct 13, 2008, at 19:03 , Matthew Fulmer wrote: > >> On Mon, Oct 13, 2008 at 05:50:56PM +0200, Adrian Lienhard wrote: >>> method categories cleanup by Lukas Renggli #241: >>> - remove empty categories >>> - sort all categories >>> - make all categories lower case >>> - replace #initialize-release/#class-initialization with >>> #initialization >>> >>> >>> Please note that this takes quite long to load because it touches >>> most >>> packages. >> >> Are there two updates named 10093, or is there a typo? Stef made >> something else called 10093 yesterday >> >> -- >> Matthew Fulmer -- http://mtfulmer.wordpress.com/ >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

Oct. 13, 2008

Re: [Pharo-project] [ANN] 10093

by StÃ©phane Ducasse

no problem. Tx lukas Indeed cleaning start there too. Stef On Oct 13, 2008, at 6:48 PM, Matthew Fulmer wrote: > I'm sorry. I am starting a pointless bikeshed discussion. Good > work. > > On Mon, Oct 13, 2008 at 09:40:57AM -0700, Matthew Fulmer wrote: >> Maybe I should be a little more specific. This seems more like >> something the package maintainers should be doing. But then, I >> guess you guys are the package maintainers for the stuff in the >> core. >> >> On Mon, Oct 13, 2008 at 05:50:56PM +0200, Adrian Lienhard wrote: >>> method categories cleanup by Lukas Renggli #241: >>> - remove empty categories >> >> This one makes sense. >> >>> - sort all categories >> >> Sort by what? I try to sort categories by usefulness/publicness >> >>> - make all categories lower case >> >> I see no point in doing this. >> >>> - replace #initialize-release/#class-initialization with >>> #initialization >> >> what about #'initialize release' and #'class initialization'? >> >> Is there a coding convention you are trying to invent? >> >> -- >> Matthew Fulmer -- http://mtfulmer.wordpress.com/ >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > -- > Matthew Fulmer -- http://mtfulmer.wordpress.com/ > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

Oct. 13, 2008

Re: [Pharo-project] [ANN] 10093

by Adrian Lienhard

Its a typo, sorry. Correct: 10094 Adrian On Oct 13, 2008, at 19:03 , Matthew Fulmer wrote: > On Mon, Oct 13, 2008 at 05:50:56PM +0200, Adrian Lienhard wrote: >> method categories cleanup by Lukas Renggli #241: >> - remove empty categories >> - sort all categories >> - make all categories lower case >> - replace #initialize-release/#class-initialization with >> #initialization >> >> >> Please note that this takes quite long to load because it touches >> most >> packages. > > Are there two updates named 10093, or is there a typo? Stef made > something else called 10093 yesterday > > -- > Matthew Fulmer -- http://mtfulmer.wordpress.com/ > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Oct. 13, 2008