Setzen des höchstwertigen Vorzeichen-Bits (Lint-Warnung)

Question

Ich arbeite an einer Funktion, wo ich ein 3-Byte-Array konvertieren und das 3-Byte-Array als einzelnes int32 zurückgeben muss. Der Code ist wie folgt: pbData ist ein Zeiger auf das Byte-Array.

const byte bMSBitPosNeg = 0x80; 
const byte bMSBNeg = 0xFF; 
int32 i32Num = 0; 

//Big - Endian 
if (pbData != NULL) 
{ 
    if ((pbData[0] & bMSBitPosNeg) == bMSBitPosNeg) //Negative 
    { 
     i32Num |= (bMSBNeg * 0x1000000); //Force MSB to 0xFF as 3 bytes are 
             //converted to 4 bytes 
    } 

    i32Num |= (pbData[0] << 16); 
    i32Num |= (pbData[1] << 8); 
    i32Num |= (pbData[2]); 

} 
return i32Num; 

Input: {0x00,0xBB, 0xA3 } Output: 48035 // positive Zahl

Input: {0xFF, 0x44,0x5d } Output: -48.035 // negative Zahl

Der Code funktioniert wie erwartet, aber ich bekomme eine Flusenwarnung. Warnung 648: Überlauf in Rechenkonstante für den Betrieb: 'Multiplikation'

Ich brauche die Funktionalität, aber nicht die Warnung.Wie kann ich es unterdrücken?

Answer 1

Sie können Multiplikation mit Bitverschiebung ersetzen oder sogar das Bitmuster manuell für eine bessere Lesbarkeit berechnen:

const int32 bMSBNeg = (int32)0xFF000000; 

Sie können auch all diese Berechnungen vermeiden, indem die Anzahl in den oberen drei Bytes konstruieren, und dann verschieben es durch 8 Bits mit der Division nach unten:

if (pbData != NULL) { 
    i32Num = ((pbData[0] << 24) | (pbData[1] << 16) | (pbData[2] << 8))/256; 
} 

Anmerkung: 8 durch Verschiebung statt durch 256 dividiert nach Implementierung definierte Verhalten auf den Standard wären.