Wireshark · Wireshark-commits: [Wireshark-commits] master f274902: Qt: Add a play button to the RTP Stream Analysis dialog.

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f274902be581b52db68c66f4864d85b7ea8fefe4
Submitter: Gerald Combs (gerald@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f274902 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Qt: Add a play button to the RTP Stream Analysis dialog.
    
    Rename the "Play Call" button to "Play Streams". Move the button
    creation code to a common routine. Use it to add a "Play Streams" button
    to the RTP Stream Analysis, similar to the GTK+ UI.
    
    Don't restrict RTP to IPv[46] as suggested by Michal. I don't have any
    RTP-over-Bluetooth captures so I can't test this directly.
    
    Change-Id: I4703cac1d5bf5b3ff0255d36da2c5164feb0547d
    Reviewed-on: https://code.wireshark.org/review/10888
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    

Actions performed:

    from  03d8534   GTPv2: display Sequence Number as BASE_HEX_DEC
    adds  f274902   Qt: Add a play button to the RTP Stream Analysis dialog.


Summary of changes:
 ui/qt/decode_as_dialog.ui     |    3 ---
 ui/qt/rtp_analysis_dialog.cpp |   47 +++++++++++++++++++++++++++++++++++------
 ui/qt/rtp_analysis_dialog.h   |    8 +++++++
 ui/qt/rtp_player_dialog.cpp   |   10 +++++++++
 ui/qt/rtp_player_dialog.h     |    9 ++++++++
 ui/qt/voip_calls_dialog.cpp   |    3 +--
 6 files changed, 69 insertions(+), 11 deletions(-)