[FFmpeg-devel] [PATCH 0/8] Remove some warnings while building with ICC

Diego 'Flameeyes' Pettenò flameeyes
Thu Oct 2 16:38:36 CEST 2008


Since I've been testing with different compilers to make sure I don't
add regressions to the configure with the cflags/ldflags checks, I've
finally set up icc in a way that works, and built FFmpeg with that.

By default ICC produces a _huge_ amount of warnings, I've tried
removing some by changing the code a bit. In particular I find a bit
stupid forICC to declare itself as GCC 4.3.0 even though it does not
support the full set of attributes that GCC 4.1.0 supports.

There are still lots of warnings, especially regarding the enumerated
types mixed with other types, but solving some of them might end up
causing API/ABI breakage, so I'd rather not touch those just yet.

---

Diego 'Flameeyes' Petten? (8):
      Reduce ICC warnings regarding "enumerated type mixed with another type".
      Split away the "found" case in ff_h264_find_frame_end().
      Remove a double const.
      Avoid assignment inside if.
      Properly check for the last element of enc_multbl.
      Don't use the force_align_arg_pointer attribute with ICC.
      Don't use the cold attribute with ICC.
      Don't use the alloc_size attribute with ICC.


 libavcodec/audioconvert.c |    2 +-
 libavcodec/h264_parser.c  |   10 ++++++----
 libavcodec/wmaenc.c       |    3 ++-
 libavdevice/audio.c       |    2 +-
 libavformat/aiff.c        |    4 ++--
 libavformat/asf.c         |    3 ++-
 libavformat/au.c          |    3 ++-
 libavformat/mov.c         |    4 ++--
 libavformat/mpeg.c        |    4 +++-
 libavformat/mpegts.c      |    3 ++-
 libavformat/rtsp.c        |    3 ++-
 libavformat/segafilm.c    |    8 ++++----
 libavformat/sol.c         |    5 +++--
 libavformat/westwood.c    |    2 +-
 libavutil/aes.c           |    2 +-
 libavutil/common.h        |    2 +-
 libavutil/internal.h      |    2 +-
 libavutil/mem.h           |    2 +-
 18 files changed, 37 insertions(+), 27 deletions(-)




More information about the ffmpeg-devel mailing list