On Tue, Sep 12, 2017 at 06:56:34AM -0400, Frediano Ziglio wrote: > > > > Currently, ENABLE_EXTRA_CHECK is defined to be 0 or 1, and has to be > > defined this way, otherwise this will break display-channel.c > > compilation. > > This is different from most AC_DEFINE() preprocessor constants which are > > usually either set to 1, or unset. > > > > This commit switches ENABLE_EXTRA_CHECK setting in configure.ac to the > > usual way, and defines a second constant to 0/1 depending on whether > > it's set or not in display-channel.c. > > > > Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > > This patch is simply not working, the code is never enabled Yeah, changed the symbol to the wrong name ;) s/HAVE_EXTRA_CHECKS/ENABLE_EXTRA_CHECKS/ Christophe _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel