[FFmpeg-devel] [PATCH] lavu: add av_get_color_string() and use it in -colors option
Stefano Sabatini
stefasab at gmail.com
Mon Oct 21 10:44:34 CEST 2013
On date Monday 2013-10-21 07:20:55 +0000, Paul B Mahol encoded:
> On 10/20/13, Stefano Sabatini <stefasab at gmail.com> wrote:
> > On date Thursday 2013-10-17 19:06:39 +0200, Stefano Sabatini encoded:
> >> On date Tuesday 2013-10-15 11:58:44 +0200, Michael Niedermayer encoded:
> >> > On Sat, Mar 12, 2011 at 01:36:05PM +0100, Stefano Sabatini wrote:
> >> > > ---
> >> > > cmdutils.c | 13 +++++++++++++
> >> > > cmdutils.h | 6 ++++++
> >> > > cmdutils_common_opts.h | 1 +
> >> > > doc/fftools-common-opts.texi | 3 +++
> >> > > libavutil/parseutils.c | 18 ++++++++++++++++++
> >> > > libavutil/parseutils.h | 14 ++++++++++++++
> >> > > 6 files changed, 55 insertions(+), 0 deletions(-)
> >>
> >> New patches, provide a more flexible approach.
> >
> > Ping.
>
> I already commented, but perhaps you like to ignore my comments.
"It's ugly" is hardly a useful comment.
So, what about:
const char *av_get_color_name(const char **bufp, const uint16_t **rgbp, int color_idx)
?
--
FFmpeg = Forgiving & Fundamentalist Marvellous Political Elitarian Gadget
More information about the ffmpeg-devel
mailing list