[FFmpeg-devel] [PATCH] s302menc: Add S302M encoder.

[Kieran Kunhya]

> +/* The number has to be a multiple of 192 */
> +#define S302M_NUMBER_SAMPLES 4800 /* Is 100ms of audio too big? */

It doesn't say anywhere in the spec as far as I can see that the
number has to be a multiple of 192.

There should be a private option to use the correct sample number
pattern - at the moment the muxer
is out of spec because 4800 is a totally arbitrary number. The number
of samples should match that
corresponding to the duration of the video frame (including using the
appropriate pattern if necessary).