break in do while

[Top S. (topsoft)]

Hallo zusammen,

ich habe folgenden Code auf einem ATmega644P am laufen:

do{
    if( get_key_press(1<<KEY_UP)){                              // maximal 5 sekunden
      #ifdef debug
        softuart_puts("Taste hoch");
        softuart_putnewline();
      #endif
      break;
    }
    if(get_key_press(1<<KEY_DOWN)){                              // maximal 5 sekunden
      #ifdef debug 
        softuart_puts("Taste runter");
        softuart_putnewline();
      #endif
      break;
    }
  }while(timeout > 0);                                  // warte auf Tastendruck aber aber max 5sec

wenn ich die 'break' deaktiviere (//break;) wird beim betätigen der 
Tasten der entsprechende Text (Taste hoch, Taste runter) ausgegeben. 
Hardware ist also OK. Wenn aber die 'break' drin sind erfolgt keine 
Ausgabe mehr! Das 'break' soll zum verlassen der Timeoutschleife führen, 
geht aber nicht. :-(

Ich habe keine Ahnung warum ihr vieleicht?

Gruß Rene

[Stefan B. (stefan)]

Lass dir vom Compiler (Welcher?) Listings mit gemischtem C und ASM Code 
erzeugen und vergleiche. Bist du sicher, dass in beiden Quellcodes 
debug als wahr definiert ist? Zwischen den Compilerläufen auch mal 
make clean aufrufen, um alte Objektdateien zu löschen.

[Peter D. (peda)]

Dürfte ein volatile Problem sein.
Der GCC ist da sehr rabiat im Wegoptimieren, wenn die globale Variable 
(timeout) nicht volatile ist.


Peter

[JL (Gast)]

OK, nicht die erwartete Antwort aber:

warum dieses blöde break?
ist doch absolut unleserlich, da zu schnell übersehen bei der 
Fehlersuche

besser ist doch statt break: local_var_break = TRUE
und in der while ((timeout > 0) || (local_var_break != FALSE))

JL

[Mario (Gast)]

Warum nicht einfach die Variable Timeout auf 0 setzen, anstatt der break 
Funktion?

Gruß Mario