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

[Thierry Foucu]

On Tue, Apr 10, 2012 at 3:40 PM, Kieran Kunhya <kierank at ob-encoder.com>wrote:

> > +/* 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).
>

Thanks for the comment..

I will be adding a private option to set the video fps and compute the
number of sample per frame.

I will work on a new patch this week.



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>