diff options
author | Michael Mann <mmann78@netscape.net> | 2013-09-30 15:21:09 +0000 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2013-09-30 15:21:09 +0000 |
commit | cf804429123802edb55003ebfd21ebea5718d10b (patch) | |
tree | d2a468c4f0aeb706413a6c50c2e12026673a6b34 /epan/dissectors/packet-rtnet.c | |
parent | 2211ab0fba17d71cc25f6b0d026f04d241bbced1 (diff) |
Convert some proto_tree_add_string_format calls to something more appropriate.
There seem to be several cases of proto_tree_add_string_format where a "string" value/filter doesn't really make sense because it's always empty, and is just being used as a "filterable subtree header (placeholder)". They appear to be more for "presense" than "value" and should probably be FT_NONE, although I'd almost argue for removing the filter in favor of proto_tree_add_text.
svn path=/trunk/; revision=52296
Diffstat (limited to 'epan/dissectors/packet-rtnet.c')
-rw-r--r-- | epan/dissectors/packet-rtnet.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-rtnet.c b/epan/dissectors/packet-rtnet.c index 093d280c20..81abc149f8 100644 --- a/epan/dissectors/packet-rtnet.c +++ b/epan/dissectors/packet-rtnet.c @@ -607,8 +607,8 @@ dissect_rtmac(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) { type_str = "Unknown"; } } - proto_tree_add_string_format(rtmac_tree, hf_rtmac_header_type, tvb, offset, 2, - type_str, "Type: %s (0x%04x)", type_str, type); + proto_tree_add_string_format_value(rtmac_tree, hf_rtmac_header_type, tvb, offset, 2, + type_str, "%s (0x%04x)", type_str, type); offset += 2; proto_tree_add_item(rtmac_tree, hf_rtmac_header_ver, tvb, offset, 1, ENC_BIG_ENDIAN); |