warning: left shift count >= width of type

[Peter (Gast)]

Hallo,

folgende Zeile erzeugt mir die Warnung "warning: left shift count >= 
width of type":

unsigned long int file_size = (i2c_get_byte(true) + (i2c_get_byte(true) << 8) + (i2c_get_byte(true) << 16)) + (i2c_get_byte(false) << 24);

Wie kann ich diese Warnung unterdrücken?

Der Code soll im Grunde einfach nur eine 4 Byte breite Zahl aus dem 
I2C-EEPROM lesen und entsprechend zusammensetzen.

lg, Peter

[Thomas B. (detritus)]

unsigned long int file_size = 
 (i2c_get_byte(true) +
 (i2c_get_byte(true) << 8) + 
 (unsigned long int)(i2c_get_byte(true) << 16)) +
 (unsigned long int)(i2c_get_byte(false) << 24);

Für den Compiler sind solche Konstrukte erstmal ints (hier wohl 16 bit 
lang). Will man mehr um mehr als 15 schieben, muss man auf einen 
grösseren Typ casten.

Edit: Ok, mein Nachfolger hat recht, man muss alles casten. Was ich 
geschrieben hab, gilt für konstante werte, die beim compilieren 
ausgerechnet werden.

[Sven P. (Gast)]

wenn i2c_get_byte() als

unsigned char i2c_get_byte(...);

vereinbart ist, wird erst mal mit Bytes gerechnet, und die sind nunmal 
acht Bit breit.
Wenn du jetzt

(i2c_get_byte(true) << 8)

machst, rechnen wir immer noch mit den 8-Bit breiten Bytes, aber du 
schiebst quasi mit "<<8" bis hin zur 16. Stelle nach links, und das geht 
beim Byte eben nicht.

Lösung: Zwing dem Compiler was Breiteres auf (Casting):

unsigned long int file_size =
  ((unsigned long int) i2c_get_byte(true) +
  ((unsigned long int) i2c_get_byte(true) << 8) +
  ((unsigned long int) i2c_get_byte(true) << 16)) +
  ((unsigned long int) i2c_get_byte(false) << 24);

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

Sven Pauli wrote:

> Wenn du jetzt
>

> (i2c_get_byte(true) << 8)
>

> machst, rechnen wir immer noch mit den 8-Bit breiten Bytes, ...

Nein, machen wir nicht.  Wir rechnen dann mit `int' (aber mit einem
vorzeichenbehafteten!), weil das die integer promotion rules der
Sprache C so verlangen.

[Peter (Gast)]

Wow, bin wieder einmal beeindruckt von den prompten Antworten hier :-)

Vielen Dank,
jetzt funktioniert alles!

lg, Peter

[Phillip H. (philharmony)]

Der Beitrag ist zwar schon älter aber ich habe genau das selbe Problem:
Ich habe es so gemacht wie oben beschrieben

unsigned long int val =
 ((unsigned long int) (*(var->value) << 16)) +
 ((unsigned long int) (*((var->value)+1)));

trotzdem bekomme ich die Warnung.
var->value ist int.
AVR-Studio, Optimierung auf -Os

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

Phillip Hommel schrieb:

>  ((unsigned long int) (*(var->value) << 16)) +

Stall zugesperrt als das Pferd schon ausgebüchst ist.

[Phillip H. (philharmony)]

Ah, du meinst daß das Bit-Schieben eine Klammer zu weit "innen" erfolgt?

[Stefan E. (sternst)]

Phillip Hommel schrieb:

> Ich habe es so gemacht wie oben beschrieben

Nein, hast du nicht. Du musst "*(var->value)" casten, nicht das Ergebnis 
vom Shift.

[Phillip H. (philharmony)]

Jo, hab den feinen Unterschieb bemerkt.
Danke ;)

[Joachim B. (jojo84)]

Holla zusammen!

Ich habe auch gerade das oben genannte Problem, aber in einer Abfrage:

if( x &= (1<<25) )
  {
  ;
  }

Erklärt sihc ja eigentlich von selbst: x ist ein long und ich will Bit 
25 abfragen.
Aber ich check nicht ganz, wo ich dann casten muß.
Würd mich freuen, wenn mich jemand fix aufklärt :)

VG

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

1L<<25

[Joachim B. (jojo84)]

@ A.K.:
OH!
Das ist ja toll! DANKE!

Kannst du mir vielleicht auch kurz erklären, warum das so ist? einfach 
eintippern ist zwar schön einfach, aber ich würd das gern verstehen :)

VG

[Stefan E. (sternst)]

Bist du sicher, dass du dort ein "&=" haben möchtest?

[Stefan E. (sternst)]

Joachim A. schrieb:

> Kannst du mir vielleicht auch kurz erklären, warum das so ist? einfach
> eintippern ist zwar schön einfach, aber ich würd das gern verstehen :)

Weil die 1 sonst nur ein int ist, also 16 Bit groß.

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

Joachim A. schrieb:

> Kannst du mir vielleicht auch kurz erklären, warum das so ist? einfach
> eintippern ist zwar schön einfach, aber ich würd das gern verstehen :)

1L ist (long)1

Wenn dir das auch nix nützt, dann solltest du mal einen Blick in ein C 
Handbuch riskieren.

[Joachim B. (jojo84)]

@Stefan:
Ups, da mit dem "&=" hab ich nicht aufgepasst, danke ;) ...
Und die Erklärung hat auch geholfen. Ich mach jetzt schon ne weile C, 
aber ich hab nie darüber nachgedacht, daß Konstante, wie die 1 da, auch 
int's sind...

Aber ok, wieder was gelernt :)