[MPlayer-dev-eng] [PATCH] Huge batch of compiler warning fixes
Jindrich Makovicka
makovick at kmlinux.fjfi.cvut.cz
Mon Nov 4 22:35:02 CET 2002
Dominik Mierzejewski wrote:
>On Monday, 04 November 2002, Arpi wrote:
>
>
>>Hi,
>>
>>
>>
>>>A couple of patches, actually:
>>>First one adds -Wall to CFLAGS in places they were missing, so that we
>>>can catch all warnings.
>>>
>>>
>>_WHY_ ?
>>
>>
>
>Why? To annoy developers into cleaning up their code, for example. :-) I
>thought we're almost done with killing all warnings after my previous
>patch, but then I noticed that not every Makefile sets -Wall, so I added
>it. And boom! Few lines of warnings turned into few hundred. We shouldn't
>turn a blind eye on that.
>
>
Yes, nowadays there are so many warnings that they lose their meaning
because you simply have to ignore them. I tried to check all the
warnings generated by --enable-debug twice, and in both cases I found at
least one real bug (well, no surprise when the 2> output was about
100Kbytes).
--
Jindrich Makovicka
More information about the MPlayer-dev-eng
mailing list