Makros für Bitmanipulation

.

  "Meine Vermutung ist, dass eventuell ein Type-Cast notwendig ist
  --> Kann es sein, dass Makros mit INT-Werten arbeiten?!?"

Macros "arbeiten" überhaupt nicht mit irgendwelchen Datentypen. Sie
sind nur ein Textersatzmechanismus des Präprozessors - Du solltest also
Deine Macros durch manuellen Textersatz auch selbst testen können.

#define GETBIT(ucByte, ucBit) ((ucByte & ucBit) > 0 ? 1 : 0)

Das würde ich (primär aus Gründen der Lesbarkeit) ändern. Lass diesen
if-operator ?: komplett weg, er bringt in dem Fall genau gar nichts, da
eh keine negativen Werte auftreten können.

Hintergrund: In C gilt die Regel, dass jede (ganze) Zahl implizit in
einen bool-Wert konvertiert werden kann (also wahr oder falsch,
"bool" selbst gibts ja erst bei C99). Dabei wird jede Zahl ungleich 0
als wahr aufgefasst; d.h. die Befehle while(1);, while(2); und
while(-100); bewirken genau das gleiche, nämlich jeweils eine
Endlosschleife.

Dein :?-Konstrukt ist in etwa vergleichbar mit folgender genialen
Konstruktion (die man leider manchmal immer noch sehen muss):
if(x)
  return true;
else
  return false;

...wer so etwas schreibt (das machst du ja zum Glück nicht) zeigt, dass
er if-Abfragen und Aussagen nicht wirklich verstanden hat.

Hi Chris,

Ja, ich verstehe, was du mit dem if-Beispiel meinst. Ich kann ja gleich
den Wert von x zurückgeben. Werde das GETBIT-Makro entsprechend
umbauen.

Gruß Ralf

#define setBit(x,y)    ((x) |=  (y))
#define clearBit(x,y)   ((x) &= ~(y))
#define toggleBit(x,y)   ((x) ^=  (y))

#define bitIsSet(x,y)   (((x) & (y))==(y))
#define bitIsClear(x,y) (((x) & (y))!=(y))


Klammern dienen der Sicherheit.
Übersetzt man mit -E als Parameter (GCC) kann man sich auch ansehen was
die Textverarbeitung daraus macht.
Vorsicht bei Sachen wie:

if(bitIsSet(a,b++))

daraus wird:

(((a) & (b++))==(b++))

und du hast einen tollen Bug :)

> #define bitIsSet(x,y)   (((x) & (y))==(y))

Auch du kannst dir hier das ==(y) sparen.

Für bitIsClear würd ich kein eigenes Makro schreiben, das schreibt man
viel schneller und genauso lesbar als if(!bitIsSet(...)).

>Auch du kannst dir hier das ==(y) sparen.

uups, stimmt :)

dadurch besteht die Gefahr von doppelt veränderten Variablen auch nicht
mehr (y++), danke für den Tip

>Auch du kannst dir hier das ==(y) sparen.

Stimmt doch nicht!
Ich verwende den Macro auch um mehre Bits auf einmal abzufragen.
z.B.
if(bitIsSet(PORT1,TASTE1 | TASTE2))

Dann stimmt der Name des Makros nicht. "bit" ist immer noch Singular.

Das Problem im Hintergrund ist...

in 99% aller Fälle findet man ein Makro das so aussieht

#define TRUE (1)
#define FALE (0)

Dabei ist aber erste Teil (das TRUE) eigentlich falsch ;-)

Es müsste eigentlich heissen

#define FALSE (0)
#define TRUE  (!FALSE)

denn alles was nicht explizit FALSE (sprich NULL) ist, ist TRUE.