diff options
-rw-r--r-- | gtk/bootp_stat.c | 6 | ||||
-rw-r--r-- | gtk/diameter_stat.c | 24 | ||||
-rw-r--r-- | gtk/gtp_stat.c | 28 | ||||
-rw-r--r-- | gtk/iax2_analysis.c | 20 | ||||
-rw-r--r-- | gtk/io_stat.c | 22 | ||||
-rw-r--r-- | gtk/mcast_stream_dlg.c | 40 | ||||
-rw-r--r-- | gtk/rtp_analysis.c | 4 | ||||
-rw-r--r-- | gtk/rtp_stream_dlg.c | 42 |
8 files changed, 91 insertions, 95 deletions
diff --git a/gtk/bootp_stat.c b/gtk/bootp_stat.c index a7cfead7a7..9f50a508ee 100644 --- a/gtk/bootp_stat.c +++ b/gtk/bootp_stat.c @@ -151,11 +151,11 @@ static void dhcpstat_draw(void *psp) { dhcpstat_t *sp=psp; - guint index; + guint idx; - index=sp->index; + idx=sp->index; g_hash_table_foreach( sp->hash, (GHFunc) dhcp_draw_message_type, NULL ); - if (index != sp->index){ + if (idx != sp->index){ /* We have inserted a new entry corresponding to a status code , * let's resize the table */ gtk_table_resize ( GTK_TABLE(sp->table_message_type), sp->index % 2 , 4); diff --git a/gtk/diameter_stat.c b/gtk/diameter_stat.c index 97297a7acd..11e380dbd3 100644 --- a/gtk/diameter_stat.c +++ b/gtk/diameter_stat.c @@ -91,18 +91,18 @@ diameterstat_packet(void *pdiameter, packet_info *pinfo, epan_dissect_t *edt _U_ { const diameter_req_ans_pair_t *diameter=pdi; diameterstat_t *fs=(diameterstat_t *)pdiameter; - int* index = NULL; + int* idx = NULL; - index = (int*) g_hash_table_lookup(cmd_str_hash, diameter->cmd_str); - if (index == NULL) { - index = g_malloc(sizeof(int)); - *index = (int) g_hash_table_size(cmd_str_hash); - g_hash_table_insert(cmd_str_hash, (gchar*) diameter->cmd_str, index); - init_srt_table_row(&fs->diameter_srt_table, *index, (const char*) diameter->cmd_str); + idx = (int*) g_hash_table_lookup(cmd_str_hash, diameter->cmd_str); + if (idx == NULL) { + idx = g_malloc(sizeof(int)); + *idx = (int) g_hash_table_size(cmd_str_hash); + g_hash_table_insert(cmd_str_hash, (gchar*) diameter->cmd_str, idx); + init_srt_table_row(&fs->diameter_srt_table, *idx, (const char*) diameter->cmd_str); } - add_srt_table_data(&fs->diameter_srt_table, *index, &diameter->req_time, pinfo); + add_srt_table_data(&fs->diameter_srt_table, *idx, &diameter->req_time, pinfo); return 1; } @@ -144,7 +144,7 @@ gtk_diameterstat_init(const char *optarg, void *userdata _U_) GtkWidget *vbox; GtkWidget *bbox; GtkWidget *close_bt; - int* index; + int* idx; if(!strncmp(optarg,"diameter,",9)){ filter=optarg+9; @@ -153,10 +153,10 @@ gtk_diameterstat_init(const char *optarg, void *userdata _U_) } diameter=g_malloc(sizeof(diameterstat_t)); - index = g_malloc(sizeof(int)); - *index = 0; + idx = g_malloc(sizeof(int)); + *idx = 0; cmd_str_hash = g_hash_table_new(g_str_hash,g_str_equal); - g_hash_table_insert(cmd_str_hash, (gchar *)"Unknown", index); + g_hash_table_insert(cmd_str_hash, (gchar *)"Unknown", idx); diameter->win = dlg_window_new("diameter-stat"); /* transient_for top_level */ gtk_window_set_destroy_with_parent (GTK_WINDOW(diameter->win), TRUE); diff --git a/gtk/gtp_stat.c b/gtk/gtp_stat.c index 8aea0f07d6..bfec670e84 100644 --- a/gtk/gtp_stat.c +++ b/gtk/gtp_stat.c @@ -87,7 +87,7 @@ gtpstat_packet(void *pgtp, packet_info *pinfo, epan_dissect_t *edt _U_, const vo { const gtp_msg_hash_t *gtp=psi; gtpstat_t *fs=(gtpstat_t *)pgtp; - int index=0; + int idx=0; /* we are only interested in reply packets */ if(gtp->is_request){ @@ -99,25 +99,25 @@ gtpstat_packet(void *pgtp, packet_info *pinfo, epan_dissect_t *edt _U_, const vo } /* Only use the commands we know how to handle, this is not a comprehensive list */ - /* Redoing the message indexing is bit reduntant,*/ - /* but using message type as such would yield a long gtp_srt_table.*/ - /* Only a fragtion of the messages are matchable req/resp pairs,*/ - /* it's just doesn't feel feasible*/ + /* Redoing the message indexing is bit reduntant, */ + /* but using message type as such would yield a long gtp_srt_table. */ + /* Only a fraction of the messages are matchable req/resp pairs, */ + /* it just doesn't feel feasible. */ switch(gtp->msgtype){ - case GTP_MSG_ECHO_REQ: index=0; - break; - case GTP_MSG_CREATE_PDP_REQ: index=1; - break; - case GTP_MSG_UPDATE_PDP_REQ: index=2; - break; - case GTP_MSG_DELETE_PDP_REQ: index=3; - break; + case GTP_MSG_ECHO_REQ: idx=0; + break; + case GTP_MSG_CREATE_PDP_REQ: idx=1; + break; + case GTP_MSG_UPDATE_PDP_REQ: idx=2; + break; + case GTP_MSG_DELETE_PDP_REQ: idx=3; + break; default: return 0; } - add_srt_table_data(&fs->gtp_srt_table, index, >p->req_time, pinfo); + add_srt_table_data(&fs->gtp_srt_table, idx, >p->req_time, pinfo); return 1; } diff --git a/gtk/iax2_analysis.c b/gtk/iax2_analysis.c index bbfdd7d06c..4bd35c3356 100644 --- a/gtk/iax2_analysis.c +++ b/gtk/iax2_analysis.c @@ -989,7 +989,7 @@ static void dialog_graph_draw(user_data_t* user_data) user_data->dlg.dialog_graph.pixmap_width-right_x_border+1, user_data->dlg.dialog_graph.pixmap_height-bottom_y_border); for(i=0;i<=10;i++){ - int xwidth, lwidth; + int xwidth; xwidth=5; if(!(i%5)){ @@ -1095,7 +1095,6 @@ static void dialog_graph_draw(user_data_t* user_data) user_data->dlg.dialog_graph.pixmap_height-bottom_y_border+xlen+1); if(xlen==17){ - int lwidth; if(user_data->dlg.dialog_graph.interval>=1000){ g_snprintf(label_string, sizeof(label_string), "%ds", current_interval/1000); } else if(user_data->dlg.dialog_graph.interval>=100){ @@ -1161,7 +1160,6 @@ static void dialog_graph_draw(user_data_t* user_data) x_pos=draw_width-1-user_data->dlg.dialog_graph.pixels_per_tick*((last_interval-interval)/user_data->dlg.dialog_graph.interval+1)+left_x_border; if(user_data->dlg.dialog_graph.graph[i].items[interval/user_data->dlg.dialog_graph.interval].flags & (STAT_FLAG_WRONG_SEQ|STAT_FLAG_MARKER)){ - int lwidth; if (user_data->dlg.dialog_graph.graph[i].items[interval/user_data->dlg.dialog_graph.interval].flags & STAT_FLAG_WRONG_SEQ){ g_strlcpy(label_string, "x", sizeof(label_string)); } else { @@ -2109,7 +2107,7 @@ static void save_voice_as_destroy_cb(GtkWidget *win _U_, user_data_t *user_data) /* XXX what about endians here? could go something wrong? */ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t *user_data) { - int to_fd, forw_fd, rev_fd, fread = 0, rread = 0, fwritten, rwritten; + int to_fd, forw_fd, rev_fd, fread_cnt = 0, rread = 0, fwritten, rwritten; gchar f_pd[1] = {0}; gchar r_pd[1] = {0}; gint16 sample; @@ -2168,7 +2166,7 @@ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t * case SAVE_FORWARD_DIRECTION_MASK: { progbar_count = user_data->forward.saveinfo.count; progbar_quantum = user_data->forward.saveinfo.count/100; - while ((fread = read(forw_fd, f_pd, 1)) > 0) { + while ((fread_cnt = read(forw_fd, f_pd, 1)) > 0) { if(stop_flag) break; if((count > progbar_nextstep) && (count <= progbar_count)) { @@ -2195,7 +2193,7 @@ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t * } fwritten = ws_write(to_fd, pd, 2); - if ((fwritten < 2) || (fwritten < 0) || (fread < 0)) { + if ((fwritten < 2) || (fwritten < 0) || (fread_cnt < 0)) { ws_close(forw_fd); ws_close(rev_fd); ws_close(to_fd); @@ -2281,11 +2279,11 @@ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t * *f_pd = SILENCE_PCMA; break; } - fread = 1; + fread_cnt = 1; f_write_silence--; } else if(r_write_silence > 0) { - fread = read(forw_fd, f_pd, 1); + fread_cnt = read(forw_fd, f_pd, 1); switch (user_data->reversed.statinfo.reg_pt) { case AST_FORMAT_ULAW: *r_pd = SILENCE_PCMU; @@ -2298,10 +2296,10 @@ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t * r_write_silence--; } else { - fread = read(forw_fd, f_pd, 1); + fread_cnt = read(forw_fd, f_pd, 1); rread = read(rev_fd, r_pd, 1); } - if ((rread == 0) && (fread == 0)) + if ((rread == 0) && (fread_cnt == 0)) break; if ((user_data->forward.statinfo.pt == AST_FORMAT_ULAW) && (user_data->reversed.statinfo.pt == AST_FORMAT_ULAW)){ sample = (ulaw2linear(*r_pd) + ulaw2linear(*f_pd)) / 2; @@ -2322,7 +2320,7 @@ static gboolean copy_file(gchar *dest, gint channels, gint format, user_data_t * rwritten = ws_write(to_fd, pd, 2); - if ((rwritten < 2) || (rread < 0) || (fread < 0)) { + if ((rwritten < 2) || (rread < 0) || (fread_cnt < 0)) { ws_close(forw_fd); ws_close(rev_fd); ws_close(to_fd); diff --git a/gtk/io_stat.c b/gtk/io_stat.c index 2f6c250d10..d1d6439003 100644 --- a/gtk/io_stat.c +++ b/gtk/io_stat.c @@ -385,7 +385,7 @@ gtk_iostat_packet(void *g, packet_info *pinfo, epan_dissect_t *edt, const void * switch(git->calc_type){ guint64 t, pt; /* time in us */ - int i; + int j; case CALC_TYPE_LOAD: /* it is a LOAD calculation of a relative time field. * add the time this call spanned to each @@ -394,7 +394,7 @@ gtk_iostat_packet(void *g, packet_info *pinfo, epan_dissect_t *edt, const void * */ t=new_time->secs; t=t*1000000+new_time->nsecs/1000; - i=idx; + j=idx; /* handle current interval */ pt=pinfo->fd->rel_ts.secs*1000000+pinfo->fd->rel_ts.nsecs/1000; pt=pt%(git->io->interval*1000); @@ -402,16 +402,16 @@ gtk_iostat_packet(void *g, packet_info *pinfo, epan_dissect_t *edt, const void * pt=t; } while(t){ - git->items[i].time_tot.nsecs+=(int) (pt*1000); - if(git->items[i].time_tot.nsecs>1000000000){ - git->items[i].time_tot.secs++; - git->items[i].time_tot.nsecs-=1000000000; + git->items[j].time_tot.nsecs+=(int) (pt*1000); + if(git->items[j].time_tot.nsecs>1000000000){ + git->items[j].time_tot.secs++; + git->items[j].time_tot.nsecs-=1000000000; } - if(i==0){ + if(j==0){ break; } - i--; + j--; t-=pt; if(t > (guint32) (git->io->interval*1000)){ pt=git->io->interval*1000; @@ -635,13 +635,13 @@ get_it_value(io_stat_t *io, int graph_id, int idx) static void -print_time_scale_string(char *buf, int buf_len, guint32 t, guint32 t_max, gboolean log) +print_time_scale_string(char *buf, int buf_len, guint32 t, guint32 t_max, gboolean log_flag) { - if(t_max>=10000000 || (log && t_max>=1000000)){ + if(t_max>=10000000 || (log_flag && t_max>=1000000)){ g_snprintf(buf, buf_len, "%ds",t/1000000); } else if(t_max>=1000000){ g_snprintf(buf, buf_len, "%d.%1ds",t/1000000,(t%1000000)/100000); - } else if(t_max>=10000 || (log && t_max>=1000)){ + } else if(t_max>=10000 || (log_flag && t_max>=1000)){ g_snprintf(buf, buf_len, "%dms",t/1000); } else if(t_max>=1000){ g_snprintf(buf, buf_len, "%d.%1dms",t/1000,(t%1000)/100); diff --git a/gtk/mcast_stream_dlg.c b/gtk/mcast_stream_dlg.c index 408ad36e90..f556742bf0 100644 --- a/gtk/mcast_stream_dlg.c +++ b/gtk/mcast_stream_dlg.c @@ -220,15 +220,15 @@ mcaststream_on_filter (GtkButton *button _U_, /****************************************************************************/ /* when the user selects a row in the stream list */ static void -mcaststream_on_select_row(GtkCList *clist, - gint row _U_, - gint column _U_, - GdkEventButton *event _U_, - gpointer user_data _U_) +mcaststream_on_select_row(GtkCList *clist_lcl, + gint row _U_, + gint column _U_, + GdkEventButton *event _U_, + gpointer user_data _U_) { gchar label_text[80]; - selected_stream_fwd = gtk_clist_get_row_data(GTK_CLIST(clist), row); + selected_stream_fwd = gtk_clist_get_row_data(GTK_CLIST(clist_lcl), row); g_snprintf(label_text, sizeof(label_text), "Selected: %s:%u -> %s:%u", get_addr_name(&(selected_stream_fwd->src_addr)), selected_stream_fwd->src_port, @@ -254,40 +254,40 @@ typedef struct column_arrows { /****************************************************************************/ static void -mcaststream_click_column_cb(GtkCList *clist, gint column, gpointer data) +mcaststream_click_column_cb(GtkCList *clist_lcl, gint column, gpointer data) { column_arrows *col_arrows = (column_arrows *) data; int i; - gtk_clist_freeze(clist); + gtk_clist_freeze(clist_lcl); for (i=0; i<NUM_COLS; i++) { gtk_widget_hide(col_arrows[i].ascend_pm); gtk_widget_hide(col_arrows[i].descend_pm); } - if (column == clist->sort_column) { - if (clist->sort_type == GTK_SORT_ASCENDING) { - clist->sort_type = GTK_SORT_DESCENDING; + if (column == clist_lcl->sort_column) { + if (clist_lcl->sort_type == GTK_SORT_ASCENDING) { + clist_lcl->sort_type = GTK_SORT_DESCENDING; gtk_widget_show(col_arrows[column].descend_pm); } else { - clist->sort_type = GTK_SORT_ASCENDING; + clist_lcl->sort_type = GTK_SORT_ASCENDING; gtk_widget_show(col_arrows[column].ascend_pm); } } else { - clist->sort_type = GTK_SORT_ASCENDING; + clist_lcl->sort_type = GTK_SORT_ASCENDING; gtk_widget_show(col_arrows[column].ascend_pm); - gtk_clist_set_sort_column(clist, column); + gtk_clist_set_sort_column(clist_lcl, column); } - gtk_clist_thaw(clist); + gtk_clist_thaw(clist_lcl); - gtk_clist_sort(clist); + gtk_clist_sort(clist_lcl); } /****************************************************************************/ static gint -mcaststream_sort_column(GtkCList *clist, gconstpointer ptr1, gconstpointer ptr2) +mcaststream_sort_column(GtkCList *clist_lcl, gconstpointer ptr1, gconstpointer ptr2) { char *text1 = NULL; char *text2 = NULL; @@ -296,10 +296,10 @@ mcaststream_sort_column(GtkCList *clist, gconstpointer ptr1, gconstpointer ptr2) const GtkCListRow *row1 = (const GtkCListRow *) ptr1; const GtkCListRow *row2 = (const GtkCListRow *) ptr2; - text1 = GTK_CELL_TEXT (row1->cell[clist->sort_column])->text; - text2 = GTK_CELL_TEXT (row2->cell[clist->sort_column])->text; + text1 = GTK_CELL_TEXT (row1->cell[clist_lcl->sort_column])->text; + text2 = GTK_CELL_TEXT (row2->cell[clist_lcl->sort_column])->text; - switch(clist->sort_column){ + switch(clist_lcl->sort_column){ case 0: case 2: return strcmp (text1, text2); diff --git a/gtk/rtp_analysis.c b/gtk/rtp_analysis.c index 223c031eb6..1d0d4b8ab4 100644 --- a/gtk/rtp_analysis.c +++ b/gtk/rtp_analysis.c @@ -1016,7 +1016,7 @@ static void dialog_graph_draw(user_data_t* user_data) user_data->dlg.dialog_graph.pixmap_width-right_x_border+1, user_data->dlg.dialog_graph.pixmap_height-bottom_y_border); for(i=0;i<=10;i++){ - int xwidth, lwidth; + int xwidth; xwidth=5; if(!(i%5)){ @@ -1122,7 +1122,6 @@ static void dialog_graph_draw(user_data_t* user_data) user_data->dlg.dialog_graph.pixmap_height-bottom_y_border+xlen+1); if(xlen==17){ - int lwidth; if(user_data->dlg.dialog_graph.interval>=1000){ g_snprintf(label_string, sizeof(label_string), "%ds", current_interval/1000); } else if(user_data->dlg.dialog_graph.interval>=100){ @@ -1185,7 +1184,6 @@ static void dialog_graph_draw(user_data_t* user_data) x_pos=draw_width-1-user_data->dlg.dialog_graph.pixels_per_tick*((last_interval-interval)/user_data->dlg.dialog_graph.interval+1)+left_x_border; if(user_data->dlg.dialog_graph.graph[i].items[interval/user_data->dlg.dialog_graph.interval].flags & (STAT_FLAG_WRONG_SEQ|STAT_FLAG_MARKER)){ - int lwidth; if (user_data->dlg.dialog_graph.graph[i].items[interval/user_data->dlg.dialog_graph.interval].flags & STAT_FLAG_WRONG_SEQ){ g_strlcpy(label_string,"x",sizeof(label_string)); } else { diff --git a/gtk/rtp_stream_dlg.c b/gtk/rtp_stream_dlg.c index 4dc95f48e6..4a88e06017 100644 --- a/gtk/rtp_stream_dlg.c +++ b/gtk/rtp_stream_dlg.c @@ -563,17 +563,17 @@ rtpstream_on_analyse (GtkButton *button _U_, /****************************************************************************/ /* when the user selects a row in the stream list */ static void -rtpstream_on_select_row(GtkCList *clist, - gint row _U_, - gint column _U_, - GdkEventButton *event _U_, - gpointer user_data _U_) +rtpstream_on_select_row(GtkCList *clist_lcl, + gint row _U_, + gint column _U_, + GdkEventButton *event _U_, + gpointer user_data _U_) { gchar label_text[80]; /* update the labels */ if (event==NULL || event->state & GDK_SHIFT_MASK) { - selected_stream_rev = gtk_clist_get_row_data(GTK_CLIST(clist), row); + selected_stream_rev = gtk_clist_get_row_data(GTK_CLIST(clist_lcl), row); g_snprintf(label_text, sizeof(label_text), "Reverse: %s:%u -> %s:%u, SSRC=0x%X", get_addr_name(&(selected_stream_rev->src_addr)), selected_stream_rev->src_port, @@ -584,7 +584,7 @@ rtpstream_on_select_row(GtkCList *clist, gtk_label_set_text(GTK_LABEL(label_rev), label_text); } else { - selected_stream_fwd = gtk_clist_get_row_data(GTK_CLIST(clist), row); + selected_stream_fwd = gtk_clist_get_row_data(GTK_CLIST(clist_lcl), row); g_snprintf(label_text, sizeof(label_text), "Forward: %s:%u -> %s:%u, SSRC=0x%X", get_addr_name(&(selected_stream_fwd->src_addr)), selected_stream_fwd->src_port, @@ -614,40 +614,40 @@ typedef struct column_arrows { /****************************************************************************/ static void -rtpstream_click_column_cb(GtkCList *clist, gint column, gpointer data) +rtpstream_click_column_cb(GtkCList *clist_lcl, gint column, gpointer data) { column_arrows *col_arrows = (column_arrows *) data; int i; - gtk_clist_freeze(clist); + gtk_clist_freeze(clist_lcl); for (i=0; i<NUM_COLS; i++) { gtk_widget_hide(col_arrows[i].ascend_pm); gtk_widget_hide(col_arrows[i].descend_pm); } - if (column == clist->sort_column) { - if (clist->sort_type == GTK_SORT_ASCENDING) { - clist->sort_type = GTK_SORT_DESCENDING; + if (column == clist_lcl->sort_column) { + if (clist_lcl->sort_type == GTK_SORT_ASCENDING) { + clist_lcl->sort_type = GTK_SORT_DESCENDING; gtk_widget_show(col_arrows[column].descend_pm); } else { - clist->sort_type = GTK_SORT_ASCENDING; + clist_lcl->sort_type = GTK_SORT_ASCENDING; gtk_widget_show(col_arrows[column].ascend_pm); } } else { - clist->sort_type = GTK_SORT_ASCENDING; + clist_lcl->sort_type = GTK_SORT_ASCENDING; gtk_widget_show(col_arrows[column].ascend_pm); - gtk_clist_set_sort_column(clist, column); + gtk_clist_set_sort_column(clist_lcl, column); } - gtk_clist_thaw(clist); + gtk_clist_thaw(clist_lcl); - gtk_clist_sort(clist); + gtk_clist_sort(clist_lcl); } /****************************************************************************/ static gint -rtpstream_sort_column(GtkCList *clist, gconstpointer ptr1, gconstpointer ptr2) +rtpstream_sort_column(GtkCList *clist_lcl, gconstpointer ptr1, gconstpointer ptr2) { char *text1 = NULL; char *text2 = NULL; @@ -656,10 +656,10 @@ rtpstream_sort_column(GtkCList *clist, gconstpointer ptr1, gconstpointer ptr2) const GtkCListRow *row1 = (const GtkCListRow *) ptr1; const GtkCListRow *row2 = (const GtkCListRow *) ptr2; - text1 = GTK_CELL_TEXT (row1->cell[clist->sort_column])->text; - text2 = GTK_CELL_TEXT (row2->cell[clist->sort_column])->text; + text1 = GTK_CELL_TEXT (row1->cell[clist_lcl->sort_column])->text; + text2 = GTK_CELL_TEXT (row2->cell[clist_lcl->sort_column])->text; - switch(clist->sort_column){ + switch(clist_lcl->sort_column){ case 0: case 2: case 5: |