On Fri, Feb 28, 2014 at 08:12:27PM +0200, Martin Storsjö wrote:
> --- a/tools/qt-faststart.c
> +++ b/tools/qt-faststart.c
> @@ -41,10 +41,10 @@
>  
> -#define BE_32(x) ((((uint8_t*)(x))[0] << 24) |  \
> -                  (((uint8_t*)(x))[1] << 16) |  \
> -                  (((uint8_t*)(x))[2] <<  8) |  \
> -                   ((uint8_t*)(x))[3])
> +#define BE_32(x) (((uint32_t)(((uint8_t*)(x))[0]) << 24) |  \
> +                             (((uint8_t*)(x))[1] << 16)  |  \
> +                             (((uint8_t*)(x))[2] <<  8)  |  \
> +                              ((uint8_t*)(x))[3])

nit: align the <<

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

Reply via email to