-Wstrict-overflow

[Bauform B. (bauformb)]

Guten Morgen!

Alte Programme provozieren neuerdings Warnungen. Dies ist kein 
unlösbares Problem, mir ist klar, wieso, weshalb, warum. Nur rein aus 
Neugier: wie geht ihr mit sowas um?

mkeproms.c: In function 'main':
mkeproms.c:118:10: warning: assuming signed overflow does not occur
when simplifying division [-Wstrict-overflow]
  118 |          top = ((en + 1) * 8192) / 256;

Das Programm baut in 27C64 EPROMs eine Checksum pro Chip ein, also für 
jeweils 8192 Byte. Von der Startadresse braucht man nur das High Byte, 
also wird durch 256 geteilt. Das kann man doch garnicht anders 
formulieren?!1!11

Das muss für maximal 5 EPROMs funktionieren, aber im Original kann der 
gcc das nicht wissen. Es hilft allerdings auch nicht, wenn ich ihm das 
sage:

if (ecnt <= 5) {
   chklist = 0x01F0;
   for (en = 0; en < ecnt; en++) {
      top = ((en + 1) * 8192) / 256;
      hi = (chksum[en] / 256) & 255;
      lo = chksum[en] & 255;
      target[chklist++] = top;
      target[chklist++] = hi;
      target[chklist++] = lo;
      printf ("%2.2X%2.2X\n", hi, lo);
   }
   target[i1] = 0;
}

[Oliver S. (oliverso)]

Eine checksum ist selten signed. Also rechne unsigned, und die Warnung 
ist weg.

Oliver

[Yalu X. (yalu) (Moderator)]

Generell ist man gut beraten, Integer-Divisionen und Right-Shifts nur
auf unsigned-Operanden anzuwenden.

Im konkreten Fall könntest du aber auch

  top = ((en + 1) * 8192) / 256;

durch

  top = (en + 1) * (8192 / 256);

ersetzen.

[Rolf M. (rmagnus)]

Bauform B. schrieb:
> Alte Programme provozieren neuerdings Warnungen. Dies ist kein
> unlösbares Problem, mir ist klar, wieso, weshalb, warum.

Dann hätte dir aber eigentlich klar sein müssen, dass die Datentypen der 
Variablen dafür essenziell sind. Du hast allerdings vergessen, uns die 
zu nennen.