[libvirt PATCH v2] src: workaround warning triggered in glib 2.69
Peter Krempa
pkrempa at redhat.com
Fri Jul 23 13:38:14 UTC 2021
On Fri, Jul 23, 2021 at 14:19:25 +0100, Daniel P. Berrangé wrote:
> Libvirt is using the G_GNUC_FALLTHROUGH macro provided by glib since
> version 2.60. Since we need to support older glib, we also have some
> compatibility code to define it if missing.
>
> We set the GLIB_VERSION_MAX_ALLOWED macro to ensure we get warnings
> when we use an API that dates from a glib version newer than our
> minimum benchmark. Historically this didn't get enforced for (most)
> macros, but GLib 2.69 has addressed that gap.
>
> This causes our usage of G_GNUC_FALLTHROUGH to trigger warnings.
> GLib is right to warn, because it does not know that we have added
> our own fallback for older versions.
>
> The only way to squelch this warning though, is to fully undefine
> the GLib provided G_GNUC_FALLTHROUGH and use our own in its place.
> We'll be able to remove all this compat burden when we finally
> update the min glib version to be >= 2.60
>
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
> src/internal.h | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
Reviewed-by: Peter Krempa <pkrempa at redhat.com>
More information about the libvir-list
mailing list