EricssonResearch / openwebrtc-gst-plugins

OpenWebRTC specific GStreamer plugins
BSD 2-Clause "Simplified" License
51 stars 46 forks source link

Build error with clang #44

Open ossy-szeged opened 9 years ago

ossy-szeged commented 9 years ago

We build openwebrtc-gst-plugins in WebKit project with jhbuild and got the following build error:

In file included from gstvideorepair.c:30: In file included from ./gstvideorepair.h:29: In file included from /home/webkit/WebKit/WebKitBuild/DependenciesEFL/Root/include/gstreamer-1.0/gst/gst.h:88: /home/webkit/WebKit/WebKitBuild/DependenciesEFL/Root/include/gstreamer-1.0/gst/gstcompat.h:54:38: error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality] if (((((GstPad_)(pad))->direction) == GST_PAD_SRC))


/home/webkit/WebKit/WebKitBuild/DependenciesEFL/Root/include/gstreamer-1.0/gst/gstcompat.h:54:38: note: remove extraneous parentheses around the comparison to silence this warning
  if (((((GstPad_)(pad))->direction) == GST_PAD_SRC))
      ~                              ^             ~
/home/webkit/WebKit/WebKitBuild/DependenciesEFL/Root/include/gstreamer-1.0/gst/gstcompat.h:54:38: note: use '=' to turn this equality comparison into an assignment
  if (((((GstPad*)(pad))->direction) == GST_PAD_SRC))
                                     ^~
                                     =
1 error generated.

---

line54:   if (GST_PAD_IS_SRC (pad))

Of course this warning can be silenced in gstreamer with fixing this line, but 
"if GST_PAD_IS_SRC (pad)" would be very ugly. Additionally Wparentheses-equality
isn't a useful warning at all, it would be better to simply suppress it.

I'm going to suppress this warning in WebKit - https://bugs.webkit.org/show_bug.cgi?id=145837,
but it would be great if we can fix it in the upstream too.
ossy-szeged commented 9 years ago

45 wasn't accepted and I don't have time to find better fix, feel free to pick it up, if anybody else is interested in fixing this issue.