Wireshark-commits: [Wireshark-commits] master 5e59d50: Qt: Fix profile copy and delete
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 12 Jul 2019 13:24:02 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5e59d50b46a24acff4c631373d62349abd85d70b
Submitter: "Roland Knall <rknall@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

5e59d50 by Roland Knall (rknall@xxxxxxxxx):

    Qt: Fix profile copy and delete
    
    Delete lead to an issue with the sort model, where indeces where
    no longer correctly associated with the source model.
    
    Copy did not check if the name already existed in the dialog
    
    Change-Id: I3668dffb1e1956f04a5b9e21c3fcdac266e0cc0c
    Reviewed-on: https://code.wireshark.org/review/33911
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Roland Knall <rknall@xxxxxxxxx>
    

Actions performed:

    from  57d67f8   LOG3GPP: do not dereference a NULL pointer (CID 1450541)
     add  5e59d50   Qt: Fix profile copy and delete


Summary of changes:
 ui/qt/models/profile_model.cpp      | 66 ++++++++++++++++++++++++++++---------
 ui/qt/models/profile_model.h        |  8 ++---
 ui/qt/profile_dialog.cpp            | 19 ++++++-----
 ui/qt/widgets/profile_tree_view.cpp |  4 ++-
 ui/qt/widgets/profile_tree_view.h   |  4 +++
 5 files changed, 71 insertions(+), 30 deletions(-)