Wireshark-commits: [Wireshark-commits] master c664f72: Qt: Clear the packet list selection before f
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: Wed, 22 Jun 2016 05:00:33 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c664f72f36ed1ac4e50c020cd3afb1ccf55094c8
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c664f72 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>
    

Actions performed:

    from  debbf7e   Move AT_VINES address type to VINES dissector.
    adds  c664f72   Qt: Clear the packet list selection before freezing it.


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