[svn-commits] mmichelson: branch 1.4 r90059 - /branches/1.4/main/pbx.c

[SVN commits to the Digium repositories]

Author: mmichelson
Date: Wed Nov 28 16:08:50 2007
New Revision: 90059

URL: http://svn.digium.com/view/asterisk?view=rev&rev=90059
Log:
Removing some seemingly pointless code. This sets a channel variable for every priority
executed in the dialplan if you have debug set to anything non-zero. This seems pointless
due to the fact that these channel variables are not referenced anywhere else in the code and
their names are esoteric enough that they would not be practical to reference in the dialplan. Plus
the fact that this behavior isn't documented anywhere means that the change is not likely to cause
any disruption. If anything, this may actually cause a slight performance increase if running with
debug on.

The motivating influence for this code change is the eventwhencalled option for queues. If set to
vars, all channel variables will be output to the manager. These unnecessary channel variables make
the output a lot more difficult to deal with.


Modified:
    branches/1.4/main/pbx.c

Modified: branches/1.4/main/pbx.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/main/pbx.c?view=diff&rev=90059&r1=90058&r2=90059
==============================================================================
--- branches/1.4/main/pbx.c (original)
+++ branches/1.4/main/pbx.c Wed Nov 28 16:08:50 2007
@@ -1825,13 +1825,7 @@
 			c->priority = priority;
 			pbx_substitute_variables(passdata, sizeof(passdata), c, e);
 			if (option_debug) {
-				char atmp[80];
-				char atmp2[EXT_DATA_SIZE+100];
 				ast_log(LOG_DEBUG, "Launching '%s'\n", app->name);
-				snprintf(atmp, sizeof(atmp), "STACK-%s-%s-%d", context, exten, priority);
-				snprintf(atmp2, sizeof(atmp2), "%s(\"%s\", \"%s\") %s",
-					app->name, c->name, passdata, "in new stack");
-				pbx_builtin_setvar_helper(c, atmp, atmp2);
 			}
 			if (option_verbose > 2) {
 				char tmp[80], tmp2[80], tmp3[EXT_DATA_SIZE];