Часы, немного переделал и стали отставать Опции

[_Diman_]

Раньше в прерывании было

Код

ISR(SIG_OVERFLOW2)
{
clock_++;
if (clock_==128)
    {                                                          
    clock_=0;
    hour[h_sek]++;
      }
}

Функция часов

Код

if (hour[h_sek]==60)
   {
       hour[h_sek] =0;
    hour[h_min]++;
….

Сделал

Код

ISR(SIG_OVERFLOW2)
{
clock_++;
if (clock_==128)
    {                                                          
    clock_=0;
    isr_1sek_=1;
      }
}

Код

void clock(void){
cli();    
  if ( isr_1sek_ )//прошла секунда
    {
       isr_1sek_=0;
    sei();
    if (++hour[h_sek]==60)
        {
        hour[h_sek] =0;
        hour[h_min]++;
        if (hour[h_min]==60)
….
sei();

Часы начали отставать. Не могу понять почему? Ведь событие происходит раз в секунду, пропустить его ни как нельзя, даже если программа зависнет на пол секунды, чего нет на самом деле. Вот, чешу репу, чего я не так сделал?

[rezident]

_Diman_, я бы на вашем месте инкремент переменной делал после проверки условия на выход ее за границу допустимых значений, а не проверку условия после инкремента.

Код

if (hour[h_sec]<59)
  hour[h_sec]+=1;
else
  { hour[h_sec]=0;
    if (hour[h_min]<59)
      hour[h_min]+=1;
    esle
    { hour[h_min]=0;

Кроме этого переменная isr_1sek_ должна быть типа volatile.
Ну и вдобавок, если у вас структура времени, то может стоить оформить ее как структуру, а не простой массив значений.
P.S. исходник не смотрел
P.P.S. смущает строчка

Код

if (++hour[h_sek]==60)

. Я не настолько большой знаток стандартов Си, чтобы понять, что по стандарту раньше делается после индексирования и извлечения значения, инкремент значения или сравнение?

[_Diman_]

Я думаю тут не из-за этого, но можно и так.

Код

if (++hour[h_sek]>=60)

Инкремент после был бы:

Код

if (hour[h_sek]++==60)

На счет volatile, я смотрю листинг. Привык к WinAVR-20060421. Там обычно не прокатывает вот такое

Код

ISR(){
clock_--;
}

clock_=60;
while(clock_)

В функции часов, есть одно место "не очень" коррекция хода в час,

Код

if (++hour[h_min]==60)
{        
//cli();
if ( setup.corr_flag&BIT(fl_corr_znak_pl) )//корекция часов
    clock_+=setup.correction;//для убыстрения
    else
    clock_= 0x00-setup.correction;
//sei();

Но это раньше работало, здесь я тоже особых проблем не вижу.
Ладно, не буду голову забивать, буду разбираться в симуляторе.