On Mar 25, 2011, at 2:44 PM, Sean McGovern wrote: > Hi folks, > > I'd like to ask permission to go on warnings patrol based on the reports > submitted to FATE. Some targets have over 700+ now. The immediate aim being > only to squelch noisy warnings without changing code. > > If this is OK, is there any particular module you prefer I start in?
Start at the top and go down? > > I would only be focussing on versions of gcc still listed as supported on the > gcc.gnu.org homepage. Currently this includes: 4.3.5, 4.4.5, 4.5.2 as well as > 4.6 and trunk. 4.6 added these warnings: /misc/fate/work/satyr/src/libavcodec/get_bits.h:305:1: warning: variable 're_cache' set but not used [-Wunused-but-set-variable] which should be suppressed (only if they mention re_cache - the rest might be real code problems), and: /misc/fate/work/satyr/src/libavcodec/aacdec.c: In function 'aac_decode_frame_int': /misc/fate/work/satyr/src/libavcodec/aacdec.c:2179:54: warning: to be safe all intermediate pointers in cast from 'float **' to 'const float **' must be 'const' qualified [-Wcast-qual] which are probably a compiler bug that should be reported. IIRC I saw no way to fix them. > > I don't have access to ICC (unless it is available free for Solaris 10 or OS > X 10.6?) You can install ICC in Linux in VirtualBox for free. If you really like reading warnings, go through these: http://astrange.ithinksw.net/clang/ffmpeg/ but note that the chance of false positives is even higher. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
