[asterisk-commits] seanbright: trunk r136888 - /trunk/channels/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Sat Aug 9 07:53:58 CDT 2008


Author: seanbright
Date: Sat Aug  9 07:53:57 2008
New Revision: 136888

URL: http://svn.digium.com/view/asterisk?view=rev&rev=136888
Log:
Merge more changes from the resolve-shadow-warnings branch (henceforth known
as RSW since i am too lazy to keep typing it all out).  This time a few of
the channels.

Modified:
    trunk/channels/chan_agent.c
    trunk/channels/chan_alsa.c
    trunk/channels/chan_console.c
    trunk/channels/chan_features.c
    trunk/channels/chan_jingle.c
    trunk/channels/chan_phone.c

Modified: trunk/channels/chan_agent.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_agent.c?view=diff&rev=136888&r1=136887&r2=136888
==============================================================================
--- trunk/channels/chan_agent.c (original)
+++ trunk/channels/chan_agent.c Sat Aug  9 07:53:57 2008
@@ -268,7 +268,7 @@
 {
 	int res, i;
 	struct agent_pvt *p;
-	char basename[AST_CHANNEL_NAME], *tmp;
+	char base[AST_CHANNEL_NAME], *tmp;
 	const char *device;
 	enum ast_device_state state;
 
@@ -297,11 +297,11 @@
 	AST_LIST_TRAVERSE(&agents, p, list) {
 		ast_mutex_lock(&p->lock);
 		if (p->chan) {
-			ast_copy_string(basename, p->chan->name, sizeof(basename));
-			if ((tmp = strrchr(basename, '-'))) {
+			ast_copy_string(base, p->chan->name, sizeof(base));
+			if ((tmp = strrchr(base, '-'))) {
 				*tmp = '\0';
 			}
-			if (strcasecmp(p->chan->name, device) == 0 || strcasecmp(basename, device) == 0) {
+			if (strcasecmp(p->chan->name, device) == 0 || strcasecmp(base, device) == 0) {
 				p->inherited_devicestate = state;
 				ast_devstate_changed(state, "Agent/%s", p->agent);
 			}
@@ -1363,7 +1363,7 @@
 	int groupoff;
 	int waitforagent=0;
 	int hasagent = 0;
-	struct timeval tv;
+	struct timeval now;
 
 	s = data;
 	if ((s[0] == '@') && (sscanf(s + 1, "%d", &groupoff) == 1)) {
@@ -1382,8 +1382,8 @@
 		    ast_strlen_zero(p->loginchan)) {
 			if (p->chan)
 				hasagent++;
-			tv = ast_tvnow();
-			if (!p->lastdisc.tv_sec || (tv.tv_sec >= p->lastdisc.tv_sec)) {
+			now = ast_tvnow();
+			if (!p->lastdisc.tv_sec || (now.tv_sec >= p->lastdisc.tv_sec)) {
 				p->lastdisc = ast_tv(0, 0);
 				/* Agent must be registered, but not have any active call, and not be in a waiting state */
 				if (!p->owner && p->chan) {
@@ -1404,11 +1404,11 @@
 			if (!p->pending && ((groupmatch && (p->group & groupmatch)) || !strcmp(data, p->agent))) {
 				if (p->chan || !ast_strlen_zero(p->loginchan))
 					hasagent++;
-				tv = ast_tvnow();
+				now = ast_tvnow();
 #if 0
-				ast_log(LOG_NOTICE, "Time now: %ld, Time of lastdisc: %ld\n", tv.tv_sec, p->lastdisc.tv_sec);
+				ast_log(LOG_NOTICE, "Time now: %ld, Time of lastdisc: %ld\n", now.tv_sec, p->lastdisc.tv_sec);
 #endif
-				if (!p->lastdisc.tv_sec || (tv.tv_sec >= p->lastdisc.tv_sec)) {
+				if (!p->lastdisc.tv_sec || (now.tv_sec >= p->lastdisc.tv_sec)) {
 					p->lastdisc = ast_tv(0, 0);
 					/* Agent must be registered, but not have any active call, and not be in a waiting state */
 					if (!p->owner && p->chan) {
@@ -1726,7 +1726,7 @@
 	char username[AST_MAX_BUF];
 	char location[AST_MAX_BUF] = "";
 	char talkingto[AST_MAX_BUF] = "";
-	char moh[AST_MAX_BUF];
+	char music[AST_MAX_BUF];
 	int count_agents = 0;		/*!< Number of agents configured */
 	int online_agents = 0;		/*!< Number of online agents */
 	int offline_agents = 0;		/*!< Number of offline agents */
@@ -1780,9 +1780,9 @@
 				offline_agents++;
 			}
 			if (!ast_strlen_zero(p->moh))
-				snprintf(moh, sizeof(moh), " (musiconhold is '%s')", p->moh);
+				snprintf(music, sizeof(music), " (musiconhold is '%s')", p->moh);
 			ast_cli(a->fd, "%-12.12s %s%s%s%s\n", p->agent, 
-				username, location, talkingto, moh);
+				username, location, talkingto, music);
 			count_agents++;
 		}
 		ast_mutex_unlock(&p->lock);
@@ -1804,7 +1804,7 @@
 	char username[AST_MAX_BUF];
 	char location[AST_MAX_BUF] = "";
 	char talkingto[AST_MAX_BUF] = "";
-	char moh[AST_MAX_BUF];
+	char music[AST_MAX_BUF];
 	int count_agents = 0;           /* Number of agents configured */
 	int online_agents = 0;          /* Number of online agents */
 	int agent_status = 0;           /* 0 means offline, 1 means online */
@@ -1848,9 +1848,9 @@
 				strncat(location, " (Confirmed)", sizeof(location) - strlen(location) - 1);
 		}
 		if (!ast_strlen_zero(p->moh))
-			snprintf(moh, sizeof(moh), " (musiconhold is '%s')", p->moh);
+			snprintf(music, sizeof(music), " (musiconhold is '%s')", p->moh);
 		if (agent_status)
-			ast_cli(a->fd, "%-12.12s %s%s%s%s\n", p->agent, username, location, talkingto, moh);
+			ast_cli(a->fd, "%-12.12s %s%s%s%s\n", p->agent, username, location, talkingto, music);
 		count_agents++;
 		ast_mutex_unlock(&p->lock);
 	}

Modified: trunk/channels/chan_alsa.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_alsa.c?view=diff&rev=136888&r1=136887&r2=136888
==============================================================================
--- trunk/channels/chan_alsa.c (original)
+++ trunk/channels/chan_alsa.c Sat Aug  9 07:53:57 2008
@@ -560,12 +560,12 @@
 	return tmp;
 }
 
-static struct ast_channel *alsa_request(const char *type, int format, void *data, int *cause)
-{
-	int oldformat = format;
+static struct ast_channel *alsa_request(const char *type, int fmt, void *data, int *cause)
+{
+	int oldformat = fmt;
 	struct ast_channel *tmp = NULL;
 
-	if (!(format &= AST_FORMAT_SLINEAR)) {
+	if (!(fmt &= AST_FORMAT_SLINEAR)) {
 		ast_log(LOG_NOTICE, "Asked to get a channel of format '%d'\n", oldformat);
 		return NULL;
 	}

Modified: trunk/channels/chan_console.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_console.c?view=diff&rev=136888&r1=136887&r2=136888
==============================================================================
--- trunk/channels/chan_console.c (original)
+++ trunk/channels/chan_console.c Sat Aug  9 07:53:57 2008
@@ -311,7 +311,7 @@
 			.suggestedLatency = (1.0 / 50.0), /* 20 ms */
 			.device = paNoDevice,
 		};
-		PaDeviceIndex index, num_devices, def_input, def_output;
+		PaDeviceIndex idx, num_devices, def_input, def_output;
 
 		if (!(num_devices = Pa_GetDeviceCount()))
 			return res;
@@ -319,23 +319,23 @@
 		def_input = Pa_GetDefaultInputDevice();
 		def_output = Pa_GetDefaultOutputDevice();
 
-		for (index = 0; 
-			index < num_devices && (input_params.device == paNoDevice 
+		for (idx = 0; 
+			idx < num_devices && (input_params.device == paNoDevice 
 				|| output_params.device == paNoDevice); 
-			index++) 
+			idx++) 
 		{
-			const PaDeviceInfo *dev = Pa_GetDeviceInfo(index);
+			const PaDeviceInfo *dev = Pa_GetDeviceInfo(idx);
 
 			if (dev->maxInputChannels) {
-				if ( (index == def_input && !strcasecmp(pvt->input_device, "default")) ||
+				if ( (idx == def_input && !strcasecmp(pvt->input_device, "default")) ||
 					!strcasecmp(pvt->input_device, dev->name) )
-					input_params.device = index;
+					input_params.device = idx;
 			}
 
 			if (dev->maxOutputChannels) {
-				if ( (index == def_output && !strcasecmp(pvt->output_device, "default")) ||
+				if ( (idx == def_output && !strcasecmp(pvt->output_device, "default")) ||
 					!strcasecmp(pvt->output_device, dev->name) )
-					output_params.device = index;
+					output_params.device = idx;
 			}
 		}
 
@@ -922,7 +922,7 @@
 
 static char *cli_list_available(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
 {
-	PaDeviceIndex index, num, def_input, def_output;
+	PaDeviceIndex idx, num, def_input, def_output;
 
 	if (cmd == CLI_INIT) {
 		e->command = "console list available";
@@ -950,16 +950,16 @@
 
 	def_input = Pa_GetDefaultInputDevice();
 	def_output = Pa_GetDefaultOutputDevice();
-	for (index = 0; index < num; index++) {
-		const PaDeviceInfo *dev = Pa_GetDeviceInfo(index);
+	for (idx = 0; idx < num; idx++) {
+		const PaDeviceInfo *dev = Pa_GetDeviceInfo(idx);
 		if (!dev)
 			continue;
 		ast_cli(a->fd, "=== ---------------------------------------------------------\n"
 		               "=== Device Name: %s\n", dev->name);
 		if (dev->maxInputChannels)
-			ast_cli(a->fd, "=== ---> %sInput Device\n", (index == def_input) ? "Default " : "");
+			ast_cli(a->fd, "=== ---> %sInput Device\n", (idx == def_input) ? "Default " : "");
 		if (dev->maxOutputChannels)
-			ast_cli(a->fd, "=== ---> %sOutput Device\n", (index == def_output) ? "Default " : "");
+			ast_cli(a->fd, "=== ---> %sOutput Device\n", (idx == def_output) ? "Default " : "");
 		ast_cli(a->fd, "=== ---------------------------------------------------------\n===\n");
 	}
 

Modified: trunk/channels/chan_features.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_features.c?view=diff&rev=136888&r1=136887&r2=136888
==============================================================================
--- trunk/channels/chan_features.c (original)
+++ trunk/channels/chan_features.c Sat Aug  9 07:53:57 2008
@@ -151,40 +151,40 @@
 }
 #endif
 
-static void restore_channel(struct feature_pvt *p, int index)
+static void restore_channel(struct feature_pvt *p, int idx)
 {
 	/* Restore timing/alertpipe */
-	p->subs[index].owner->timingfd = p->subs[index].timingfdbackup;
-	p->subs[index].owner->alertpipe[0] = p->subs[index].alertpipebackup[0];
-	p->subs[index].owner->alertpipe[1] = p->subs[index].alertpipebackup[1];
-	ast_channel_set_fd(p->subs[index].owner, AST_ALERT_FD, p->subs[index].alertpipebackup[0]);
-	ast_channel_set_fd(p->subs[index].owner, AST_TIMING_FD, p->subs[index].timingfdbackup);
-}
-
-static void update_features(struct feature_pvt *p, int index)
-{
-	int x;
-	if (p->subs[index].owner) {
+	p->subs[idx].owner->timingfd = p->subs[idx].timingfdbackup;
+	p->subs[idx].owner->alertpipe[0] = p->subs[idx].alertpipebackup[0];
+	p->subs[idx].owner->alertpipe[1] = p->subs[idx].alertpipebackup[1];
+	ast_channel_set_fd(p->subs[idx].owner, AST_ALERT_FD, p->subs[idx].alertpipebackup[0]);
+	ast_channel_set_fd(p->subs[idx].owner, AST_TIMING_FD, p->subs[idx].timingfdbackup);
+}
+
+static void update_features(struct feature_pvt *p, int idx)
+{
+	int x;
+	if (p->subs[idx].owner) {
 		for (x=0; x<AST_MAX_FDS; x++) {
-			if (index) 
-				ast_channel_set_fd(p->subs[index].owner, x, -1);
+			if (idx) 
+				ast_channel_set_fd(p->subs[idx].owner, x, -1);
 			else
-				ast_channel_set_fd(p->subs[index].owner, x, p->subchan->fds[x]);
+				ast_channel_set_fd(p->subs[idx].owner, x, p->subchan->fds[x]);
 		}
-		if (!index) {
+		if (!idx) {
 			/* Copy timings from master channel */
-			p->subs[index].owner->timingfd = p->subchan->timingfd;
-			p->subs[index].owner->alertpipe[0] = p->subchan->alertpipe[0];
-			p->subs[index].owner->alertpipe[1] = p->subchan->alertpipe[1];
-			if (p->subs[index].owner->nativeformats != p->subchan->readformat) {
-				p->subs[index].owner->nativeformats = p->subchan->readformat;
-				if (p->subs[index].owner->readformat)
-					ast_set_read_format(p->subs[index].owner, p->subs[index].owner->readformat);
-				if (p->subs[index].owner->writeformat)
-					ast_set_write_format(p->subs[index].owner, p->subs[index].owner->writeformat);
+			p->subs[idx].owner->timingfd = p->subchan->timingfd;
+			p->subs[idx].owner->alertpipe[0] = p->subchan->alertpipe[0];
+			p->subs[idx].owner->alertpipe[1] = p->subchan->alertpipe[1];
+			if (p->subs[idx].owner->nativeformats != p->subchan->readformat) {
+				p->subs[idx].owner->nativeformats = p->subchan->readformat;
+				if (p->subs[idx].owner->readformat)
+					ast_set_read_format(p->subs[idx].owner, p->subs[idx].owner->readformat);
+				if (p->subs[idx].owner->writeformat)
+					ast_set_write_format(p->subs[idx].owner, p->subs[idx].owner->writeformat);
 			}
 		} else{
-			restore_channel(p, index);
+			restore_channel(p, idx);
 		}
 	}
 }
@@ -432,7 +432,7 @@
 	return tmp;
 }
 
-static struct ast_channel *features_new(struct feature_pvt *p, int state, int index)
+static struct ast_channel *features_new(struct feature_pvt *p, int state, int idx)
 {
 	struct ast_channel *tmp;
 	int x,y;
@@ -441,8 +441,8 @@
 		ast_log(LOG_WARNING, "Called upon channel with no subchan:(\n");
 		return NULL;
 	}
-	if (p->subs[index].owner) {
-		ast_log(LOG_WARNING, "Called to put index %d already there!\n", index);
+	if (p->subs[idx].owner) {
+		ast_log(LOG_WARNING, "Called to put index %d already there!\n", idx);
 		return NULL;
 	}
 	/* figure out what you want the name to be */
@@ -451,7 +451,7 @@
 			ast_free(b2);
 		asprintf(&b2, "%s/%s-%d", p->tech, p->dest, x);
 		for (y=0;y<3;y++) {
-			if (y == index)
+			if (y == idx)
 				continue;
 			if (p->subs[y].owner && !strcasecmp(p->subs[y].owner->name, b2))
 				break;
@@ -474,7 +474,7 @@
 	tmp->rawreadformat = p->subchan->rawreadformat;
 	tmp->nativeformats = p->subchan->readformat;
 	tmp->tech_pvt = p;
-	p->subs[index].owner = tmp;
+	p->subs[idx].owner = tmp;
 	if (!p->owner)
 		p->owner = tmp;
 	ast_module_ref(ast_module_info->self);

Modified: trunk/channels/chan_jingle.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_jingle.c?view=diff&rev=136888&r1=136887&r2=136888
==============================================================================
--- trunk/channels/chan_jingle.c (original)
+++ trunk/channels/chan_jingle.c Sat Aug  9 07:53:57 2008
@@ -161,7 +161,7 @@
 };
 
 static const char desc[] = "Jingle Channel";
-static const char type[] = "Jingle";
+static const char channel_type[] = "Jingle";
 
 static int global_capability = AST_FORMAT_ULAW | AST_FORMAT_ALAW | AST_FORMAT_GSM | AST_FORMAT_H263;
 
@@ -1464,7 +1464,7 @@
 }
 
 /*! \brief Part of PBX interface */
-static struct ast_channel *jingle_request(const char *type, int format, void *data, int *cause)
+static struct ast_channel *jingle_request(const char *request_type, int format, void *data, int *cause)
 {
 	struct jingle_pvt *p = NULL;
 	struct jingle *client = NULL;
@@ -1882,7 +1882,7 @@
 	ast_cli_register_multiple(jingle_cli, ARRAY_LEN(jingle_cli));
 	/* Make sure we can register our channel type */
 	if (ast_channel_register(&jingle_tech)) {
-		ast_log(LOG_ERROR, "Unable to register channel class %s\n", type);
+		ast_log(LOG_ERROR, "Unable to register channel class %s\n", channel_type);
 		return -1;
 	}
 	return 0;

Modified: trunk/channels/chan_phone.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_phone.c?view=diff&rev=136888&r1=136887&r2=136888
==============================================================================
--- trunk/channels/chan_phone.c (original)
+++ trunk/channels/chan_phone.c Sat Aug  9 07:53:57 2008
@@ -844,18 +844,18 @@
 	return 0;
 }
 
-static struct ast_channel *phone_new(struct phone_pvt *i, int state, char *context)
+static struct ast_channel *phone_new(struct phone_pvt *i, int state, char *cntx)
 {
 	struct ast_channel *tmp;
-	struct phone_codec_data codec;
+	struct phone_codec_data queried_codec;
 	tmp = ast_channel_alloc(1, state, i->cid_num, i->cid_name, "", i->ext, i->context, 0, "Phone/%s", i->dev + 5);
 	if (tmp) {
 		tmp->tech = cur_tech;
 		ast_channel_set_fd(tmp, 0, i->fd);
 		/* XXX Switching formats silently causes kernel panics XXX */
 		if (i->mode == MODE_FXS &&
-		    ioctl(i->fd, PHONE_QUERY_CODEC, &codec) == 0) {
-			if (codec.type == LINEAR16)
+		    ioctl(i->fd, PHONE_QUERY_CODEC, &queried_codec) == 0) {
+			if (queried_codec.type == LINEAR16)
 				tmp->nativeformats =
 				tmp->rawreadformat =
 				tmp->rawwriteformat =
@@ -876,7 +876,7 @@
 		if (state == AST_STATE_RING)
 			tmp->rings = 1;
 		tmp->tech_pvt = i;
-		ast_copy_string(tmp->context, context, sizeof(tmp->context));
+		ast_copy_string(tmp->context, cntx, sizeof(tmp->context));
 		if (!ast_strlen_zero(i->ext))
 			ast_copy_string(tmp->exten, i->ext, sizeof(tmp->exten));
 		else
@@ -1017,7 +1017,7 @@
 	struct phone_pvt *i;
 	int tonepos = 0;
 	/* The tone we're playing this round */
-	struct timeval tv = {0,0};
+	struct timeval wait = {0,0};
 	int dotone;
 	/* This thread monitors all the frame relay interfaces which are not yet in use
 	   (and thus do not have a separate thread) indefinitely */
@@ -1048,7 +1048,7 @@
 				if (i->dialtone && i->mode != MODE_SIGMA) {
 					/* Remember we're going to have to come back and play
 					   more dialtones */
-					if (ast_tvzero(tv)) {
+					if (ast_tvzero(wait)) {
 						/* If we're due for a dialtone, play one */
 						if (write(i->fd, DialTone + tonepos, 240) != 240)
 							ast_log(LOG_WARNING, "Dial tone write error\n");
@@ -1068,13 +1068,13 @@
 			tonepos += 240;
 			if (tonepos >= sizeof(DialTone))
 					tonepos = 0;
-			if (ast_tvzero(tv)) {
-				tv = ast_tv(30000, 0);
-			}
-			res = ast_select(n + 1, &rfds, NULL, &efds, &tv);
+			if (ast_tvzero(wait)) {
+				wait = ast_tv(30000, 0);
+			}
+			res = ast_select(n + 1, &rfds, NULL, &efds, &wait);
 		} else {
 			res = ast_select(n + 1, &rfds, NULL, &efds, NULL);
-			tv = ast_tv(0,0);
+			wait = ast_tv(0,0);
 			tonepos = 0;
 		}
 		/* Okay, select has finished.  Let's see what happened.  */




More information about the asterisk-commits mailing list