It's redundant with spice_warn_if_fail(), and can even be confusing. Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx> --- common/log.h | 6 ------ 1 file changed, 6 deletions(-) diff --git a/common/log.h b/common/log.h index a56105a..0e03f59 100644 --- a/common/log.h +++ b/common/log.h @@ -95,12 +95,6 @@ void spice_log(const char *log_domain, } \ } G_STMT_END -#define spice_warn_if(x) G_STMT_START { \ - if G_UNLIKELY(x) { \ - spice_warning("condition `%s' reached", #x); \ - } \ -} G_STMT_END - #define spice_assert(x) G_STMT_START { \ if G_LIKELY(x) { } else { \ spice_error("assertion `%s' failed", #x); \ -- 2.5.0 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel