Re: [Boost-build] [PATCH] -ftemplate-depth

From: Jurko Gospodnetić (jurko.gospodnetic_at_[hidden])
Date: 2008-07-20 06:25:55

   Hi Ilya.

   Thanks for the feedback!

>>   Anyone else has any suggestions on this patch?
> 
> I think it is better to append -ftemplate-depth-N to the OPTIONS 
> variable. Or duplicate it in compile.c++.pch and compile.c.pch actions.

   I'll look into that now. However, the current patch was implemented 
so it does not change the current compiler options. And I do not think 
any patches that get committed before the Boost 1.36.0 library release 
will be changing that.

>> Perhaps placing the feature definition in a different module or 
>> renaming it to gcc.template-depth? I though about placing it in the 
>> tools/builtin.jam module but the feature is currently used only in the 
>> gcc toolset so I decided against it.
> 
> qcc also have "template-depth".

   Hehe, right. :-) I guess gcc.jam and qcc.jam looked too similar when 
I grepped for template-depth. :-)))

   OK, I'll move the feature definition to somewhere more general, 
however it seems qcc and gcc use different default values for this 
feature so I'll have to be careful not to mess this up.

   Thanks again for the feedback!

   Best regards,
     Jurko Gospodnetić