OrderedCollection>>at:ifAbsentPut:?

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
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

Let's rewrite it...



_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

2011/9/7 Julian Fitzell <[hidden email]>:
> Let's rewrite it...

Agree.

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

I assume that we won't hold up 3.0.6 for the fix to Issue 686?

Dale

----- Original Message -----
| From: "Philippe Marschall" <[hidden email]>
| To: "Seaside - developer list" <[hidden email]>
| Sent: Tuesday, September 6, 2011 9:35:57 PM
| Subject: Re: [Seaside-dev] OrderedCollection>>at:ifAbsentPut:?
|
| 2011/9/7 Julian Fitzell <[hidden email]>:
| > Let's rewrite it...
|
| Agree.
|
| Cheers
| Philippe
| _______________________________________________
| seaside-dev mailing list
| [hidden email]
| http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
|
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

I see no reason to.

John O'Keefe [|]
Principal Smalltalk Architect, Instantions Inc.

_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev