[Tarantool-patches] [PATCH 5/7] fiber: add dynamic option for parent backtrace

[Cyrill Gorcunov]

On Fri, Jul 09, 2021 at 12:50:40PM +0300, Egor Elchinov wrote:
> > > -#ifdef ENABLE_BACKTRACE
> > > +#if ENABLE_BACKTRACE
> > 
> > Please don't squash unrelated changes into the patch. If you prefer
> > to shange #ifdef to #if better make it as a separate commit (though
> > I don't understand yet what is wrong with existing code)
> Changes rebased into refactoring commit.
> 
> I believe this is more conventional way to check the ENABLE_BACKTRACE option
> because it is defined as
> #cmakedefine ENABLE_BACKTRACE 1
> but not
> #cmakedefine ENABLE_BACKTRACE

For this particular case we don't check the _value_ of defined variable,
we rather interested if it is defined at all. In general for conditional compilation
we use #ifdef you know, and all our current code uses #ifdef ENABLE_BACKTRACE
so I personally don't see much point in renaming statement. Still,
up to you, but if you prefer #if form could you please make it as a separate
patch.

	Cyrill