Wireshark-commits: [Wireshark-commits] master-2.0 bbed186: Qt: Clear the packet list selection befo
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 22 Jun 2016 18:37:23 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bbed18623328ec8fcf6be7254d36d6525737a8d8
Submitter: Gerald Combs (gerald@xxxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

bbed186 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Qt: Clear the packet list selection before freezing it.
    
    Call selectionModel()->clear() before removing our model. This will
    hopefully make sure we don't end up with an invalid selection in
    selectionChanged().
    
    Clear the selection model before clearing the model while we're here.
    
    Change-Id: I1007eeaa480afa0fb31dafc15a49ca6c247b30c8
    Reviewed-on: https://code.wireshark.org/review/16062
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Kenny Root <kenny@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit c664f72f36ed1ac4e50c020cd3afb1ccf55094c8)
    Reviewed-on: https://code.wireshark.org/review/16074
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  ea60c82   Qt: Make the syntax line edit less greedy.
    adds  bbed186   Qt: Clear the packet list selection before freezing it.


Summary of changes:
 ui/qt/packet_list.cpp |    2 ++
 1 file changed, 2 insertions(+)