[asterisk-commits] seanbright: trunk r136408 - in /trunk: codecs/ formats/ utils/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Thu Aug 7 10:16:48 CDT 2008
Author: seanbright
Date: Thu Aug 7 10:16:48 2008
New Revision: 136408
URL: http://svn.digium.com/view/asterisk?view=rev&rev=136408
Log:
More merges from resolve-shadow warnings:
utils/
codecs/
and a change I missed from formats/
Modified:
trunk/codecs/Makefile
trunk/codecs/codec_dahdi.c
trunk/formats/msgsm.h
trunk/utils/astman.c
trunk/utils/extconf.c
trunk/utils/frame.c
trunk/utils/muted.c
trunk/utils/smsq.c
Modified: trunk/codecs/Makefile
URL: http://svn.digium.com/view/asterisk/trunk/codecs/Makefile?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/codecs/Makefile (original)
+++ trunk/codecs/Makefile Thu Aug 7 10:16:48 2008
@@ -46,7 +46,7 @@
$(if $(filter codec_lpc10,$(EMBEDDED_MODS)),modules.link,codec_lpc10.so): $(LIBLPC10)
$(LIBILBC):
- @$(MAKE) -C ilbc all ASTCFLAGS="$(filter-out -Wmissing-prototypes -Wmissing-declarations,$(ASTCFLAGS)) $(AST_NO_STRICT_OVERFLOW)"
+ @$(MAKE) -C ilbc all ASTCFLAGS="$(filter-out -Wmissing-prototypes -Wmissing-declarations -Wshadow,$(ASTCFLAGS)) $(AST_NO_STRICT_OVERFLOW)"
$(if $(filter codec_ilbc,$(EMBEDDED_MODS)),modules.link,codec_ilbc.so): $(LIBILBC)
Modified: trunk/codecs/codec_dahdi.c
URL: http://svn.digium.com/view/asterisk/trunk/codecs/codec_dahdi.c?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/codecs/codec_dahdi.c (original)
+++ trunk/codecs/codec_dahdi.c Thu Aug 7 10:16:48 2008
@@ -362,12 +362,12 @@
static void unregister_translators(void)
{
- struct translator *cur;
+ struct translator *current;
AST_LIST_LOCK(&translators);
- while ((cur = AST_LIST_REMOVE_HEAD(&translators, entry))) {
- ast_unregister_translator(&cur->t);
- ast_free(cur);
+ while ((current = AST_LIST_REMOVE_HEAD(&translators, entry))) {
+ ast_unregister_translator(¤t->t);
+ ast_free(current);
}
AST_LIST_UNLOCK(&translators);
}
Modified: trunk/formats/msgsm.h
URL: http://svn.digium.com/view/asterisk/trunk/formats/msgsm.h?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/formats/msgsm.h (original)
+++ trunk/formats/msgsm.h Thu Aug 7 10:16:48 2008
@@ -349,105 +349,105 @@
}
#define writeGSM_33(c1) { \
- gsm_byte *c = (c1); \
- *c++ = ((GSM_MAGIC & 0xF) << 4) /* 1 */ \
+ gsm_byte *__c = (c1); \
+ *__c++ = ((GSM_MAGIC & 0xF) << 4) /* 1 */ \
| ((LARc[0] >> 2) & 0xF); \
- *c++ = ((LARc[0] & 0x3) << 6) \
+ *__c++ = ((LARc[0] & 0x3) << 6) \
| (LARc[1] & 0x3F); \
- *c++ = ((LARc[2] & 0x1F) << 3) \
+ *__c++ = ((LARc[2] & 0x1F) << 3) \
| ((LARc[3] >> 2) & 0x7); \
- *c++ = ((LARc[3] & 0x3) << 6) \
+ *__c++ = ((LARc[3] & 0x3) << 6) \
| ((LARc[4] & 0xF) << 2) \
| ((LARc[5] >> 2) & 0x3); \
- *c++ = ((LARc[5] & 0x3) << 6) \
+ *__c++ = ((LARc[5] & 0x3) << 6) \
| ((LARc[6] & 0x7) << 3) \
| (LARc[7] & 0x7); \
- *c++ = ((Nc[0] & 0x7F) << 1) \
+ *__c++ = ((Nc[0] & 0x7F) << 1) \
| ((bc[0] >> 1) & 0x1); \
- *c++ = ((bc[0] & 0x1) << 7) \
+ *__c++ = ((bc[0] & 0x1) << 7) \
| ((Mc[0] & 0x3) << 5) \
| ((xmaxc[0] >> 1) & 0x1F); \
- *c++ = ((xmaxc[0] & 0x1) << 7) \
+ *__c++ = ((xmaxc[0] & 0x1) << 7) \
| ((xmc[0] & 0x7) << 4) \
| ((xmc[1] & 0x7) << 1) \
| ((xmc[2] >> 2) & 0x1); \
- *c++ = ((xmc[2] & 0x3) << 6) \
+ *__c++ = ((xmc[2] & 0x3) << 6) \
| ((xmc[3] & 0x7) << 3) \
| (xmc[4] & 0x7); \
- *c++ = ((xmc[5] & 0x7) << 5) /* 10 */ \
+ *__c++ = ((xmc[5] & 0x7) << 5) /* 10 */ \
| ((xmc[6] & 0x7) << 2) \
| ((xmc[7] >> 1) & 0x3); \
- *c++ = ((xmc[7] & 0x1) << 7) \
+ *__c++ = ((xmc[7] & 0x1) << 7) \
| ((xmc[8] & 0x7) << 4) \
| ((xmc[9] & 0x7) << 1) \
| ((xmc[10] >> 2) & 0x1); \
- *c++ = ((xmc[10] & 0x3) << 6) \
+ *__c++ = ((xmc[10] & 0x3) << 6) \
| ((xmc[11] & 0x7) << 3) \
| (xmc[12] & 0x7); \
- *c++ = ((Nc[1] & 0x7F) << 1) \
+ *__c++ = ((Nc[1] & 0x7F) << 1) \
| ((bc[1] >> 1) & 0x1); \
- *c++ = ((bc[1] & 0x1) << 7) \
+ *__c++ = ((bc[1] & 0x1) << 7) \
| ((Mc[1] & 0x3) << 5) \
| ((xmaxc[1] >> 1) & 0x1F); \
- *c++ = ((xmaxc[1] & 0x1) << 7) \
+ *__c++ = ((xmaxc[1] & 0x1) << 7) \
| ((xmc[13] & 0x7) << 4) \
| ((xmc[14] & 0x7) << 1) \
| ((xmc[15] >> 2) & 0x1); \
- *c++ = ((xmc[15] & 0x3) << 6) \
+ *__c++ = ((xmc[15] & 0x3) << 6) \
| ((xmc[16] & 0x7) << 3) \
| (xmc[17] & 0x7); \
- *c++ = ((xmc[18] & 0x7) << 5) \
+ *__c++ = ((xmc[18] & 0x7) << 5) \
| ((xmc[19] & 0x7) << 2) \
| ((xmc[20] >> 1) & 0x3); \
- *c++ = ((xmc[20] & 0x1) << 7) \
+ *__c++ = ((xmc[20] & 0x1) << 7) \
| ((xmc[21] & 0x7) << 4) \
| ((xmc[22] & 0x7) << 1) \
| ((xmc[23] >> 2) & 0x1); \
- *c++ = ((xmc[23] & 0x3) << 6) \
+ *__c++ = ((xmc[23] & 0x3) << 6) \
| ((xmc[24] & 0x7) << 3) \
| (xmc[25] & 0x7); \
- *c++ = ((Nc[2] & 0x7F) << 1) /* 20 */ \
+ *__c++ = ((Nc[2] & 0x7F) << 1) /* 20 */ \
| ((bc[2] >> 1) & 0x1); \
- *c++ = ((bc[2] & 0x1) << 7) \
+ *__c++ = ((bc[2] & 0x1) << 7) \
| ((Mc[2] & 0x3) << 5) \
| ((xmaxc[2] >> 1) & 0x1F); \
- *c++ = ((xmaxc[2] & 0x1) << 7) \
+ *__c++ = ((xmaxc[2] & 0x1) << 7) \
| ((xmc[26] & 0x7) << 4) \
| ((xmc[27] & 0x7) << 1) \
| ((xmc[28] >> 2) & 0x1); \
- *c++ = ((xmc[28] & 0x3) << 6) \
+ *__c++ = ((xmc[28] & 0x3) << 6) \
| ((xmc[29] & 0x7) << 3) \
| (xmc[30] & 0x7); \
- *c++ = ((xmc[31] & 0x7) << 5) \
+ *__c++ = ((xmc[31] & 0x7) << 5) \
| ((xmc[32] & 0x7) << 2) \
| ((xmc[33] >> 1) & 0x3); \
- *c++ = ((xmc[33] & 0x1) << 7) \
+ *__c++ = ((xmc[33] & 0x1) << 7) \
| ((xmc[34] & 0x7) << 4) \
| ((xmc[35] & 0x7) << 1) \
| ((xmc[36] >> 2) & 0x1); \
- *c++ = ((xmc[36] & 0x3) << 6) \
+ *__c++ = ((xmc[36] & 0x3) << 6) \
| ((xmc[37] & 0x7) << 3) \
| (xmc[38] & 0x7); \
- *c++ = ((Nc[3] & 0x7F) << 1) \
+ *__c++ = ((Nc[3] & 0x7F) << 1) \
| ((bc[3] >> 1) & 0x1); \
- *c++ = ((bc[3] & 0x1) << 7) \
+ *__c++ = ((bc[3] & 0x1) << 7) \
| ((Mc[3] & 0x3) << 5) \
| ((xmaxc[3] >> 1) & 0x1F); \
- *c++ = ((xmaxc[3] & 0x1) << 7) \
+ *__c++ = ((xmaxc[3] & 0x1) << 7) \
| ((xmc[39] & 0x7) << 4) \
| ((xmc[40] & 0x7) << 1) \
| ((xmc[41] >> 2) & 0x1); \
- *c++ = ((xmc[41] & 0x3) << 6) /* 30 */ \
+ *__c++ = ((xmc[41] & 0x3) << 6) /* 30 */ \
| ((xmc[42] & 0x7) << 3) \
| (xmc[43] & 0x7); \
- *c++ = ((xmc[44] & 0x7) << 5) \
+ *__c++ = ((xmc[44] & 0x7) << 5) \
| ((xmc[45] & 0x7) << 2) \
| ((xmc[46] >> 1) & 0x3); \
- *c++ = ((xmc[46] & 0x1) << 7) \
+ *__c++ = ((xmc[46] & 0x1) << 7) \
| ((xmc[47] & 0x7) << 4) \
| ((xmc[48] & 0x7) << 1) \
| ((xmc[49] >> 2) & 0x1); \
- *c++ = ((xmc[49] & 0x3) << 6) \
+ *__c++ = ((xmc[49] & 0x3) << 6) \
| ((xmc[50] & 0x7) << 3) \
| (xmc[51] & 0x7); \
}
Modified: trunk/utils/astman.c
URL: http://svn.digium.com/view/asterisk/trunk/utils/astman.c?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/utils/astman.c (original)
+++ trunk/utils/astman.c Thu Aug 7 10:16:48 2008
@@ -626,7 +626,7 @@
return 0;
}
-static int login(char *hostname)
+static int manager_login(char *hostname)
{
newtComponent form;
newtComponent cancel;
@@ -756,7 +756,7 @@
newtCls();
newtDrawRootText(0, 0, "Asterisk Manager (C)2002, Linux Support Services, Inc.");
newtPushHelpLine("Welcome to the Asterisk Manager!");
- if (login(argv[1])) {
+ if (manager_login(argv[1])) {
newtFinished();
exit(1);
}
Modified: trunk/utils/extconf.c
URL: http://svn.digium.com/view/asterisk/trunk/utils/extconf.c?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/utils/extconf.c (original)
+++ trunk/utils/extconf.c Thu Aug 7 10:16:48 2008
@@ -1247,7 +1247,7 @@
static void *safe_system_prev_handler;
/*! \brief NULL handler so we can collect the child exit status */
-static void null_sig_handler(int signal)
+static void null_sig_handler(int sig)
{
}
@@ -2639,8 +2639,8 @@
};
-static char *config = "extensions.conf";
-static char *registrar = "conf2ael";
+static char *config_filename = "extensions.conf";
+static char *global_registrar = "conf2ael";
static char userscontext[AST_MAX_EXTENSION] = "default";
static int static_config = 0;
static int write_protect_config = 1;
@@ -3791,9 +3791,9 @@
}
if (process_buf) {
- char *buf = ast_strip(process_buf);
- if (!ast_strlen_zero(buf)) {
- if (process_text_line(cfg, &cat, buf, lineno, filename, withcomments, suggested_include_file)) {
+ char *stripped_process_buf = ast_strip(process_buf);
+ if (!ast_strlen_zero(stripped_process_buf)) {
+ if (process_text_line(cfg, &cat, stripped_process_buf, lineno, filename, withcomments, suggested_include_file)) {
cfg = NULL;
break;
}
@@ -5249,16 +5249,16 @@
static struct ast_context *__ast_context_create(struct ast_context **extcontexts, const char *name, const char *registrar, int existsokay)
{
- struct ast_context *tmp, **local_contexts;
+ struct ast_context *tmp, **loc_contexts;
int length = sizeof(struct ast_context) + strlen(name) + 1;
if (!extcontexts) {
ast_wrlock_contexts();
- local_contexts = &contexts;
+ loc_contexts = &contexts;
} else
- local_contexts = extcontexts;
-
- for (tmp = *local_contexts; tmp; tmp = tmp->next) {
+ loc_contexts = extcontexts;
+
+ for (tmp = *loc_contexts; tmp; tmp = tmp->next) {
if (!strcasecmp(tmp->name, name)) {
if (!existsokay) {
ast_log(LOG_WARNING, "Tried to register context '%s', already in use\n", name);
@@ -5275,10 +5275,10 @@
strcpy(tmp->name, name);
tmp->root = NULL;
tmp->registrar = registrar;
- tmp->next = *local_contexts;
+ tmp->next = *loc_contexts;
tmp->includes = NULL;
tmp->ignorepats = NULL;
- *local_contexts = tmp;
+ *loc_contexts = tmp;
if (option_debug)
ast_log(LOG_DEBUG, "Registered context '%s'\n", tmp->name);
if (option_verbose > 2)
@@ -5916,7 +5916,7 @@
/* All categories but "general" or "globals" are considered contexts */
if (!strcasecmp(cxt, "general") || !strcasecmp(cxt, "globals"))
continue;
- con=ast_context_find_or_create(&local_contexts,NULL,cxt, registrar);
+ con=ast_context_find_or_create(&local_contexts,NULL,cxt, global_registrar);
if (con == NULL)
continue;
@@ -6004,7 +6004,7 @@
lastpri = ipri;
if (!ast_opt_dont_warn && !strcmp(realext, "_."))
ast_log(LOG_WARNING, "The use of '_.' for an extension is strongly discouraged and can have unexpected behavior. Please use '_X.' instead at line %d\n", v->lineno);
- if (ast_add_extension2(con, 0, realext, ipri, label, cidmatch, appl, strdup(data), ast_free, registrar)) {
+ if (ast_add_extension2(con, 0, realext, ipri, label, cidmatch, appl, strdup(data), ast_free, global_registrar)) {
ast_log(LOG_WARNING, "Unable to register extension at line %d\n", v->lineno);
}
}
@@ -6013,12 +6013,12 @@
} else if (!strcasecmp(v->name, "include")) {
memset(realvalue, 0, sizeof(realvalue));
pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
- if (ast_context_add_include2(con, realvalue, registrar))
+ if (ast_context_add_include2(con, realvalue, global_registrar))
ast_log(LOG_WARNING, "Unable to include context '%s' in context '%s'\n", v->value, cxt);
} else if (!strcasecmp(v->name, "ignorepat")) {
memset(realvalue, 0, sizeof(realvalue));
pbx_substitute_variables_helper(NULL, v->value, realvalue, sizeof(realvalue) - 1);
- if (ast_context_add_ignorepat2(con, realvalue, registrar))
+ if (ast_context_add_ignorepat2(con, realvalue, global_registrar))
ast_log(LOG_WARNING, "Unable to include ignorepat '%s' in context '%s'\n", v->value, cxt);
} else if (!strcasecmp(v->name, "switch") || !strcasecmp(v->name, "lswitch") || !strcasecmp(v->name, "eswitch")) {
char *stringp= realvalue;
@@ -6033,7 +6033,7 @@
data = strsep(&stringp, ""); /* XXX what for ? */
if (!data)
data = "";
- if (ast_context_add_switch2(con, appl, data, !strcasecmp(v->name, "eswitch"), registrar))
+ if (ast_context_add_switch2(con, appl, data, !strcasecmp(v->name, "eswitch"), global_registrar))
ast_log(LOG_WARNING, "Unable to include switch '%s' in context '%s'\n", v->value, cxt);
} else {
ast_log(LOG_WARNING, "==!!== Unknown directive: %s at line %d -- IGNORING!!!\n", v->name, v->lineno);
@@ -6193,12 +6193,12 @@
{
struct ast_context *con;
- if(!pbx_load_config(config))
+ if(!pbx_load_config(config_filename))
return -1 /* AST_MODULE_LOAD_DECLINE*/;
/* pbx_load_users(); */ /* does this affect the dialplan? */
- ast_merge_contexts_and_delete(&local_contexts, registrar);
+ ast_merge_contexts_and_delete(&local_contexts, global_registrar);
for (con = NULL; (con = ast_walk_contexts(con));)
ast_context_verify_includes(con);
Modified: trunk/utils/frame.c
URL: http://svn.digium.com/view/asterisk/trunk/utils/frame.c?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/utils/frame.c (original)
+++ trunk/utils/frame.c Thu Aug 7 10:16:48 2008
@@ -1008,7 +1008,7 @@
return 6.0 * log( value / 32767) / log( 2);
}
-void readawaysamples( FILE *in, size_t size)
+void readawaysamples( FILE *input, size_t size)
{
short *buffer;
int samplesread, count;
@@ -1023,8 +1023,8 @@
else
count = size;
- samplesread = fread( buffer, sizeof(*buffer), count, in);
- if (ferror( in) != 0)
+ samplesread = fread( buffer, sizeof(*buffer), count, input);
+ if (ferror( input) != 0)
fatalperror("Error reading input file");
size -= samplesread;
}
Modified: trunk/utils/muted.c
URL: http://svn.digium.com/view/asterisk/trunk/utils/muted.c?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/utils/muted.c (original)
+++ trunk/utils/muted.c Thu Aug 7 10:16:48 2008
@@ -410,19 +410,19 @@
#endif
#ifndef __Darwin__
-static int mutedlevel(int orig, int mutelevel)
+static int mutedlevel(int orig, int level)
{
int l = orig >> 8;
int r = orig & 0xff;
- l = (float)(mutelevel) * (float)(l) / 100.0;
- r = (float)(mutelevel) * (float)(r) / 100.0;
+ l = (float)(level) * (float)(l) / 100.0;
+ r = (float)(level) * (float)(r) / 100.0;
return (l << 8) | r;
#else
-static float mutedlevel(float orig, float mutelevel)
+static float mutedlevel(float orig, float level)
{
float master = orig;
- master = mutelevel * master / 100.0;
+ master = level * master / 100.0;
return master;
#endif
Modified: trunk/utils/smsq.c
URL: http://svn.digium.com/view/asterisk/trunk/utils/smsq.c?view=diff&rev=136408&r1=136407&r2=136408
==============================================================================
--- trunk/utils/smsq.c (original)
+++ trunk/utils/smsq.c Thu Aug 7 10:16:48 2008
@@ -321,83 +321,82 @@
fclose (f);
/* set up user data variables */
{
- char temp[481];
- int n,
- p;
- for (n = 0, p = 0; p < udl; p++)
+ char tmp[481];
+ int n, x;
+ for (n = 0, x = 0; x < udl; x++)
{
- unsigned short v = ud[p];
+ unsigned short v = ud[x];
if (v)
{
if (v < 0x80)
- temp[n++] = v;
+ tmp[n++] = v;
else if (v < 0x800)
{
- temp[n++] = (0xC0 + (v >> 6));
- temp[n++] = (0x80 + (v & 0x3F));
+ tmp[n++] = (0xC0 + (v >> 6));
+ tmp[n++] = (0x80 + (v & 0x3F));
} else
{
- temp[n++] = (0xE0 + (v >> 12));
- temp[n++] = (0x80 + ((v >> 6) & 0x3F));
- temp[n++] = (0x80 + (v & 0x3F));
+ tmp[n++] = (0xE0 + (v >> 12));
+ tmp[n++] = (0x80 + ((v >> 6) & 0x3F));
+ tmp[n++] = (0x80 + (v & 0x3F));
}
}
}
- temp[n] = 0;
- setenv ("ud", temp, 1);
- for (n = 0, p = 0; p < udl; p++)
+ tmp[n] = 0;
+ setenv ("ud", tmp, 1);
+ for (n = 0, x = 0; x < udl; x++)
{
- unsigned short v = ud[p];
+ unsigned short v = ud[x];
if (v < ' ' || v == '\\')
{
- temp[n++] = '\\';
+ tmp[n++] = '\\';
if (v == '\\')
- temp[n++] = '\\';
+ tmp[n++] = '\\';
else if (v == '\n')
- temp[n++] = 'n';
+ tmp[n++] = 'n';
else if (v == '\r')
- temp[n++] = 'r';
+ tmp[n++] = 'r';
else if (v == '\t')
- temp[n++] = 't';
+ tmp[n++] = 't';
else if (v == '\f')
- temp[n++] = 'f';
+ tmp[n++] = 'f';
else
{
- temp[n++] = '0' + (v >> 6);
- temp[n++] = '0' + ((v >> 3) & 7);
- temp[n++] = '0' + (v & 7);
+ tmp[n++] = '0' + (v >> 6);
+ tmp[n++] = '0' + ((v >> 3) & 7);
+ tmp[n++] = '0' + (v & 7);
}
} else if (v < 0x80)
- temp[n++] = v;
+ tmp[n++] = v;
else if (v < 0x800)
{
- temp[n++] = (0xC0 + (v >> 6));
- temp[n++] = (0x80 + (v & 0x3F));
+ tmp[n++] = (0xC0 + (v >> 6));
+ tmp[n++] = (0x80 + (v & 0x3F));
} else
{
- temp[n++] = (0xE0 + (v >> 12));
- temp[n++] = (0x80 + ((v >> 6) & 0x3F));
- temp[n++] = (0x80 + (v & 0x3F));
+ tmp[n++] = (0xE0 + (v >> 12));
+ tmp[n++] = (0x80 + ((v >> 6) & 0x3F));
+ tmp[n++] = (0x80 + (v & 0x3F));
}
}
- temp[n] = 0;
- setenv ("ude", temp, 1);
- for (p = 0; p < udl && ud[p] < 0x100; p++);
- if (p == udl)
+ tmp[n] = 0;
+ setenv ("ude", tmp, 1);
+ for (x = 0; x < udl && ud[x] < 0x100; x++);
+ if (x == udl)
{
- for (n = 0, p = 0; p < udl; p++)
+ for (n = 0, x = 0; x < udl; x++)
{
- sprintf (temp + n, "%02X", ud[p]);
+ sprintf (tmp + n, "%02X", ud[x]);
n += 2;
}
- setenv ("ud8", temp, 1);
+ setenv ("ud8", tmp, 1);
}
- for (n = 0, p = 0; p < udl; p++)
+ for (n = 0, x = 0; x < udl; x++)
{
- sprintf (temp + n, "%04X", ud[p]);
+ sprintf (tmp + n, "%04X", ud[x]);
n += 4;
}
- setenv ("ud16", temp, 1);
+ setenv ("ud16", tmp, 1);
}
/* run the command */
system (process);
More information about the asterisk-commits
mailing list