Reported: https://pharo.fogbugz.com/f/cases/12002/DeepCopier-isItTimeToCheckVariables-... On Fri, Oct 25, 2013 at 12:57 PM, Marcus Denker <marcus.denker@inria.fr>wrote:
Yes, this can be cleanup up.
On 25 Oct 2013 at 11:10:34, Martin Dias (tinchodias@gmail.com<//tinchodias@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