Re: [Pharo-dev] DeepCopier>>isItTimeToCheckVariables

2013-10-25 Thread Martin Dias
xD yes, DeepCopier, our old friend! but it's not about replacing by Fuel or something like that. It was just a random pick for "fix a triviality a day", what Marcus proposed some weeks ago. On Fri, Oct 25, 2013 at 3:56 PM, Mariano Martinez Peck < marianop...@gmail.com> wrote: > Martin wa

Re: [Pharo-dev] DeepCopier>>isItTimeToCheckVariables

2013-10-25 Thread Mariano Martinez Peck
Martin watching the DeepCopier??? I am afraid to ask why ;) On Fri, Oct 25, 2013 at 6:10 AM, Martin Dias 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... Unuse

Re: [Pharo-dev] DeepCopier>>isItTimeToCheckVariables

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

Re: [Pharo-dev] DeepCopier>>isItTimeToCheckVariables

2013-10-25 Thread Marcus Denker
Yes, this can be cleanup up. On 25 Oct 2013 at 11:10:34, Martin Dias (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? isItTim

[Pharo-dev] DeepCopier>>isItTimeToCheckVariables

2013-10-25 Thread Martin Dias
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 totalSecon