Wireshark-commits: [Wireshark-commits] master ebcfc1b: Improve the display of metrics.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 1 Apr 2017 06:34:12 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ebcfc1ba8e85bd37b03f8ae5fd2f48ce07239bca
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ebcfc1b by Guy Harris (guy@xxxxxxxxxxxx):

    Improve the display of metrics.
    
    Give the value and the "supported" bit names that reflect what they are.
    
    Show both the "supported" bit and the value as bitfields, because that's
    what they are.
    
    Show the "supported" bit before the value.
    
    Show the metric value as decimal; there's nothing particularly
    hexadecimal about it.
    
    The "supported" bit is 1 if it's *not* supported and 0 if it *is*
    supported.
    
    Fetch the byte for the metric within dissect_metric(), rather than in
    the call to dissect_metric().
    
    Change-Id: Ief3bb74b273df06e07066ccdede38a2eeedc6db2
    Reviewed-on: https://code.wireshark.org/review/20836
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  aa82f52   You can't get a string value from an FT_NONE string.
    adds  ebcfc1b   Improve the display of metrics.


Summary of changes:
 epan/dissectors/packet-isis-lsp.c |   78 +++++++++++++++++++++----------------
 1 file changed, 45 insertions(+), 33 deletions(-)