[FFmpeg-devel] [PATCH] Proposal: undefine DEBUG_SEEK in libavformat/utils.c
Benoit Fouet
benoit.fouet
Tue Oct 16 21:48:44 CEST 2007
Michael Niedermayer wrote:
> Hi
>
> On Tue, Oct 16, 2007 at 11:26:01AM +0200, Benoit Fouet wrote:
>
>> Benoit Fouet wrote:
>>
>>> Michael Niedermayer wrote:
>>>
>>>
>>>> Hi
>>>>
>>>> On Mon, Oct 15, 2007 at 02:52:36PM +0200, Thorsten Jordan wrote:
>>>>
>>>>
>>>>
>>>>> Hallo,
>>>>>
>>>>> the attached patch does $subj. While test code is a good idea, it
>>>>> shouldn't be needed for production code. The define leads to intensive
>>>>> seeking in transport streams at line 1241ff if DEBUG_SEEK is defined.
>>>>>
>>>>>
>>>>>
>>>> patch ok
>>>>
>>>>
>>>>
>>>>
>>> Applied
>>>
>>>
>>>
>> this breaks seek regression test (i missed that) as follows, what should
>> be done ?
>>
>
> hmm, revert and wait for an explanation of the (poor) patch author
> its his patch he has to explain why that happens if he wants the patch
> applied :)
>
>
ok, done
--
Ben
More information about the ffmpeg-devel
mailing list