diff options
author | Anders Broman <anders.broman@ericsson.com> | 2007-10-23 05:50:00 +0000 |
---|---|---|
committer | Anders Broman <anders.broman@ericsson.com> | 2007-10-23 05:50:00 +0000 |
commit | 1950ffc2141ffdb7a0b8ec8291b0bf99ac12d9d6 (patch) | |
tree | a3358964f8621efae38bb4df4903f6b9f3f46ea5 /epan/dissectors/packet-h1.c | |
parent | e78cce5fbd7a11b0d3122b6d64968c15ea3106f8 (diff) |
Apply the small performance enhancment patches for:
- if offset is 0, tvb_length is the same as tvb_length_remaining, just faster.
Replace
- col_append_fstr() with faster col_append_str()
- col_add_str() with col_set_str()
when it's safe
svn path=/trunk/; revision=23252
Diffstat (limited to 'epan/dissectors/packet-h1.c')
-rw-r--r-- | epan/dissectors/packet-h1.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-h1.c b/epan/dissectors/packet-h1.c index f0a77819f0..c088f83fe6 100644 --- a/epan/dissectors/packet-h1.c +++ b/epan/dissectors/packet-h1.c @@ -110,7 +110,7 @@ static gboolean dissect_h1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) unsigned int position = 3; unsigned int offset=0; - if (tvb_length_remaining(tvb, 0) < 2) + if (tvb_length(tvb) < 2) { /* Not enough data captured to hold the "S5" header; don't try to interpret it as H1. */ @@ -125,7 +125,7 @@ static gboolean dissect_h1(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) if (check_col (pinfo->cinfo, COL_PROTOCOL)) col_set_str (pinfo->cinfo, COL_PROTOCOL, "H1"); if (check_col (pinfo->cinfo, COL_INFO)) - col_add_str (pinfo->cinfo, COL_INFO, "S5: "); + col_set_str (pinfo->cinfo, COL_INFO, "S5: "); if (tree) { ti = proto_tree_add_item (tree, proto_h1, tvb, offset, 16, FALSE); |