On 02 Apr 2014, at 01:11, Vittorio Giovara <[email protected]> wrote:
> On Tue, Mar 25, 2014 at 1:12 PM, wm4 <[email protected]> wrote: >> --- >> doc/APIchanges | 5 ++++ >> libavcodec/avcodec.h | 67 >> ---------------------------------------------------- >> libavutil/frame.c | 10 ++++++++ >> libavutil/frame.h | 30 +++++++++++++++++++++++ >> libavutil/pixfmt.h | 62 ++++++++++++++++++++++++++++++++++++++++++++++++ >> libavutil/version.h | 2 +- >> 6 files changed, 108 insertions(+), 68 deletions(-) >> > > So any further opinions on this patch? > I was waiting for the other patch that converts frames to pointer to > land but we could see if there are further comments on this. > Any preference for "colorspace" or "color_space"? The first is the > name in AVCodecContext the second would be a uniform name like the > newly introduced fields. What name does FFmpeg use? It would make it easier for downstreams if they match, and incompatibility for the sake of it seems pointless, IMO. Tim _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
