[FFmpeg-devel] [PATCH] lavfi/video: remove duplicated function ff_null_start_frame_keep_ref()

Stefano Sabatini stefasab at gmail.com
Thu Aug 2 12:49:40 CEST 2012


On date Thursday 2012-08-02 12:06:31 +0200, Nicolas George encoded:
> Le sextidi 16 thermidor, an CCXX, Stefano Sabatini a écrit :
> > Semantic for the function ff_null_start_frame() was changed in
> > 07bad27810cd, and it has now the same behavior of
> > ff_null_start_frame_keep_ref(), thus it makes no sense to keep both of
> > them.
> 

> I find the name "null_start_frame" (either with or without keep_ref) very
> deceptive, but that is unrelated. So LGTM.

Same for me, but I prefer to avoid too many changes with qatar when
possible.

Patch applied, thanks for the review.
-- 
FFmpeg = Funny & Fancy Moronic Practical Enhancing God


More information about the ffmpeg-devel mailing list