On 2013-09-10 10:09, Luca Barbato wrote:
On 10/09/13 09:00, Diego Biurrun wrote:
On Mon, Sep 09, 2013 at 05:32:30PM +0200, Luca Barbato wrote:
--- a/configure
+++ b/configure
@@ -2432,6 +2432,18 @@ pgi_flags(){
      done
  }

+cparser_flags(){
+    for flag; do
+        case $flag in
+            -Wno-switch)             echo -Wno-switch-enum ;;
+            -Wno-format-zero-length) ;;
+            -Wdisabled-optimization) ;;
+            -Wno-pointer-sign)       echo -Wno-other ;;
+            *)                       echo $flag ;;
+        esac
+    done
+}
+
  suncc_flags(){
      for flag; do
          case $flag in

This was previously in alphabetical order.

configure:ccc_flags(){
configure:msvc_common_flags(){
configure:msvc_flags(){
configure:icl_flags(){
configure:pgi_flags(){
configure:cparser_flags(){
configure:suncc_flags(){
configure:tms470_flags(){

I guess msvc broke it before me.

No.  icl and msvc share msvc_common_flags, that's why icl is after msvc.

@@ -2660,6 +2672,14 @@ probe_cc(){
          if [ $pfx = hostcc ]; then
              append _cflags -Dsnprintf=_snprintf
          fi
+    elif $_cc --version 2>/dev/null | grep -q ^cparser; then
+        _type=cparser
+        _ident=$($_cc --version | head -n1)
+        _depflags='-MMD'
+        _cflags_speed='-O4'
+        _cflags_size='-O2'
+        _flags='-Wno-missing-variable-declarations -Wno-empty-statement'

What do these warning flags do?

Shut down some debatable warnings such as warn for AVFormat
ff_demuxer_foo = {} not having a an extern AVFormat ff_demuxer somewhere
or av_dlog being empty normally.

Both would need some discussion to convince cparser author that should
not be on by default.

Move these to around line 4000 where the other compiler warning flags are handled.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to