Wireshark-commits: [Wireshark-commits] master 8d599e1: dissectors: use '/' when stacking protocols
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: Tue, 29 Jan 2019 21:49:18 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8d599e1952c5495aaeeaf20af9b59d406f3034a6
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

8d599e1 by Dario Lombardo (lomato@xxxxxxxxx):

    dissectors: use '/' when stacking protocols in COL_PROTOCOL.
    
    A commoly used convention when adding more then 1 protocols in
    COL_PROTOCOL (using col_set_fence) is to separate them using the
    '/' character. Some dissectors use ' ', others use '|'. Make them
    all use '/'.
    
    Change-Id: Ibcddd7500f637d96313b264122d48ac6bff1e96c
    Reviewed-on: https://code.wireshark.org/review/31804
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  30f8175   column-utils: replace TAB with SPACE in prototypes.
     add  8d599e1   dissectors: use '/' when stacking protocols in COL_PROTOCOL.


Summary of changes:
 epan/dissectors/packet-imf.c  | 2 +-
 epan/dissectors/packet-mbim.c | 2 +-
 epan/dissectors/packet-tte.c  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)