Re: [FFmpeg-devel] [PATCH 3/9] avformat/s337m: Consider container bit resolution

2020-01-13 Thread Tomas Härdin
mån 2020-01-13 klockan 14:02 + skrev Gaullier Nicolas: > > > +if (container_word_bits && (container_word_bits+7)/8 != > > > + (word_bits+7)/8) { > > > > Can it happen that word_bits is anything but 16 or 24 with a valid > > stream? If not then I'd check container_word_bits == word_bits

Re: [FFmpeg-devel] [PATCH 3/9] avformat/s337m: Consider container bit resolution

2020-01-13 Thread Gaullier Nicolas
>> +if (container_word_bits && (container_word_bits+7)/8 != >> + (word_bits+7)/8) { > >Can it happen that word_bits is anything but 16 or 24 with a valid stream? If >not then I'd check container_word_bits == word_bits && (word_bits == 16 || >word_bits == 24) or so word_bits may be 20, and

Re: [FFmpeg-devel] [PATCH 3/9] avformat/s337m: Consider container bit resolution

2020-01-12 Thread Tomas Härdin
fre 2020-01-03 klockan 16:56 +0100 skrev Nicolas Gaullier: > Prepare the support of s337m in muxers other than raw (ex: wav). > For example, this forbids reading 16 bits DolbyE stream from a 24 bit wav > file. > --- > libavformat/s337m.c | 20 ++-- > libavformat/s337m.h | 3 ++-

[FFmpeg-devel] [PATCH 3/9] avformat/s337m: Consider container bit resolution

2020-01-03 Thread Nicolas Gaullier
Prepare the support of s337m in muxers other than raw (ex: wav). For example, this forbids reading 16 bits DolbyE stream from a 24 bit wav file. --- libavformat/s337m.c | 20 ++-- libavformat/s337m.h | 3 ++- 2 files changed, 16 insertions(+), 7 deletions(-) diff --git