Shift bei uint32_t nur bis 16 Stellen?

[Stefan B. (Gast)]

Hallo!

Irgendwie habe ich ein Problem in einer Funktion bei mir und komme nicht 
auf den Fehler.

Ich habe eine Variable "data" vom Typ uint32_t und will mir von dieser 
das Bit an der Stelle "index" zurückgeben lassen.

Dafür habe ich mir folgende Funktion geschrieben:

uint8_t get_bit(uint32_t data, uint8_t index)
{
    // index may not be higher than 31 for 32 bit value
    if(index > 31)
        return 0;
    
    // extract the n-th bit by and-ing "data" with just one binary "1" in n-th bit
    // then the result is shifted right by n places to get the bit we want
    return (uint8_t)((uint32_t)(data & (uint32_t)(1 << index)) >> index);
}

Das funktioniert aber nur bis zum 16. Bit, danach kommt immer "0" 
zurück.

Warum ist das so, hab ich irgendwas übersehen?
Oder kann man das eleganter/schneller realisieren?

Das komplette Programm ist im Anhang.
Verwendet wird AVR Studio 4.18, µC ist ein ATtiny45.

Für Tipps wäre ich sehr dankbar, ich weiß echt nicht mehr weiter...

MfG Stefan

[holger (Gast)]

return (uint8_t)((uint32_t)(data & (uint32_t)((uint32_t)1 << index)) >> 
index);

[Robert (Gast)]

return (uint8_t)((data & (1L << 31)) >> index);

[Robert (Gast)]

besser noch:
return (data & (1L << index))?1:0;

Das spart das zurückshiften.

[Stefan E. (sternst)]

return (data >> index) & 1;

[Rolf Magnus (Gast)]

Zur Erklärung:

"1 << index" ist vom Typ int, da 1 vom Typ int ist. Dieser Typ ist auf 
dem AVR aber nur 16 Bit groß. Die ganze Casterei danach ändert daran 
nichts mehr.

Es sei noch angemerkt, daß die Funktion in manchen Fällen den Code 
deutlich langsamer machen kann.

Nur der Vollständigkeit halber, hier noch eine Variante: ;-)

return !!(data & (1UL << index));

[Stefan B. (Gast)]

Danke an alle, das wars!

Ist ja auch logisch dass "1" ein int und kein long ist, war wohl gestern 
schon zu spät für mich...
Und im Debugger konnte ich es mir auch nicht anzeigen lassen, 
anscheinend wurde meine extra dafür angelegte Variable wieder 
wegoptimiert.