[Pharo-dev] DeepCopier>>isItTimeToCheckVariables

Martin Dias tinchodias at gmail.com
Fri Oct 25 07:26:57 EDT 2013


Reported:
https://pharo.fogbugz.com/f/cases/12002/DeepCopier-isItTimeToCheckVariables-is-candidate-to-be-removed


On Fri, Oct 25, 2013 at 12:57 PM, Marcus Denker <marcus.denker at inria.fr>wrote:

>
>
>
> Yes, this can be cleanup up.
>
> On 25 Oct 2013 at 11:10:34, Martin Dias (tinchodias at gmail.com<//tinchodias at gmail.com>)
> wrote:
>
>  Hi,
>
> This method in DeepCopier has no senders (in Pharo 3) and it's the only
> who references the class variable NextVariableCheckTime.
>
> The time stamp says 1999... Unused code to remove?
>
>
> isItTimeToCheckVariables
>
> | now isIt |
> NextVariableCheckTime ifNil: [
> NextVariableCheckTime := Time totalSeconds.
> ^ true].
> now := Time totalSeconds.
> isIt := NextVariableCheckTime < now.
> isIt ifTrue: ["update time for next check"
> NextVariableCheckTime := now + self intervalForChecks].
> ^isIt
>
>
> Cheers,
> Martín
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.pharo.org/pipermail/pharo-dev_lists.pharo.org/attachments/20131025/5ea2c97e/attachment-0002.html>


More information about the Pharo-dev mailing list