[FFmpeg-devel] [PATCH] fix playback of partial DV files

[Baptiste Coudurier]

Hi Reimar,

Reimar D?ffinger wrote:
> On Mon, Feb 09, 2009 at 10:48:20PM +0100, Reimar D?ffinger wrote:
>> On Mon, Feb 09, 2009 at 01:24:23PM -0800, Baptiste Coudurier wrote:
>>> You could avoid the seek by filling c->buf directly, would this be hard ?
>> No, just didn't see it. I attached an updated version, I admit I did not
>> test it, compilation is too slow on this PC.
>>
>>> Do you see a need in changing the test in the while loop like in the
>>> probe function ?
>> I don't think it matters much.
>> But if you use
>> (state & 0xffffff7f) != 0x1f07003f
>> you can use just "unsigned" instead of uint32_t.
>> A tiny bit faster and smaller code (since many architectures might be
>> able to use instructions with immediates) might be
>> uint32_t state;
>> if ((state | 0x80) != 0x1f0700bf)
>>
>> I do think it is preferable to use the same variant everywhere.
> 
> Ping, what should be done about these patches?

I'd say apply them, they should work, and won't cause much harm.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org