[FFmpeg-devel] [PATCH] lavu: add av_get_color_string() and use it in -colors option

[Paul B Mahol]

On 10/21/13, Stefano Sabatini <stefasab at gmail.com> wrote:
> 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)

Color can be given in other colorspaces. But looking that this function use
is quite limited I'm not motivated to think about it more.

> ?
> --
> FFmpeg = Forgiving & Fundamentalist Marvellous Political Elitarian Gadget
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>