[squeak-dev] stream>>#nextPutAll: changed
to returncollectioninstead of the stream?
Levente Uzonyi
leves at elte.hu
Sun Jun 27 15:04:43 UTC 2010
On Sun, 27 Jun 2010, Rob Withers wrote:
> All Crypto except SSL and X509 passes unit tests - green. The X509 tests
> fail because the example certificates are expired.
>
> Back to the return value of #nextPutAll:, I am surprised that such basic
> protocol and classes are changing this late in the game. How are we to know
> that the current implementation is locked down and invariant?
It was a simple protocol cleanup. Stream >> #nextPutAll: returned the
argument, but some subclasses didn't. Since Squeak 4.1 you can rely
on the return value of #nextPutAll:, since all implementors return the
argument.
Levente
>
> just askin',
> Rob
>
> --------------------------------------------------
> From: "Rob Withers" <reefedjib at yahoo.com>
> Sent: Sunday, June 27, 2010 10:28 AM
> To: "The general-purpose Squeak developers list"
> <squeak-dev at lists.squeakfoundation.org>
> Subject: Re: [squeak-dev] stream>>#nextPutAll: changed to
> returncollectioninstead of the stream?
>
>> Hey David,
>>
>> Yeah, I fixed it this way:
>>
>> writeStream := ByteArray new writeStream
>> nextPutAll: self clientVersion asByteArray;
>> ...
>> nextPutAll: encodedCompressionMethods;
>> yourself.
>>
>> There is too much noise in the link you gave to find useful Crypto work. I
>> found one package. They should really integrate it into the Monticello
>> HTTP Crypto repository, which is where you will find SSL:
>>
>> http://www.squeaksource.com/Cryptography
>>
>> I'll need some time to make sure it is working correctly again.
>>
>> Cheers,
>> Rob
>> ----
>> The Modern Whig Party of Virginia - http://vawhigs.org
>>
>>
>>
>> --------------------------------------------------
>> From: "C. David Shaffer" <cdshaffer at acm.org>
>> Sent: Sunday, June 27, 2010 10:16 AM
>> To: "The general-purpose Squeak developers list"
>> <squeak-dev at lists.squeakfoundation.org>
>> Subject: Re: [squeak-dev] stream>>#nextPutAll: changed to return
>> collectioninstead of the stream?
>>
>>> On 06/27/10 09:01, Rob Withers wrote:
>>>> This has completely broken SSL. :-(
>>>>
>>>>
>>> I hit this as well. Here's a patch (although you likely already fixed
>>> it yourself). BTW, when using SSL you definitely want the improvements
>>> to Cryptography found on:
>>>
>>> http://croquet-src-01.oit.duke.edu:8886/Contributions
>>>
>>> If someone is doing updates of Cryptography/SSL they might want to look
>>> into getting this code into SqueakSource as having to search the
>>> universe for a working SSL is kind of frustrating :-)
>>>
>>> David
>>>
>>>
>>
>>
>>
>>>
>>>
>>
>
>
More information about the Squeak-dev
mailing list
|