Hi Thanks Stef for fighting with the leak and thanks Max for the fast response.
I think was me who added the method #on:do:, and my reasoning was: - FileStream has #fileNamed: and #fileNamed:do: - PositionableStream has #on: - I needed the analog to FileStream>>fileNamed:do: in PositionableStream So, I created #on:do:. It looked consistent. Martín On Sun, Aug 4, 2013 at 5:02 PM, Stéphane Ducasse <[email protected]> wrote: > impressive :) > Tx guys. > We are fighting with memory leaks like mad :) > > Stef > > On Aug 4, 2013, at 4:49 PM, Max Leske <[email protected]> wrote: > >> Done. >> >> Name: ConfigurationOfFuel-MaxLeske.207 >> Author: MaxLeske >> Time: 4 August 2013, 4:49:17.037 pm >> UUID: 5cc59075-e365-4a58-a20b-22e40b2b70c9 >> Ancestors: ConfigurationOfFuel-MartinDias.206 >> >> * created new version 1.9.2 for development >> * pointed development to 1.9.2 >> * includes changes Camillo wanted to introduce into 1.9.1 (DateAndTime fix) >> >> >> On 04.08.2013, at 16:35, Max Leske <[email protected]> wrote: >> >>> Yeah, no problem. I'll get on it right away. >>> >>> Max >>> >>> On 04.08.2013, at 16:10, Stéphane Ducasse <[email protected]> wrote: >>> >>>> Hi guys >>>> >>>> can you rename this method? >>>> >>>> on: aCollectionOrStream do: aBlock >>>> "Evaluates a block with a new stream based on the collection or >>>> stream. Answers the result of the block evaluation. Follows the style of >>>> FileStream>>fileNamed:do:." >>>> | aStream | >>>> aStream := self on: aCollectionOrStream. >>>> [ ^ aBlock value: aStream ] ensure: [ aStream close ] >>>> >>>> >>>> Please do not use on:do: for something that is not about exception (and >>>> announcements). >>>> >>>> It is too confusing and refactor after. >>>> >>>> Stef >>> >> >> >> _______________________________________________ >> Pharo-fuel mailing list >> [email protected] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-fuel > > > _______________________________________________ > Pharo-fuel mailing list > [email protected] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-fuel
