[FFmpeg-devel] [PATCH 0/8] User controllable padding v3
michaelni at gmx.at
Fri Jan 24 23:17:26 CET 2014
On Fri, Jan 24, 2014 at 07:50:47PM +0100, James Darnley wrote:
> I'm not sure if anyone saw the last time I posted these patches so I
> will repeat the changelog I sent last time and reword it for clarity.
> I followed some of the suggestions from first time I posted this. I
> made the maximum value INT_MAX and the default 1024. I added the
> accessors. I've updated the fate tests. The changes are file sizes and
> position in files, both of which agree with the changes I made. I also
> caught a typo in a comment.
> For this third round I found that the avi muxer adds some padding, so I
> let the new option control it. This change resulted in the many changes
> to fate results because it now add 1024 bytes of padding rather than
> 1016. To silence this I could change the default to silence these
> changes. The 1024 bytes is a somewhat arbitrary value anyway.
> A last question. Should I split up the fate patch and merge it into the
> patches which cause the changes?
patchset applied without changing any defaults
If you still want to change the defaults, that requires testing
with various players and a reason why.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel