Wireshark-commits: [Wireshark-commits] master-2.4 8737448: Qt: Write recent before applying profile
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, 31 Jan 2018 11:49:37 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=873744861286b93f4b42686f388e9568b81ca9d9
Submitter: Stig Bjørlykke (stig@xxxxxxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

8737448 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    Qt: Write recent before applying profile changes
    
    The selected profile may be renamed in apply_profile_changes()
    so write the recent file before renaming the directory to ensure that
    we reload the latest changes.
    
    Change-Id: I8988a00647926d93b0a1903090aadc8c61f1a34e
    Reviewed-on: https://code.wireshark.org/review/25516
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 55e74ba841126797b2a5b82142fb1db8f61a1288)
    Reviewed-on: https://code.wireshark.org/review/25523
    Reviewed-by: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    

Actions performed:

    from  782b3db   UI: Rename profiles without delete
    adds  8737448   Qt: Write recent before applying profile changes


Summary of changes:
 ui/qt/profile_dialog.cpp        | 14 ++++++++++++--
 ui/qt/wireshark_application.cpp |  3 +++
 ui/qt/wireshark_application.h   |  1 +
 3 files changed, 16 insertions(+), 2 deletions(-)