Comment # 6
on bug 11419
from Guy Harris
(In reply to Alexis La Goutte from comment #5)
> I think it is a little endian... (Like rest of 802.11 dissector...)
> But fix encoding arg don't generate a warning for this case ?
Apparently it didn't.
The comment at the beginning of that script says
#
# A program to fix encoding args for certain Wireshark API function calls
# from TRUE/FALSE to ENC_?? as appropriate (and possible)
# - proto_tree_add_item
# - proto_tree_add_bits_item
# - proto_tree_add_bits_ret_val
# - proto_tree_add_bitmask
# - proto_tree_add_bitmask_text !! ToDo: encoding arg not last arg
# - tvb_get_bits
# - tvb_get_bits16
# - tvb_get_bits24
# - tvb_get_bits32
# - tvb_get_bits64
# - ptvcursor_add
# - ptvcursor_add_no_advance
# - ptvcursor_add_with_subtree !! ToDo: encoding arg not last arg
#
# ToDo: Rework program so that it can better be used to *validate*
encoding-args
so it sounds as if it's not yet an API checker (or that the comment is out of
date).
You are receiving this mail because:
- You are watching all bug changes.