Wireshark · Wireshark-commits: [Wireshark-commits] master 7171ee5: [ieee80211] Rework QoS Capability and QoS info code.

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7171ee57021f659bdb5870e7a210cbd164a1be3a
Submitter: Bill Meier (wmeier@xxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

7171ee5 by Bill Meier (wmeier@xxxxxxxxxxx):

    [ieee80211] Rework QoS Capability and QoS info code.
    
    Specifically:
    - Replace/remove much of 'dissect_qos_capability()' code to use
      existing 'fixed field' code to dissect QoS_Info field.
      Note: Much of the code added in g40d6131 to dissect the QoS Info
            field duplicated already existing (but unused) "fixed field"
            code to do same.
    - Rework some QoS Info field filter names, variable names and
      value-strings to better match the text used in 802.11-2012.
    
    - Rename 'dissect_qos_info()' to 'dissect_wme_qos_info()' to
      reflect the fact that this code is only for the (now preumably obsolete)
      original WME(WMM) specification for the QoS Info field.
    
    Change-Id: Id89780dfe60b2e4c63332bdb946cc29c67b5127a
    Reviewed-on: https://code.wireshark.org/review/9903
    Petri-Dish: Bill Meier <wmeier@xxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-by: Bill Meier <wmeier@xxxxxxxxxxx>
    

Actions performed:

    from  f657e6a   [GTPv2] Mark a parameter as unused.
    adds  7171ee5   [ieee80211] Rework QoS Capability and QoS info code.


Summary of changes:
 epan/dissectors/packet-ieee80211.c |  244 +++++++++++-------------------------
 1 file changed, 75 insertions(+), 169 deletions(-)