Wireshark-commits: [Wireshark-commits] master 4461f72: Add special handling of the (derived) AppId

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 22 Apr 2015 05:51:51 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4461f7271d3cfb6941c3d9ad84d3da1892697cc3
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

4461f72 by Jeff Morriss (jeff.morriss.ws@xxxxxxxxx):

    Add special handling of the (derived) AppId AVP type so that we can maintain
    one (and only one) Application ID list.
    
    This means we don't have to list all the Application IDs several times in the
    XML file.  Collapse those lists into one (much more complete, now) list while
    also fixing URIs for several of the specifications (and making it clear for
    others that we don't have access to the specification--by declaring their
    URI to be "none").
    
    Add a bunch more entries to the list of application IDs.
    
    Change-Id: Ia5c96b1f6f1fe3a9521b3d70142889e3881fae5e
    Reviewed-on: https://code.wireshark.org/review/8147
    Petri-Dish: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  ecc3369   Misc. Diameter XML cleanups.
    adds  4461f72   Add special handling of the (derived) AppId AVP type so that we can maintain one (and only one) Application ID list.


Summary of changes:
 diameter/VerizonWireless.xml      |    2 +-
 diameter/dictionary.xml           |  422 +++++++++++++------------------------
 epan/dissectors/packet-diameter.c |   38 +++-
 3 files changed, 180 insertions(+), 282 deletions(-)