[FFmpeg-devel] [FFmpeg-commits] add ff_index_search_timestamp and ff_add_index_entry

[Ronald S. Bultje]

Hi,

On Sun, Feb 6, 2011 at 2:07 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Sun, Feb 06, 2011 at 12:25:49PM -0500, Ronald S. Bultje wrote:
>> On Sun, Feb 6, 2011 at 12:08 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> > On Fri, Feb 04, 2011 at 03:51:19AM +0100, Peter Ross ?wrote:
>> >> Module: ffmpeg
>> >> Branch: master
>> >> Commit: e6fb5a4f78a67ed815e39ba8ac3893fd631b9b1a
>> >>
>> > [...]
>> >> Date: ? Sat Jan 22 20:03:22 2011 +1100
>> >>
>> >> add ff_index_search_timestamp and ff_add_index_entry
>> >>
>> >> Signed-off-by: Ronald S. Bultje <rsbultje at gmail.com>
>> >
>> > Its renaming 2 functions and adding wrapers not adding 2 new functions
>> > and the reason for that change is undocumented
>> >
>> > I havnt seen such poor log messages since a long time
>>
>> Index entries can then be filled outside the AVStream, which makes
>> generic search functions not use them. This was out for quite some
>> time, not sure how you could've missed the patchsets.
>
> I wonder how you could have missed the log message being bad

Yeah, I agree, isn't quite up to standards. Will do better in the future.

Ronald