verstehe diese Compiler-Warnung nicht

[brechbunkt (Gast)]

Hallo,

ich habe den nachfolgenden array aus strings und möchte den jeweiligen 
Tag ausgeben können, was doch eigentlich kein Problem sein sollte:

char *weekDay[] = {"ERR", "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"};

print_dbg("Heute ist ");
print_dbg(weekDay[1]);

Soweit funktioniert das ganze auch, aber ich bekomme komischer weise 
diese Warnung:

Warning      array subscript has type 'char'

Ich kann mir allerdings nicht erklären, was der Compiler hier zu meckern 
hat. Kann mir jemand einen Tipp vielleicht geben?

[Dirk (Gast)]

Der Compiler interpretiert bei dir die "1" als signed char, was zu 
unsinnigen negativen Indizes (>127) führen kann.

[(prx) A. K. (prx)]

Und weshalb sollte der Compiler die lexikalische Konstante 1 als signed 
char betrachten? Per Sprachdefinition von C ist die vom Typ int.

[Peter II (Gast)]

Dirk schrieb:
> Der Compiler interpretiert bei dir die "1" als signed char, was zu
> unsinnigen negativen Indizes (>127) führen kann.

auch negative Indizes sind gültig.

[(prx) A. K. (prx)]

Nicht bei Arrays, nur bei Pointern, und das Überlaufverhalten von 
vorzeichenbehafteten Datentypen ist nicht definiert, also nicht per se 
als negativ zu sehen.

Diese in meinen Augen etwas fragwürdige Warning scheint tatsächlich Teil 
von GCC zu sein. Weshalb sie hier auslösen sollte ist jedoch unklar. 
Aber vielleicht ist der hier gezeigte Code ja nicht der eigentliche 
Code, nur "so ähnlich". Oder der relevante Teil fehlt.

[Peter II (Gast)]

A. K. schrieb:
> Diese in meinen Augen etwas fragwürdige Warning scheint tatsächlich Teil
> von GCC zu sein.

im Test tritt es aber nicht auf:

#include <stdio.h>

char *weekDay[] = {"ERR", "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"};

int main()
{
   printf("Heute ist ");
   printf(weekDay[1]);

   return 0;
}

wie ist denn print_dbg definiert?

[(prx) A. K. (prx)]

Peter II schrieb:
> im Test tritt es aber nicht auf:

Kann ja auch nicht, weil weit und breit kein Index vom Typ char zu sehen 
ist. So aber schon, und da kriegt man die Warnung dann auch (-Wall):

int main()
{
   char c = 1;

   printf("Heute ist ");
   printf(weekDay[c]);

   return 0;
}

Ich schrieb ja schon, dann brechbunkt hier möglicherweise den 
eigentlichen Code zwecks Demonstration stark vereinfacht und damit die 
Ursache gleich mit beseitigt hatte.

[brechbunkt (Gast)]

Ja stimmt, ich habe den Code vereinfachen wollen. Normaler weise schaut 
er so aus:

print_dbg(weekDay[ tm->tm_wday ]);

Der Wert tm_wday ist ein 8 Bit Wert von einem RTC.

Wenn ich die Zeile allerdings (dank eurer Hilfe) so abändere, ist alles 
gut:

print_dbg(weekDay[(int)(tm->tm_wday)]);

[....bitte ausfüllen.... (Gast)]

brechbunkt schrieb:
> Der Wert tm_wday ist ein 8 Bit Wert von einem RTC.

Und der ist von Typ Char!?!

[Stefan E. (sternst)]

brechbunkt schrieb:
> Wenn ich die Zeile allerdings (dank eurer Hilfe) so abändere, ist alles
> gut:

Besser wäre es allerdings, tm_wday von vornherein einen sinnvollen 
8-Bit-Typ zu geben, nämlich "unsigned char" oder "uint8_t".

[(prx) A. K. (prx)]

Stefan Ernst schrieb:
> Besser wäre es allerdings, tm_wday von vornherein einen sinnvollen
> 8-Bit-Typ zu geben, nämlich "unsigned char" oder "uint8_t".

Mit dieser Idee kommst du rund 40 Jahre zu spät. Allerdings ist der 
Unterschied bei Werten zwischen 0 und 6 nicht wirklich relevant.

Zumal für mich nicht nachvollziehbar ist, wieso negative ints gut sind 
und negative chars schlecht.

[tumb (Gast)]

Stefan Ernst schrieb:
> Besser wäre es allerdings, tm_wday von vornherein einen sinnvollen
> 8-Bit-Typ zu geben, nämlich "unsigned char" oder "uint8_t".

