[Ffmpeg-devel] [PATCH] Reduce number of warnings in snow.c

Michael Niedermayer michaelni
Thu Jan 25 00:15:39 CET 2007


Hi

On Wed, Jan 24, 2007 at 03:10:32PM +0100, Panagiotis Issaris wrote:
> Hi,
> 
> The attached patch brings the number of warnings from 33 to 16 by:
> - explicitly using braces in some cases, namely when GCC suggests:
>   * suggest parentheses around + or - inside shift

ok


>   * suggest parentheses around && within ||

not ok, feel free to silence them with a command line argument if theres
one which doesnt silence anything important


> - using the const keyword ("initialization discards qualifiers from
> pointer target type")

ok


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070125/03eee957/attachment.pgp>



More information about the ffmpeg-devel mailing list