On Thu, 2016-10-27 at 18:41 +0200, Francois Gouget wrote:
> Deprecation warnings no longer cause compilation errors.
Since when?
> Also they should either be fixed or ignored with
> G_GNUC_{BEGIN,END}_IGNORE_DEPRECATIONS.
So this patch is re-enabling the deprecated functions so that we will
notice and fix these warnings? I'd kind of like a bit more explanation
in the commit log about the motifvation for this patch.
>
> Signed-off-by: Francois Gouget <fgou...@codeweavers.com>
> ---
> src/Makefile.am | 1 -
> src/spice-util.h | 12 +++---------
> 2 files changed, 3 insertions(+), 10 deletions(-)
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 78953dd..3f81866 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -77,7 +77,6 @@ KEYMAP_GEN = $(srcdir)/keymap-gen.pl
> SPICE_COMMON_CPPFLAGS =
> \
> -DSPICE_COMPILATION \
> -DG_LOG_DOMAIN=\"GSpice\" \
> - -DSPICE_NO_DEPRECATED
> \
> -DSPICE_GTK_LOCALEDIR=\"${SPICE_GTK_LOCALEDIR}\" \
> -DPNP_IDS=\""$(PNP_IDS)"\" \
> -DUSB_IDS=\""$(USB_IDS)"\" \
> diff --git a/src/spice-util.h b/src/spice-util.h
> index 88e3a57..a01da0c 100644
> --- a/src/spice-util.h
> +++ b/src/spice-util.h
> @@ -41,22 +41,16 @@ gchar* spice_uuid_to_string(const guint8
> uuid[16]);
> #define SPICE_RESERVED_PADDING (10 * sizeof(void*))
>
> /* need to be in a public header */
> -#ifndef SPICE_GNUC_DEPRECATED_FOR
> +#ifndef SPICE_DEPRECATED_FOR
> #if __GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__ >= 5)
> -#define SPICE_GNUC_DEPRECATED_FOR(f) \
> +#define SPICE_DEPRECATED_FOR(f) \
> __attribute__((deprecated("Use " #f " instead")))
> #else
> -#define SPICE_GNUC_DEPRECATED_FOR(f) G_GNUC_DEPRECATED
> +#define SPICE_DEPRECATED_FOR(f) G_GNUC_DEPRECATED
> #endif /* __GNUC__ */
> #endif
>
> -#ifndef SPICE_NO_DEPRECATED
> -#define SPICE_DEPRECATED_FOR(f) SPICE_GNUC_DEPRECATED_FOR(f)
> #define SPICE_DEPRECATED G_GNUC_DEPRECATED
> -#else
> -#define SPICE_DEPRECATED_FOR(f)
> -#define SPICE_DEPRECATED
> -#endif
>
> G_END_DECLS
>
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel
