On Thu, Feb 15, 2018 at 10:27:15AM +0100, Diego Biurrun wrote:
> On Wed, Feb 14, 2018 at 10:54:26PM +0000, Mark Thompson wrote:
> > --- /dev/null
> > +++ b/libavfilter/vaapi_vpp.c
> > @@ -0,0 +1,370 @@
> > +int ff_vaapi_vpp_colour_standard(enum AVColorSpace av_cs)
> > +{
> > + switch(av_cs) {
> > +#define CS(av, va) case AVCOL_SPC_ ## av: return VAProcColorStandard ## va;
> > + CS(BT709, BT709);
> > + CS(BT470BG, BT601);
> > + CS(SMPTE170M, SMPTE170M);
> > + CS(SMPTE240M, SMPTE240M);
> > +#undef CS
> > + default:
> > + return VAProcColorStandardNone;
> > + }
> > +}
>
> I don't think the #define eases readability, on the contrary.
I see this is copy+pasted from other files, oh well ..
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
