Wireshark-dev: Re: [Wireshark-dev] Change in wireshark[master]: Add support for BASE_VALS_NO_UN
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Guy Harris <guy@xxxxxxxxxxxx>
Date: Mon, 10 Apr 2017 15:19:47 -0700
On Apr 10, 2017, at 1:31 PM, Michael Mann (Code Review) <code-review-do-not-reply@xxxxxxxxxxxxx> wrote:

> Michael Mann has submitted this change and it was merged.
> 
> Change subject: Add support for BASE_VALS_NO_UNKNOWN
> ......................................................................
> 
> 
> Add support for BASE_VALS_NO_UNKNOWN
> 
> BASE_VALS_NO_UNKNOWN is a special value_string value for only a single
> (maybe 2) numerical value(s).  If a field has the numerical value
> that doesn't match anything in the value_string, just the number
> is supplied for the field (no "Unknown")
> 
> Dissectors that had this use case have been converted in the patch.

The use case for this appears to be for fields that normally have a numerical value, and where particular numerical values have no special significance, but where certain numerical values have a special meaning - for example, a field that's normally a timeout in milliseconds, but where a value of 0 means "time out immediately" and a value of 0xFFFFFFFF means "never time out".

Would a more descriptive name for it be something such as BASE_SPECIAL_VALS?