Wireshark-commits: [Wireshark-commits] master 6637f4d: Fix "Save All" for Export Objects.
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 17 Mar 2015 18:47:35 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=6637f4d144c96a4bb467540c46bb47779d2be573
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

6637f4d by Guy Harris (guy@xxxxxxxxxxxx):

    Fix "Save All" for Export Objects.
    
    Fix the loop that goes through all the objects to be saved so that it
    looks at all the objects to be saved rather than repeatedly looking at
    the first object.
    
    When letting the user select a directory in which to save, restrict them
    to looking at directories.
    
    QDir doesn't do what we actually want here - QDir.setCurrent() might do
    a chdir() but it doesn't appear to set the QDir's path, so when we use
    it to construct pathnames it constructs them based on
    wsApp->lastOpenDir() rather than the directory the user selected.  Use
    Boring Old GLib's g_build_filename(), as we do in the GTK+ version, for
    now.  (It also does something we *don't* want, namely, at least as the
    documentation implies, a chdir().)
    
    Bug: 11062
    Change-Id: I705292dc2713b9dba615b0d59a4cf8313f357d82
    Reviewed-on: https://code.wireshark.org/review/7720
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  f59ad07   MIP6: Add Localized Routing for Proxy Mobile IPv6 (RFC6705)
    adds  6637f4d   Fix "Save All" for Export Objects.


Summary of changes:
 ui/qt/export_object_dialog.cpp |   45 ++++++++++++++++++++++++++--------------
 1 file changed, 29 insertions(+), 16 deletions(-)