[Seaside-dev] Re: [Seaside Issues] Issue 686 in seaside: WAUrl uses OrderedCollection>>at:ifAbsentPut:

Louis LaBrunda Lou at Keystone-Software.com
Wed Sep 7 16:13:52 UTC 2011


Hi John,

I'm a little confused by #OrderedCollection>>at:ifAbsentPut: at least on VA
Smalltalk.  #OrderedCollection can't have gaps.  So #at:ifAbsentPut: would
only make sense for adding something to the end, like #add:.  Or is this
really not about #OrderedCollection?

Lou

>I noticed this while porting 3.0.5 and added it to our compat layer.
>
>John O'Keefe [|]
>Principal Smalltalk Architect, Instantions Inc.
>On Sep 7, 2011 12:37 AM, <seaside at googlecode.com> wrote:
>> Status: Accepted
>> Owner: philippe... at gmail.com
>> Labels: Type-Portability Priority-High Version-Seaside3.0
>>
>> New issue 686 by philippe... at gmail.com: WAUrl uses
>> OrderedCollection>>at:ifAbsentPut:
>> http://code.google.com/p/seaside/issues/detail?id=686
>>
>> While porting 3.0.6 to GemStone, I came across some new code in WAUrl that
>
>> uses #at:ifAbsentPut: on an OrderedCollection (see
>> WAUrl>>privatePathParameterMapAt:)... this is new code for 3.0.6 and I
>> thought that I would call attention to it.
>>
>> OrderedCollection>>at:ifAbsentPut: is not implemented for Dolphin or
>> VisualWorks (at least the version James and I checked) so this code might
>> be trouble down the road ...
>>
>> For GemStone, I will add OrderedCollection>>at:ifAbsentPut: to the compat
>> layer, but you may want to rethink whether or not you will add Grease
>> support, changed the algorithm, or????
>>
>> Dale
>>
>>
>> _______________________________________________
>> issues mailing list
>> To unsubscribe, email issues-leave at lists.seaside.st
>> http://lists.seaside.st/listinfo/issues
-----------------------------------------------------------
Louis LaBrunda
Keystone Software Corp.
SkypeMe callto://PhotonDemon
mailto:Lou at Keystone-Software.com http://www.Keystone-Software.com


More information about the seaside-dev mailing list