Wireshark-commits: [Wireshark-commits] master 79f7edb: Qt: Don't expose ColorUtils::graph_colors_.
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, 1 Dec 2015 05:18:18 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=79f7edba15dc4c637c7f19c0770c2ba2721d765a
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

79f7edb by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Qt: Don't expose ColorUtils::graph_colors_.
    
    Make graph_colors_ private and accessible via getters. Blind attempt at
    fixing bug 11833.
    
    Bug: 11833
    Change-Id: I03b7e90c686374d2d0f046f7e5fe87e43939dc82
    Reviewed-on: https://code.wireshark.org/review/12318
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  e3fc691   [MP2T] Prevent divide by zero.
    adds  79f7edb   Qt: Don't expose ColorUtils::graph_colors_.


Summary of changes:
 ui/qt/color_utils.h            |    6 +++++-
 ui/qt/iax2_analysis_dialog.cpp |    4 ++--
 ui/qt/io_graph_dialog.cpp      |    9 +++------
 ui/qt/io_graph_dialog.h        |    6 +++++-
 ui/qt/rtp_analysis_dialog.cpp  |    4 ++--
 ui/qt/rtp_player_dialog.cpp    |    2 +-
 6 files changed, 18 insertions(+), 13 deletions(-)