Skip to content

videoaggregator: Remove extra semicolon in macro usage

Nirbheek Chauhan requested to merge nirbheek/gstreamer:fix-Wpedantic-warnings into main

This is usually necessary to allow gst-indent to treat it as a statement, but we do not run gst-indent on headers and we do not have extra semicolons in other places that this macro is used in the header. Fixes warnings when using the header:

In file included from gstreamer/subprojects/gst-plugins-base/gst-libs/gst/video/video.h:185,
                 from XYZ:9001:
gstreamer/subprojects/gst-plugins-base/gst-libs/gst/video/gstvideoaggregator.h:206:78: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
  206 | G_DEFINE_AUTOPTR_CLEANUP_FUNC(GstVideoAggregatorConvertPad, gst_object_unref);
      |                                                                              ^
gstreamer/subprojects/gst-plugins-base/gst-libs/gst/video/gstvideoaggregator.h:214:181: warning: ISO C does not allow extra ‘;’ outside of a function [-Wpedantic]
  214 | G_DECLARE_DERIVABLE_TYPE (GstVideoAggregatorParallelConvertPad, gst_video_aggregator_parallel_convert_pad, GST, VIDEO_AGGREGATOR_PARALLEL_CONVERT_PAD, GstVideoAggregatorConvertPad);
      |                                                                                                                                                                                     ^

This was added in 8a5e5dde, does not need a backport.

Edited by Nirbheek Chauhan

Merge request reports