[FFmpeg-devel] [PATCH 1/2] lavfi: merge asrc_abuffer into vsrc_buffer.

Stefano Sabatini stefasab at gmail.com
Sat Apr 28 00:26:16 CEST 2012


On date Saturday 2012-04-28 00:13:24 +0200, Nicolas George encoded:
> Le nonidi 9 floréal, an CCXX, Stefano Sabatini a écrit :
> > Nit: config_output_video -> vsrc_config_output, consistent with
> > sink_buffer.c. Same below.
> 
> Consistency is good, but I find more logical and readable to have similar
> functions sharing a prefix and distinguished by a suffix instead of the
> reverse. For example, the functions will be near to each other in a sorted
> list.
> 
> > You can save a few KBs of unused binary code in case of disable by
> > reorganizing the function order, putting under #ifdef those which
> > are not shared (see sink_buffer.c).
> 
> Are you sure this is necessary? The compilers I know will print a warning
> about an unused function (which is not a problem in an unusual
> configuration) and drop the unused code.

Feel free to push it, still I feel it's better to avoid warnings when
they can be avoided.
 
> Anyway, the next patches move that code to share more of it, and I believe
> asrc_abuffer should become part of the public API just like vsrc_buffer is
> already.

Isn't it already public (looking at installed headers)?
-- 
FFmpeg = Fantastic Fundamental Mastodontic Portentous Eretic Gladiator


More information about the ffmpeg-devel mailing list