Depecrated method in GS

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

Depecrated method in GS

BrunoBB
Hi,

I'm doing some refactoring in my app and some methods are deprecated. When see the implementation of #deprecated: my curiosity exploded.
Why the deprecated implementation is like this ?
deprecated: anExplanationString
        "Warn that the sending method has been deprecated."

        false ifTrue: [ Transcript cr; show: 'Deprecated protocol: ' , anExplanationString ].

false ifTrue: [] - it will never be executed. Or is for development purpose and at some point put true instead of false to log something in the transcript (if this is the case why not comment/uncomment the line).

Regards,
Bruno
Reply | Threaded
Open this post in threaded view
|

Re: Depecrated method in GS

GLASS mailing list
Bruno,

That's a good question. Take a look at the programmers guide for out
explanation for dealing with deprecated methods[1].

While we take care to not use deprecated methods in our product code, we
do not like to break customer applications so we have yet to actually
remove the deprecated methods from the system, sooner or later we will
go through that process so it is worth making a sweep.

Since GLASS/GsDevKit runs across multiple versions of GemStone, I have
leveraged the use of deprecated methods in the code base and have not
made a sweep, because the work of using the new method is nowhere near
the cost of the repackaging needed to continue using the methods in
older versions:)

Dale

[1]
https://downloads.gemtalksystems.com/docs/GemStone64/3.2.x/GS64-ProgGuide-3.2/15-ClassProtocol.htm#pgfId-57346

On 09/16/2015 10:11 AM, BrunoBB via Glass wrote:

> Hi,
>
> I'm doing some refactoring in my app and some methods are deprecated. When
> see the implementation of #deprecated: my curiosity exploded.
> Why the deprecated implementation is like this ?
> deprecated: anExplanationString
> "Warn that the sending method has been deprecated."
>
> false ifTrue: [ Transcript cr; show: 'Deprecated protocol: ' ,
> anExplanationString ].
>
> false ifTrue: [] - it will never be executed. Or is for development purpose
> and at some point put true instead of false to log something in the
> transcript (if this is the case why not comment/uncomment the line).
>
> Regards,
> Bruno
>
>
>
> --
> View this message in context: http://forum.world.st/Depecrated-method-in-GS-tp4850613.html
> Sent from the GLASS mailing list archive at Nabble.com.
> _______________________________________________
> Glass mailing list
> [hidden email]
> http://lists.gemtalksystems.com/mailman/listinfo/glass

_______________________________________________
Glass mailing list
[hidden email]
http://lists.gemtalksystems.com/mailman/listinfo/glass