Wireshark-bugs: [Wireshark-bugs] [Bug 9472] ANSI IS-637 SMS enhancements for CMAS (TIA-1149) and
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Thu, 19 Dec 2013 17:01:00 +0000

changed bug 9472

What Removed Added
Attachment #12350 Flags review_for_checkin? review_for_checkin-

Comment # 7 on bug 9472 from 
Comment on attachment 12350 [details]
Changes for CMAS and better text display

1. See r54244, I believe you want to follow this convention and not use
g_convert_with_iconv (looks like you missed "free maintenance" by a day)

2. has_private_data isn't used, yet it's passed through a lot of functions. 
Justification?

3. If you use proto_tree_add_uint_format_value instead of
proto_tree_add_uint_format on an hf_ field with a non-zero bitmask, you get the
"bitstring" for free (and don't need other_decode_bitfield_value).  There are
stil several places this could be applied.

4. Can you explain (for my own curiosity) the difference between "Number of
fields (MSB)" and "Number of fields (LSB)".  why are they separate fields with
different bitmasks?  Wouldn't a single 16-bit bitmask and ENC_LITTLE_ENDIAN or
ENC_BIG_ENDIAN be simpler?

You are receiving this mail because:
  • You are watching all bug changes.