Wireshark-commits: [Wireshark-commits] rev 40157: /trunk/epan/dissectors/ /trunk/epan/dissectors/:

Date: Mon, 12 Dec 2011 07:24:59 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=40157

User: etxrab
Date: 2011/12/11 11:24 PM

Log:
 From Mike Morrin:
 eliminates the global variable for tracking which nibble is
 to be decoded by taking advantage of the fact that half octet IEs always occur
 in pairs, and thus a pair can be grouped together for decoding.
 
 There was probably also some confusion caused by the macros UPPER_NIBBLE and
 LOWER_NIBBLE because the GSM bit numbering is opposite to Wireshark internal
 numbering, so I have changed these to be LEFT_NIBBLE and RIGHT_NIBBLE, which
 corresponds to the display format in Wireshark.
 
 The dissection order of half octet IEs has been adjusted where necessary to
 align with the ordering shown in the GSM specifications.
 
 https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6658

Directory: /trunk/epan/dissectors/
  Changes    Path                     Action
  +29 -42    packet-gsm_a_common.c    Modified
  +16 -24    packet-gsm_a_common.h    Modified
  +1 -2      packet-gsm_a_dtap.c      Modified
  +15 -32    packet-gsm_a_rr.c        Modified