avalon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Donald <dona...@apache.org>
Subject Re: Changes to TimeTrigger.
Date Sat, 17 Mar 2001 08:19:30 GMT
At 11:27  16/3/01 -0800, Ram C wrote:
>---- Peter Donald <donaldp@apache.org> wrote:
>> At 10:40  15/3/01 -0800, Ram C wrote:
>> >That was exactly the problem...
>> 
>> ahhh - I see ;)
>> 
>> >In the original code, the 'offset' is lost because the m_startTime
>> was
>> >either the current time or (current time + offset). The
PeriodicTimeTrigger
>> >had a time sensitive information (current time) within itself.
>> >
>> >Whenever a trigger is reset, i have to reset this time sensitive
information
>> >in PeriodicTimeTrigger.
>> >
>> >I had two choice.....
>> >1) Update the original trigger to somehow reset this time sensitive
>> information.
>> 
>> I like this approach - if I get what you mean. Do you mean add a method
>> "void reset();" to Trigger interface? If so I think that would be much
>> better as later on when when we have triggers that are not tied to
>> time (ie
>> fire trigger when a certain system event occurs) this will work well.
>> What
>> do you think ?
>> 
>
>I got this to work with the reset()...the code is much simpler now...no
>clonning crap ..(what was i thinking....must be on drugs...:-)
>
>I'll commit these changes along with the 'syncronized' BinaryHeap fix.

excellent !
;)

Cheers,

Pete

*-----------------------------------------------------*
| "Faced with the choice between changing one's mind, |
| and proving that there is no need to do so - almost |
| everyone gets busy on the proof."                   |
|              - John Kenneth Galbraith               |
*-----------------------------------------------------*


---------------------------------------------------------------------
To unsubscribe, e-mail: avalon-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: avalon-dev-help@jakarta.apache.org


Mime
View raw message