[MPlayer-dev-eng] [PATCH] Compiler warnings almost gone :)
Dominik Mierzejewski
dominik at rangers.eu.org
Fri Jan 10 11:41:23 CET 2003
On Friday, 10 January 2003, Alex Beregszaszi wrote:
> Hi,
>
> > After the last few patches I'm almost done with eliminating compiler
> > warnings. :) Only a few worth mentioning remain. Two, in particular,
> > are probably easy to fix by the maintainers:
> >
> > pl_format.c:231: warning: suggest parentheses around + or - inside
> > shift vo_dga.c:580: warning: suggest parentheses around && within ||
> >
> > Is the attached patch a good fix?
>
> The vo_dga fix should be trivial, but the pl_format not.
> I think it should be (1<<31)-1 instead your version ( (1<<(31-1)) ).
Maybe so, but according to C standard - has higher priority than <<, so it
was executed in that order anyway. I don't pretend to understand this code,
so I don't know what is the right solution here.
> Anyway, do not commit it wihtout Anders answer.
Sure, just asking.
--
MPlayer RPMs maintainer: http://www.piorunek.pl/~dominik/linux/pkgs/mplayer/
"The Universe doesn't give you any points for doing things that are easy."
-- Sheridan to Garibaldi in Babylon 5:"The Geometry of Shadows"
More information about the MPlayer-dev-eng
mailing list