[MPlayer-dev-eng] [PATCH] Optionally pad with silence

[Reimar Döffinger]

On 13 Mar 2011, at 23:57, Tobias Diedrich <ranma at tdiedrich.de> wrote:
> Reimar Döffinger wrote:
>> On Wed, Mar 09, 2011 at 07:52:13PM +0100, Tobias Diedrich wrote:
>>> Optionally pad with silence.
>>> 
>>> Allow silence padding for audiostreams that begin after or end before the
>>> video stream.
>> 
>> I don't see how this patch would add padding for the "begin after" case.
>> In addition I think it might/will actually pad more than the length of
>> the video, so I expect that if you play two videos directly one after the
>> other where audio and video are exactly the same length this patch will
>> actually introduce a short pause in-between them.
>> However I can imagine that it will be useful so I'm ok with it if the
>> description is fixed.
> 
> I'll look into it, I'd like to have it work as advertised before I
> commit it. :)

I'd be in favour of doing that by changing the "advertising" part, files where audio and video do not start a the same time are problematic in many ways so it seems fine to me to ignore them initially and fix it later if it's simple enough.