Ganz und gar nicht. Der Index muss int sein, kein char - dann 
klappts auch ohne Warnung bzw. Laufzeit-Fehler.

[(prx) A. K. (prx)]

Warum muss er int sein? Damit GCC die Schnauze hält? Er muss grad gross 
genug sein, um die Werte 0-6 darstellen zu können.

[Jörg W. (dl8dtl) (Moderator)]

A. K. schrieb:
> Stefan Ernst schrieb:
>> Besser wäre es allerdings, tm_wday von vornherein einen sinnvollen
>> 8-Bit-Typ zu geben, nämlich "unsigned char" oder "uint8_t".
>
> Mit dieser Idee kommst du rund 40 Jahre zu spät.

Das Feld tm_wday der struct tm gemäß C-Standard ist vom Typ
"int".

Unser TE wird also wohl irgendwie sein eigenes Süppchen kochen.

> Zumal für mich nicht nachvollziehbar ist, wieso negative ints gut sind
> und negative chars schlecht.

Ich denke nicht, dass es um negative chars schlechthin geht, aber
um mögliche Probleme bei der sign extension zu `int'.  Bei

char c = 240;

   ...
   return array[c];

könnte das nämlich zu unliebsamen Überraschungen führen, wenn das
default `char' vorzeichenbehaftet ist, und davor zu warnen ist
die Sache m. E. Wert.

[Stefan E. (sternst)]

A. K. schrieb:
> Stefan Ernst schrieb:
>> Besser wäre es allerdings, tm_wday von vornherein einen sinnvollen
>> 8-Bit-Typ zu geben, nämlich "unsigned char" oder "uint8_t".
>
> Für diese Idee kommst du rund 40 Jahre zu spät.

Falls du auf die struct tm der time.h hinaus willst, dort ist tm_wday 
ein int, kein char. Er verwendet also irgendwas eigenes oder 
modifiziertes, und es spricht nichts dagegen, es zu ändern.

[(prx) A. K. (prx)]

Jörg Wunsch schrieb:
> signed char c = 240;

Da liegt das Hase im Pfeffer. Denn eigentlich müsste er an dieser 
Stelle warnen. Das aber traut sich ohne -pedantic nicht einmal GCC, 
weils sonst an Warnungen nur so hagelt.

[(prx) A. K. (prx)]

Stefan Ernst schrieb:
> Falls du auf die struct tm der time.h hinaus willst, dort ist tm_wday
> ein int, kein char.

Und int kann nicht negativ werden?

[Stefan E. (sternst)]

tumb schrieb:
> Stefan Ernst schrieb:
>> Besser wäre es allerdings, tm_wday von vornherein einen sinnvollen
>> 8-Bit-Typ zu geben, nämlich "unsigned char" oder "uint8_t".
>
> Ganz und gar nicht. Der Index muss int sein, kein char - dann
> klappts auch ohne Warnung bzw. Laufzeit-Fehler.

Unsinn. Ein "unsigned char", "signed char", "int8_t" oder "uint8_t" 
würde auch keine Warnung bringen.

[Stefan E. (sternst)]

A. K. schrieb:
> Stefan Ernst schrieb:
>> Falls du auf die struct tm der time.h hinaus willst, dort ist tm_wday
>> ein int, kein char.
>
> Und int kann nicht negativ werden?

Es geht bei dieser Warnung nicht darum, dass der Index negativ sein 
kann. Es geht darum, dass bei char die Signedness nicht spezifiziert 
ist.

[Jörg W. (dl8dtl) (Moderator)]

A. K. schrieb:
> Jörg Wunsch schrieb:
>> signed char c = 240;
>
> Da liegt das Hase im Pfeffer. Denn eigentlich müsste er an *dieser*
> Stelle warnen.

Das "signed" hatte ich dann noch rausgeworfen.  Wenn man das
hinschreibt, nimmt der GCC es als Hinweis, dass man sich was dabei
gedacht hat ;), und warnt nicht.  Die Warnung kommt wirklich nur für
ein "unqualifiziertes" `char', und sie soll offenbar ausschließlich
darauf hinweisen, dass bei der Benutzung eines solchen als Array-Index
eine implizite Konvertierung nach `int' erfolgt mit all ihren
potenziellen Tücken.

[....bitte ausfüllen.... (Gast)]

A. K. schrieb:
> Warum muss er int sein? Damit GCC die Schnauze hält?
Es ist nie gut, irgendwelche Warnungen im Code zu haben. Ein Programm 
sollte immer ohne jegliche Warnungen compilieren. Alles andere ist 
schlechter Stil.