On Sat, May 9, 2009 at 2:16 PM, Eric Covener <[email protected]> wrote: > On Fri, May 8, 2009 at 10:13 AM, <[email protected]> wrote: > >> + >> + /* if Includes was enabled without exec in the new config, but >> + * was enabled with exec in the base, then disable exec in the >> + * resulting options. */ >> + if ((base->opts & OPT_INC_WITH_EXEC) >> + && (new->opts & OPT_INC_WITH_EXEC) == 0) { >> + conf->opts &= ~OPT_INC_WITH_EXEC; > > > The above is wrapped in > if (new->opts & OPT_UNSET) { > > Which means checking new->opts against anything is probably not needed
Not likely the case due to test failures removing this block. -- Eric Covener [email protected]
