[FFmpeg-devel] [PATCH] lavu: add av_get_color_string() and use it in -colors option
Paul B Mahol
onemda at gmail.com
Mon Oct 21 10:54:37 CEST 2013
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
>
More information about the ffmpeg-devel
mailing list