[squeak-dev] The Inbox: Tools-mt.856.mcz

Thiede, Christoph Christoph.Thiede at student.hpi.uni-potsdam.de
Mon Aug 12 17:35:26 UTC 2019


Hi Marcel,


> > 1. It's not really nice that CompiledMethod>>#isDeprecated is styled or even
> > detected as deprecated :)
>
> Well, for a single case, I did not want to add an extra check that is usually false anyway. Want it to be as fast as possible.

Good point :)

> Well, no. If those methods are missing their #deprcated explanation, take it as a chance to really deprecate those methods:
>
> (1) double-check that there are no senders
> (2) add "self deprecated:"
> (3) maybe rewrite them to use the new interface
>
> There is so much clean-up to do. ;-)

Just to be sure: You would like to decorate each single method of a class such as HtmlFileStream with the deprecated attribute?

Best,
Christoph
________________________________
Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel
Gesendet: Montag, 12. August 2019 19:19:23
An: John Pfersich via Squeak-dev
Betreff: Re: [squeak-dev] The Inbox: Tools-mt.856.mcz

Hi Christoph.

> 1. It's not really nice that CompiledMethod>>#isDeprecated is styled or even
> detected as deprecated :)

Well, for a single case, I did not want to add an extra check that is usually false anyway. Want it to be as fast as possible.

> 2. Would it be worth a thought to also style classes and all their methods
> as deprecated, which are in *Deprecated* categories? For example, when
> browsing through a MessageTrace, I think it could be helpful to see which
> methods are no longer relevant - for the same reason as Patrick mentioned.

Well, no. If those methods are missing their #deprcated explanation, take it as a chance to really deprecate those methods:

(1) double-check that there are no senders
(2) add "self deprecated:"
(3) maybe rewrite them to use the new interface

There is so much clean-up to do. ;-)

Best,
Marcel

Am 09.08.2019 23:24:01 schrieb Christoph Thiede <christoph.thiede at student.hpi.uni-potsdam.de>:

Hi, great idea! Two thoughts:

1. It's not really nice that CompiledMethod>>#isDeprecated is styled or even
detected as deprecated :)

2. Would it be worth a thought to also style classes and all their methods
as deprecated, which are in *Deprecated* categories? For example, when
browsing through a MessageTrace, I think it could be helpful to see which
methods are no longer relevant - for the same reason as Patrick mentioned.

Best,
Christoph



--
Sent from: http://forum.world.st/Squeak-Dev-f45488.html

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20190812/e4257bcf/attachment.html>


More information about the Squeak-dev mailing list