On Tue, Nov 18, 2014 at 8:56 AM, Luca Barbato <[email protected]> wrote:
> On 18/11/14 00:57, Kieran Kunhya wrote: > >> I added ff_combine_packet since it reduces the number of lines > >> and makes the code slightly less verbose. > > > > This is very confusing since ff_combine_packet suggests an AVPacket is > involved. > > It will and it is not less confusing that using "frame" since it is a > frame-worth-packet what you get out of this machinery anyway. > > But since you are introducing new API, an argument like "the old API also was confusing, so this one might as well be" is really not something you should be using. ;) Dispense with the confusion in the new API! _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
