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.den...@inria.fr>wrote: > > > > Yes, this can be cleanup up. > > On 25 Oct 2013 at 11:10:34, Martin Dias > (tinchod...@gmail.com<//tinchod...@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 > >
