On Thu, Mar 05, 2015 at 01:59:21PM +0200, Martin Storsjö wrote:
> --- a/libavcodec/libopenh264enc.c
> +++ b/libavcodec/libopenh264enc.c
> @@ -39,6 +39,8 @@ typedef struct SVCContext {
>  
> +#define OPENH264_VER(maj, min) ((OPENH264_MAJOR > (maj)) || (OPENH264_MAJOR 
> == (maj) && OPENH264_MINOR >= (min)))

I think the name is slightly misleading OPENH264_VER_MIN or
OPENH264_VER_AT_LEAST (like we have for gcc) would better
express what this does.

Otherwise LGTM

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to