Rename proto_construct_dfilter_string() to

proto_construct_match_selected_string() to indicate what it does - and
have it return a Boolean indication of whether the string could be
built, returning the string through a pointer, and, if that pointer is
null, have it just return the Boolean and not construct the string.

Get rid of proto_can_match_selected() -
proto_construct_match_selected_string() can be used for that, which
means we have only one piece of code that knows whether a "match
selected" string can be constructed or not.

Have proto_construct_match_selected_string() support matching
zero-length FT_NONE (and FT_PCRE, but that shouldn't happen) fields even
if there's no epan_dissect_t, as such a match just checks whether the
field is present.

svn path=/trunk/; revision=19967

1 files changed, 1 insertions, 2 deletions

diff --git a/tap-protocolinfo.c b/tap-protocolinfo.c
index 484df80905..0c9f8fbfc4 100644
--- a/tap-protocolinfo.c
+++ b/tap-protocolinfo.c
@@ -76,8 +76,7 @@ protocolinfo_packet(void *prs, packet_info *pinfo, epan_dissect_t *edt, const vo
 	}
 
 	for(i=0;i<gp->len;i++){
-		str=proto_construct_dfilter_string(gp->pdata[i], NULL);
-		if(str){
+		if(proto_construct_match_selected_string(gp->pdata[i], NULL, &str)){
 			col_append_fstr(pinfo->cinfo, COL_INFO, "  %s",str);
 		}
 	}