[vlc-devel] [PATCH] video_chroma/scscale: use preprocessor instead of runtime check

Filip Roséen filip at atch.se
Wed Mar 15 15:30:35 CET 2017


Hi Rémi,

On 2017-03-15 16:06, Rémi Denis-Courmont wrote:

> > -#define ALLOW_YUVP (false)
> > + * (change it to 1 to try) */
> > +#define ALLOW_YUVP 0
> 
> I don' t really care either way. Nevertheless false is defined to 0 by 
> stdbool.h so this change should be useless.

Yes, I blame the deprecated part of my brain which sometimes get stuck
in "old systems and idiotic pre-processors"-land. See attached patch
for one that does not touch the `ALLOW_YUVP` definition.

Best Regards,\
Filip Roséen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170315/538e0c45/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-video_chroma-scscale-use-preprocessor-instead-of-run.patch
Type: text/x-diff
Size: 1437 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170315/538e0c45/attachment.patch>


More information about the vlc-devel mailing list