Re: [Wireshark-dev] [Wireshark-commits] rev 37201: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ncp2222.inc

2011-05-17 Thread Jeff Morriss
Jeff Morriss wrote: Maynard, Chris wrote: -Original Message- From: wireshark-commits-boun...@wireshark.org [mailto:wireshark- commits-boun...@wireshark.org] On Behalf Of morr...@wireshark.org Sent: Tuesday, May 17, 2011 12:08 PM To: wireshark-comm...@wireshark.org Subject: [Wireshark-com

Re: [Wireshark-dev] [Wireshark-commits] rev 37201: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ncp2222.inc

2011-05-17 Thread Guy Harris
On May 17, 2011, at 9:35 AM, Maynard, Chris wrote: > What about changing match_strval() to behave like val_to_str() does (i.e., > make IT handle the NULL case), That would require code that uses match_strval() to check whether a given numeric value has a match in the value_string table or not

Re: [Wireshark-dev] [Wireshark-commits] rev 37201: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ncp2222.inc

2011-05-17 Thread Jeff Morriss
Maynard, Chris wrote: -Original Message- From: wireshark-commits-boun...@wireshark.org [mailto:wireshark- commits-boun...@wireshark.org] On Behalf Of morr...@wireshark.org Sent: Tuesday, May 17, 2011 12:08 PM To: wireshark-comm...@wireshark.org Subject: [Wireshark-commits] rev 37201: /tru

Re: [Wireshark-dev] [Wireshark-commits] rev 37201: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-ncp2222.inc

2011-05-17 Thread Maynard, Chris
> -Original Message- > From: wireshark-commits-boun...@wireshark.org [mailto:wireshark- > commits-boun...@wireshark.org] On Behalf Of morr...@wireshark.org > Sent: Tuesday, May 17, 2011 12:08 PM > To: wireshark-comm...@wireshark.org > Subject: [Wireshark-commits] rev 37201: /trunk/epan/diss