> > On Mon, Sep 11, 2017 at 11:15:42AM +0100, Frediano Ziglio wrote: > > Usually configuration macros are defined to 0 or undefined. > > For this reason these macros are sometimes checked using #if > > and sometimes with #ifndef/#ifdef. > > As this macro is always defined with 0 or 1 it makes no sense > > to check if defined or not so check the code to avoid this > > mistake. > > IIRC, I suggested a way not to make it so odd compared to other > preprocessor symbols. > > Christophe > Yes, but your suggestion did not work and you didn't send another improvement (unless I lost it). Frediano > > > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > > --- > > cfg.mk | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/cfg.mk b/cfg.mk > > index 93d7040c5..6bd3c55b3 100644 > > --- a/cfg.mk > > +++ b/cfg.mk > > @@ -111,6 +111,14 @@ sc_copyright_format: > > halt='spell Red Hat as two words' \ > > $(_sc_search_regexp) > > > > +# ENABLE_EXTRA_CHECKS is always defined, do not allow > > +# "#ifndef ENABLE_EXTRA_CHECKS" > > +sc_extra_checks: > > + @prohibit='#[[:space:]]*ifn?def[[:space:]]+ENABLE_EXTRA_CHECKS' \ > > + in_vc_files='\.[ch]$$' \ > > + halt='ENABLE_EXTRA_CHECKS is always defined' \ > > + $(_sc_search_regexp) > > + > > # We don't use this feature of maint.mk. > > prev_version_file = /dev/null > > _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel