C Fehler: Vergleich von Const mit bool immer falsch

[DraconiX (Gast)]

Ich habe hier einen kleinen Denkfehler oder irgendwas passt nicht:

#define SSD1306_LCDWIDTH    128
#define SSD1306_LCDHEIGHT   64

//...


void drawRect(uint8_t px1, uint8_t py1, uint8_t px2, uint8_t py2)
{
    if( ((px1 || px2) > SSD1306_LCDWIDTH-1) ||
        ((py1 || py2) > SSD1306_LCDHEIGHT-1) ) return;
//....
}

Wirft mir immer folgende Warnung aus:

warning: comparison of constant '127' with boolean expression is always false [-Wbool-compare]

Woran kann das liegen? Auch wenn ich die Werte direkt eintrage kommt 
diese Warnung.

[Pete K. (pete77)]

Weil (px1 || px2) ein boolescher Ausdruck ist.

Du willst:
if (px1 > (SSD1306_LCDWIDTH-1)) || (px2 > (SSD1306_LCDWIDTH-1)) || ...

[guest (Gast)]

"false > 127" ist immer false, und "true > 127" ist auch immer false, 
ergo der ganze Ausdruck ist immer false, steht doch aber auch genau so 
in der Compilermeldung.

[Tobi (Gast)]

(px1 || px2) liefert nur true oder false.
Wenn px1 0 ist und px2 0 ist kommt false raus ansonsten kommt immer 
true.
Versuch mal
[c]
void drawRect(uint8_t px1, uint8_t py1, uint8_t px2, uint8_t py2)
{
    if( (px1 > SSD1306_LCDWIDTH-1) ||
         (px2 > SSD1306_LCDWIDTH-1) ||
         (py1 > SSD1306_LCDHEIGHT-1)||
         (py2 > SSD1306_LCDHEIGHT-1) ) return;
//....
}
[\c]

[DraconiX (Gast)]

Ah super! Okay vielen Danke euch - ja logisch, ist ja schon ein 
Vergleich vor dem Vergleich.

:-D