[FFmpeg-devel] [PATCH 0/4] User controllable padding

James Darnley james.darnley at gmail.com
Wed Jan 1 13:01:24 CET 2014


On 2013-12-30 20:16, James Almer wrote:
> Bump lavf minor in the first patch.

Oh yes, done.

> -1 for default value, 0 for no padding. Muxers can and should clip the value if needed.
> Ideally look up the corresponding limit for both flac and id3v2.

> You need to keep the default of 8192 if there's no user provided value.
> Same with the value in the id3v2 patch.

Why should there be a "default value" value?  Why would different
formats have different defaults for a feature intended to do the same
thing on them all?

I do agree that a muxer will need to clip to a valid range.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 683 bytes
Desc: OpenPGP digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140101/16907d42/attachment.asc>


More information about the ffmpeg-devel mailing list