Wireshark-commits: [Wireshark-commits] master f410121: Remove some unnecessary dependencies and cle
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: Thu, 7 Jan 2016 01:09:45 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f410121cf2a94ef06fa42863352a77ba42a5fdf7
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f410121 by João Valverde (joao.valverde@xxxxxxxxxxxxxxxxxx):

    Remove some unnecessary dependencies and cleanup some code
    
    No need for the GTK dependency on dissectors/packet-ipv6.h.
    
    Add the stream_addr typedef in follow.h to make some code simpler.
    
    Change-Id: I1cf906f58734a90263141362f2da33a140f93533
    Reviewed-on: https://code.wireshark.org/review/13063
    Reviewed-by: João Valverde <j@xxxxxx>
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  7888c43   [docsis->INIT-RNG-REQ] Correction for INFO field
    adds  f410121   Remove some unnecessary dependencies and cleanup some code


Summary of changes:
 epan/follow.c                  |   30 ++++++++++++++----------------
 epan/follow.h                  |   18 +++++++++++-------
 ui/cli/tap-follow.c            |   18 +++++++++---------
 ui/gtk/firewall_dlg.c          |    1 -
 ui/gtk/follow_http.c           |   18 ++++++------------
 ui/gtk/follow_ssl.c            |   23 ++++++++---------------
 ui/gtk/follow_tcp.c            |   22 ++++++++--------------
 ui/gtk/follow_udp.c            |   18 ++++++------------
 ui/qt/follow_stream_dialog.cpp |   18 ++++++------------
 9 files changed, 68 insertions(+), 98 deletions(-)