Wireshark-commits: [Wireshark-commits] master 99398c5: Fix display of pino protocols in display fil

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 12 Nov 2019 06:20:46 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=99398c533ece117d84c15cad54b7fc94e1ca5341
Submitter: "Pascal Quantin <pascal@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

99398c5 by Pascal Quantin (pascal@xxxxxxxxxxxxx):

    Fix display of pino protocols in display filter dropdown
    
    Qt GUI uses proto_get_first_protocol() to find the list of protocols
    and build the autocompletion list. As pinfo protocols are stored in
    another list, they are kept aside.
    Let's add them in the same list as normal protocols.
    
    Bug: 16130
    Change-Id: I9ff67ea4198a8cc6baf3ded584c48eadfb097092
    Reviewed-on: https://code.wireshark.org/review/34778
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    

Actions performed:

    from  a13d168   Fix name of the "remove Wireshark from PATH" package.
     add  99398c5   Fix display of pino protocols in display filter dropdown


Summary of changes:
 epan/proto.c | 26 ++++++++++----------------
 1 file changed, 10 insertions(+), 16 deletions(-)