diff options
author | Michael Mann <mmann78@netscape.net> | 2015-09-11 19:38:05 -0400 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2015-09-13 00:15:35 +0000 |
commit | 648cf995b108ae2cb6df83e1144b32f34ffb8028 (patch) | |
tree | d56f6e5f8e9699ba63059f6318a17276c34997f5 /epan/dissectors/packet-ansi_a.c | |
parent | e4c0e510b8468861ec03f29859bf29c5c15edb99 (diff) |
Make other_decode_bitfield_value private to proto.c.
This is further encouragement to not try to manually create a bitstring while formatting a field.
Change-Id: I4efbeb39a210cf1fd26203cd8560859276b333b0
Reviewed-on: https://code.wireshark.org/review/10494
Petri-Dish: Michael Mann <mmann78@netscape.net>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-ansi_a.c')
-rw-r--r-- | epan/dissectors/packet-ansi_a.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/epan/dissectors/packet-ansi_a.c b/epan/dissectors/packet-ansi_a.c index 3e7d555594..d157f90c00 100644 --- a/epan/dissectors/packet-ansi_a.c +++ b/epan/dissectors/packet-ansi_a.c @@ -1725,22 +1725,13 @@ content_fill_aux( int hf_content, int hf_content_fill_bits) { - static guint8 lo_masks[8] = { 0x01, 0x03, 0x07, 0x0f, 0x1f, 0x3f, 0x7f, 0xff }; - guint8 oct; - proto_tree_add_item(tree, hf_content, tvb, offset, content_len, ENC_NA); offset += content_len; if (fill_bits) { - oct = tvb_get_guint8(tvb, offset - 1); - - other_decode_bitfield_value(a_bigbuf, oct, lo_masks[fill_bits-1], 8); - proto_tree_add_uint_format(tree, hf_content_fill_bits, tvb, offset - 1, 1, - oct & lo_masks[fill_bits-1], - "%s = Fill Bits", - a_bigbuf); + proto_tree_add_bits_item(tree, hf_content_fill_bits, tvb, (offset - 1)*8, fill_bits-1, ENC_NA); } } |