Wireshark-commits: [Wireshark-commits] master-3.0 8b68a77: Fix display of pino protocols in display
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, 12 Nov 2019 08:24:01 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8b68a774fe335357fe02ee0abaa12c70a3a110d3
Submitter: "Pascal Quantin <pascal@xxxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

8b68a77 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>
    Reviewed-on: https://code.wireshark.org/review/35059
    Petri-Dish: Pascal Quantin <pascal@xxxxxxxxxxxxx>
    

Actions performed:

    from  9ca3514   Fix name of the "remove Wireshark from PATH" package.
     add  8b68a77   Fix display of pino protocols in display filter dropdown


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