Wireshark · Wireshark-commits: [Wireshark-commits] rev 21890: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-dnp.c

[guy@xxxxxxxxxxxxx]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=21890

User: guy
Date: 2007/05/22 05:32 PM

Log:
 With at least some compilers, "enum QUALITY_TYPE" is unsigned, and the
 compiler warns about checking whether it's >= 0, as it always is.
 
 Don't do that check - we don't use "type" until the switch statement,
 and GCC should issue a warning if not all valid "enum QUALITY_TYPE"
 values are handled in the switch statement.
 
 Sort the items in the switch statement in the order in which they appear
 in the enum definition, so it's clearer that it covers all values.

Directory: /trunk/epan/dissectors/
  Changes    Path            Action
  +14 -15    packet-dnp.c    Modified