Ooops, a follow-up. My choice of name: 'cal_isr' is poor. What I meant was 'cal_interrupt_source', I was wanting to allow the user to choose whether to disable all interrupt sources, or a specific timer interrupt, etc.
William On Feb 25, 7:26 am, William <[email protected]> wrote: > Greetings, > > While reviewing the library, it looks like there may be a problem in > this section: > > while seconds > 59 loop -- if seconds < 60, nor harm is done > seconds = seconds - 60 > minutes = minutes + 1 > end loop > > The interrupt routine that increments 'seconds', could occur at any > time, which means we could lose time, or worse. > > I've thought about a modification like so: > > while seconds > 59 loop -- if seconds < 60, nor harm is done > if (defined(cal_isr) == true) then > cal_isr = off > end if > seconds = seconds - 60 > if (defined(cal_isr) == true) then > cal_isr = on > end if > minutes = minutes + 1 > end loop > > What do you think? > > William -- You received this message because you are subscribed to the Google Groups "jallib" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/jallib?hl=en.
