[FFmpeg-devel] [PATCH] Make av_parse_color() support a "Random" color

Stefano Sabatini stefano.sabatini-lala
Sun May 10 21:37:21 CEST 2009


On date Sunday 2009-05-10 11:32:24 -0700, Baptiste Coudurier encoded:
> Hi,
> 
> Stefano Sabatini wrote:
> > Hi,
> > as recently suggested on ffmpeg-cvs.
> > 
> > Regards.
> > 
> > 
> > ------------------------------------------------------------------------
> > 
> > Index: ffmpeg/libavfilter/parseutils.c
> > ===================================================================
> > --- ffmpeg.orig/libavfilter/parseutils.c	2009-05-10 11:18:12.000000000 +0200
> > +++ ffmpeg/libavfilter/parseutils.c	2009-05-10 11:18:46.000000000 +0200
> > @@ -217,7 +217,7 @@
> >  
> >  int av_parse_color(uint8_t *rgba_color, const char *color_string, void *log_ctx)
> >  {
> > -    if (!strcmp(color_string, "bikeshed")) {
> > +    if (!strcmp(color_string, "Random") || !strcmp(color_string, "bikeshed")) {
> >          int rgba = ff_random_get_seed();
> >          rgba_color[0] = rgba >> 24;
> >          rgba_color[1] = rgba >> 16;
> 
> Why uppercase 'R' ?

Not a strong opinion on this, but all the other colors are uppercased
(so maybe also it should be "Bikeshed"), talking about bikesheds...

Regards.
-- 
FFmpeg = Foolish and Fundamental Magic Pitiless Esoteric Gorilla



More information about the ffmpeg-devel mailing list