[Ffmpeg-devel] [PATCH] Compilation fixes part 2

Michael Niedermayer michaelni
Fri Aug 26 01:09:57 CEST 2005


Hi

On Thu, Aug 25, 2005 at 09:36:09PM +0200, plaum at ipf.uni-stuttgart.de wrote:
> Hi again,
> 
> here's the second one. It fixes some gcc warnings
> about unhandled enum values in some switch statements.
> They simply add
> 
> default:
>     break;
> 
> at some places.

rejected, "fixing" warnings without checkig that the warnings arent pointing
to bugs is very bad
so either:
* use some gcc switch to suppress them or
* use colorgcc & apply http://www.mplayerhq.hu/~michael/colorgcc.diff
  so warnings with different severity will be shown in different colors or
* check each, explain why its ok and post a patch

[...]

-- 
Michael





More information about the ffmpeg-devel mailing list