|
Datum:
|
Hallo Gemeinde Ich dachte, dass Berechnungen in C grundsätzlich als (signed) Integer durchgeführt werden. Nun reklamiert der GCC aber folgenden Code (Auschnitt):
#define TG_RESP_LEN 10 #define TG_RESP_IDX_ERRDETECT (TG_RESP_LEN - 1) #define TG_NO_MASK 0x0f static uint8 mBufRead[TG_RESP_LEN]; static uint32 mTelegramCnt = 0; if ((mBufRead[TG_RESP_IDX_ERRDETECT] & TG_NO_MASK) == (mTelegramCnt % 16)) // <- hier |
K:\source\sys\comValve.c 262 warning: comparison between signed and unsigned |
Wenn ich die linke Hälfte des Vergleiches auf (unsigned int) caste, ist der gcc zufrieden. Ich verstehe nicht, warum die linke Hälfte angeblich vom Typ int sein soll, während die rechte Hälfte unsigned int sei.
|
Datum:
|
StinkyWinky schrieb: > Berechnungen in C grundsätzlich als (signed) Integer nein. Wenn eine unsigned genommen wird, wird mit dieser auch in unsigned gerechnet, z.B. hier: mBufRead[TG_RESP_IDX_ERRDETECT] & TG_NO_MASK Das Feldelement ist uint_8, also wird das & auch ein unsigned ergeben. Aus dem % auf der rechten Seite dagegen kommt eine signed raus.
|
Datum:
|
Klaus Wachtler schrieb: > StinkyWinky schrieb: >> Berechnungen in C grundsätzlich als (signed) Integer > > nein. > Wenn eine unsigned genommen wird, wird mit dieser auch in unsigned > gerechnet, z.B. hier: mBufRead[TG_RESP_IDX_ERRDETECT] & TG_NO_MASK > Das Feldelement ist uint_8, also wird das & auch ein unsigned > ergeben. > > Aus dem % auf der rechten Seite dagegen kommt eine signed raus. Ich denke, da hast du dich jetzt verfranst. Die rechte Seite ist unsigned. Aber die linke. Die ist zunächst uint8_t wird aber für den Vergleich auf den nächst höheren Datentyp angehoben. Das geschieht aber so, dass es auf den Datentyp hochgecastet wird, der den Wertebereich abdecken kann. Ein signed int kann alle Werte eines unsigned char abdecken. Und damit wird die linke Seite zu einem int. (Es ist allerdings noch sehr früh und mein Kaffe läuft gerade erst durch die Maschine)
|
Datum:
|
Auch wahr. Dann lasse ich nur noch mein "nein" stehen, das stimmt jedenfalls :-)
|
Datum:
|
Aha. Dann mache ich wohl ein
if ((mBufRead[TG_RESP_IDX_ERRDETECT] & TG_NO_MASK) == (uint8)(mTelegramCnt % 16)) |
daraus, dann braucht links nichts (implizit) gecastet zu werden.
Thread beobachten
Seitenaufteilung abschalten