Wireshark · Wireshark-commits: [Wireshark-commits] master-2.4 9e1f588: LISP: Switch to proto_tree_add_subtree_format()

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9e1f588c162cac7993f85c99c40a3d253e5cbcbb
Submitter: "Pascal Quantin <pascal.quantin@xxxxxxxxx>"
Changed: branch: master-2.4
Repository: wireshark

Commits:

9e1f588 by Lorand Jakab (ljakab@xxxxxxxxxx):

    LISP: Switch to proto_tree_add_subtree_format()
    
    When the LISP dissector was initially written, it followed a different
    pattern for adding subtrees with a text label, which needs to be
    modified while dissecting: proto_tree_add_item() +
    proto_item_append_text() + proto_item_add_subtree().
    
    This commit updates the code to use the more elegant
    proto_tree_add_subtree_format() call.
    
    Change-Id: Icb6424be3c9cdecbfe9bb5aa2d39f3ad3d1499e0
    Signed-off-by: Lorand Jakab <ljakab@xxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/30655
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/30703
    

Actions performed:

    from  dfc2b27   SSL: Send the record_length to the dissect_ssl3_alert and use it
     add  9e1f588   LISP: Switch to proto_tree_add_subtree_format()


Summary of changes:
 epan/dissectors/packet-lisp.c | 35 ++++++-----------------------------
 1 file changed, 6 insertions(+), 29 deletions(-)