Wireshark-commits: [Wireshark-commits] master 3ff0a55: Qt: Check if "Apply as Column" column alread
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 15 Jul 2020 03:57:14 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=3ff0a55d5c77702e946195efb5ea5253f6482032
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

3ff0a55 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    Qt: Check if "Apply as Column" column already exists
    
    Use the existing (possible hidden) column when doing "Apply as Column"
    on a field which is already used as a custom column. This will help
    prevent having multiple equal custom columns, where all will be hidden
    at startup and profile change when only one of them are configured as
    hidden.
    
    Multiple equal columns can always be manually configured using
    "Preferences -> Appearance -> Columns" if this is intended.
    
    Change-Id: Ib03893facfa3f194f3b3303645fb3f9313ec9e91
    Reviewed-on: https://code.wireshark.org/review/37861
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  6037b40   FTDI FT: Add RX payload to tree only when present
     add  3ff0a55   Qt: Check if "Apply as Column" column already exists


Summary of changes:
 ui/preference_utils.c       | 22 ++++++++++++++++++++++
 ui/preference_utils.h       |  8 ++++++++
 ui/qt/main_window_slots.cpp | 21 +++++++++++++++++++--
 3 files changed, 49 insertions(+), 2 deletions(-)