Have tap listeners specify whether the "packet" routine requires

	a protocol tree;

	the column values.

This includes stats-tree listeners.

Have the routines to build the packet list, and to retap packets, honor
those requirements.  This means that cf_retap_packets() no longer needs
an argument to specify whether to construct the column values or not, so
get rid of that argument.

This also means that there's no need for a tap to have a fake filter
to ensure that the protocol tree will be built, so don't set up a fake
"frame" filter.

While we're at it, clean up some cases where "no filter" was represented
as a null string rather than a null pointer.

Have a routine to return an indication of the number of tap listeners
with filters; use that rather than the global num_tap_filters.

Clean up some indentation and some gboolean vs. gint items.


git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@28645 f5534014-38df-0310-8fa8-9805f1628bb7

1 files changed, 2 insertions, 2 deletions

diff --git a/gtk/sctp_assoc_analyse.c b/gtk/sctp_assoc_analyse.c
index fce1b3fded..e94b2eac19 100644
--- a/gtk/sctp_assoc_analyse.c
+++ b/gtk/sctp_assoc_analyse.c
@@ -931,7 +931,7 @@ struct sctp_analyse * u_data;
 	u_data->analyse_nb   = NULL;
 	u_data->window       = NULL;
 	u_data->num_children = 0;
-	cf_retap_packets(&cfile, FALSE);
+	cf_retap_packets(&cfile);
 	sctp_analyse_cb(u_data, TRUE);
 	sctp_set_filter(NULL, u_data);
 }
@@ -954,7 +954,7 @@ void sctp_analyse_start(GtkWidget *w _U_, gpointer data _U_)
 	u_data->window       = NULL;
 	u_data->num_children = 0;
 
-	cf_retap_packets(&cfile, FALSE);
+	cf_retap_packets(&cfile);
 	sctp_analyse_cb(u_data, FALSE);
 }