[asterisk-dev] [asterisk-commits] tilghman: trunk r246789 - in /trunk: apps/ build_tools/ channels/ main/

[Kevin P. Fleming]

SVN commits to the Asterisk project wrote:

> -		<member name="DEBUG_THREADLOCALS" displayname="Enable Thread-Local-Storage Debugging">
> +		<member name="DEBUG_THREADLOCALS" displayname="Enable Thread-Local-Storage Debugging" touch_on_change="include/asterisk/strings.h">

This change does not excite me, to say the least. This now means that
this list must be kept up to date if DEBUG_THREADLOCALS is used in any
other place in the code, and if a community developer (or user) adds in
their own modules but doesn't realize the module is not listed here,
they could get unexpected behavior.

If the goal of this change is to make rebuilds faster after changing
these flags, for modules where the flag change doesn't change the code,
the ccache tool is the correct solution for that problem. It works every
time, and it does not require any declarative or advance knowledge of
which code is affected and which are not.

-- 
Kevin P. Fleming
Digium, Inc. | Director of Software Technologies
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
skype: kpfleming | jabber: kpfleming at digium.com
Check us out at www.digium.com & www.asterisk.org