Hi again, will it help if we explicitely cast the (-1) to char in the two occasions where it is used for sbr_present_flag?
- Fabian
diff --git a/libfaad/mp4.c b/libfaad/mp4.c index 72b2af6..14e607a 100644 --- a/libfaad/mp4.c +++ b/libfaad/mp4.c @@ -174,7 +174,7 @@ int8_t AudioSpecificConfigFromBitfile(bitfile *ld, #endif #ifdef SBR_DEC - mp4ASC->sbr_present_flag = -1; + mp4ASC->sbr_present_flag = (char)-1; if (mp4ASC->objectTypeIndex == 5) { uint8_t tmp; @@ -276,7 +276,7 @@ int8_t AudioSpecificConfigFromBitfile(bitfile *ld, /* no SBR signalled, this could mean either implicit signalling or no SBR in this file */ /* MPEG specification states: assume SBR on files with samplerate <= 24000 Hz */ - if (mp4ASC->sbr_present_flag == -1) + if (mp4ASC->sbr_present_flag == (char)-1) { if (mp4ASC->samplingFrequency <= 24000) {
signature.asc
Description: This is a digitally signed message part