[FFmpeg-devel] [PATCH 4/5] src_buffer: implement av_buffersrc_add_frame.
Nicolas George
nicolas.george at normalesup.org
Thu May 3 20:23:08 CEST 2012
Le quintidi 15 floréal, an CCXX, Clément Bœsch a écrit :
> I know it's already approved, but this can be simplified with a
> avfilter_get_buffer_ref_from_frame() (see comment on patch 1/5).
See my other message.
> Also another commit to rename picref into ref or bufref would be welcome I
> guess.
There are inconsistencies and bad naming all over the place, because the
different codes that were merged had different conventions. Stefano pointed
it out too.
In the first round, I kept as much original code as possible, but sure,
patches to make all this consistent would be useful once everything is
settled.
> PS: sorry if these comments have already been raised, I don't remember
> them so just in case...
No problem.
Regards,
--
Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120503/6a8051df/attachment.asc>
More information about the ffmpeg-devel
mailing list