ClyMethodGroup>>#fileOut dead code ?

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

ClyMethodGroup>>#fileOut dead code ?

Sven Van Caekenberghe-2
Hi,

ClyMethodGroup>>#fileOut seems like dead code (it sends #trailer to a WriteStream which DNUs). It also uses the deprecated FileStream.

Can it be removed ?

Sven
Reply | Threaded
Open this post in threaded view
|

Re: ClyMethodGroup>>#fileOut dead code ?

Denis Kudriashov
Hi Sven.

It is not dead. It is here to file out any method group including normal protocols.
It was copied from SystemOrganizer>>fileOutCategory:. Now it needs to be fixed in Pharo 7 and it would be nice to have it in better form

ср, 31 окт. 2018 г. в 15:39, Sven Van Caekenberghe <[hidden email]>:
Hi,

ClyMethodGroup>>#fileOut seems like dead code (it sends #trailer to a WriteStream which DNUs). It also uses the deprecated FileStream.

Can it be removed ?

Sven
Reply | Threaded
Open this post in threaded view
|

Re: ClyMethodGroup>>#fileOut dead code ?

EstebanLM
Can you add an issue into: 

https://github.com/pharo-ide/Calypso

Thanks!
Esteban


On 31 Oct 2018, at 19:57, Denis Kudriashov <[hidden email]> wrote:

Hi Sven.

It is not dead. It is here to file out any method group including normal protocols.
It was copied from SystemOrganizer>>fileOutCategory:. Now it needs to be fixed in Pharo 7 and it would be nice to have it in better form

ср, 31 окт. 2018 г. в 15:39, Sven Van Caekenberghe <[hidden email]>:
Hi,

ClyMethodGroup>>#fileOut seems like dead code (it sends #trailer to a WriteStream which DNUs). It also uses the deprecated FileStream.

Can it be removed ?

Sven