Wireshark · Wireshark-commits: [Wireshark-commits] master 289ca20: Update documentation of hf fields.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=289ca20fe1e80ee140434f682e85ab64054e17c6
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

289ca20 by Jeff Morriss (jeff.morriss@xxxxxxxxxxxx):

    Update documentation of hf fields.
    
    - Specify the valid characters in hf abbreviations as suggested in
      https://ask.wireshark.org/questions/50444/braces-inside-abbreviated-name-fieldabbrev-of-header_field_info
    - Update the valid characters for protocol abbreviations too.
    - Remove a couple old (ancient?) hf substitutions (things to replace in the
      dissector template).  I don't think PARENT_SUBFIELD or ID_VALUE have been
      used in quite a while.
    - We no longer automatically add the protocol's abbreviation to the field's
      abbreviation (it's now the dissector-writer's job).
    - Abbreviations can no longer be empty strings (since
      a146f5a2e211aa414cba98ce0b0503a690695d34).
    - When talking about hf fields reference the substitution names (to make it
      easier to find additional documentation).
    
    Change-Id: Ic80dc6a230dc727ba544e68c4a0cc746768e5081
    Reviewed-on: https://code.wireshark.org/review/14107
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  0165e67   u3v: use interface class and subclass in heuristic checks
    adds  289ca20   Update documentation of hf fields.


Summary of changes:
 doc/README.dissector |   62 ++++++++++++++++++++++++--------------------------
 1 file changed, 30 insertions(+), 32 deletions(-)