[FFmpeg-devel] lavf/matroska*: add support for signed integers

[Jan Gerber]

On 11/15/2013 04:37 PM, Michael Niedermayer wrote:
>> +    int n = 0;
>> +
>> +    if (size > 8)
>> +        return AVERROR_INVALIDDATA;
>> +
>> +    *num = avio_r8(pb);
>> +    if (*num & 0x80) {
>> +        *num = -1;
>> +    } else {
>> +        *num = 0;
>> +    }
> 
> this discards the 7 less significant bits
> i think these contain bits from the number to be read

indeed.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavf-matroska-add-support-for-signed-integers.patch
Type: text/x-diff
Size: 2500 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131115/89c74124/attachment.bin>