Re: [FFmpeg-devel] [PATCH 5/5] avformat/mov: rework ff_mov_read_chnl

2024-02-15 Thread Marton Balint
On Thu, 15 Feb 2024, Anton Khirnov wrote: Quoting Marton Balint (2024-02-12 22:15:37) - native order, and that a single channel only appears once was always assumed for less than 64 channels, obviously this was incorrect Could you add a test for the case where it's not true? Actually f

Re: [FFmpeg-devel] [PATCH 5/5] avformat/mov: rework ff_mov_read_chnl

2024-02-15 Thread Anton Khirnov
Quoting Marton Balint (2024-02-12 22:15:37) > - native order, and that a single channel only appears once was always assumed > for less than 64 channels, obviously this was incorrect Could you add a test for the case where it's not true? -- Anton Khirnov ___

[FFmpeg-devel] [PATCH 5/5] avformat/mov: rework ff_mov_read_chnl

2024-02-12 Thread Marton Balint
A lot of changes and fixes to channel layout parsing, notably - get rid of dynamic allocation of channel positions - signal unimplemented speaker positions as unknown instead of failure, but warn the user about it - native order, and that a single channel only appears once was always assumed fo