Wireshark-commits: [Wireshark-commits] rev 42792: /trunk/ /trunk/ui/gtk/: capture_dlg.c capture_fil

Date: Tue, 22 May 2012 22:17:57 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=42792

User: guy
Date: 2012/05/22 03:17 PM

Log:
 "Save As" always saves everything and, when the save is done, makes the
 new file the current file, as is the case in most if not all other GUI
 applications.
 
 A new "Export Specified Packets" menu option allows you to specify which
 packets to write out, with the default being the displayed packets (and
 those on which the displayed packets depend for, e.g.  reassembly), and
 never makes the resulting file the current file.
 
 The two operations are conceptually distinct.  Lumping them into one
 menu item, with the default for "Save As" being "displayed packets only"
 and thus making it behave like the latter operation, was causing some
 confusion; see, for example, bug 6640.
 
 Make the dialog popped up if you try to "Save As" or "Export Specified
 Packets" on top of an existing file ask the "do you want to do this?"
 question in the main part of the message, and note in the secondary text
 that doing that will overwrite what's in the file; that matches what
 TextEdit on OS X and the GNOME text editor say.

Directory: /trunk/ui/gtk/
  Changes    Path                  Action
  +1 -1      capture_dlg.c         Modified
  +299 -59   capture_file_dlg.c    Modified
  +21 -11    capture_file_dlg.h    Modified
  +1 -1      drag_and_drop.c       Modified
  +1 -1      file_import_dlg.c     Modified
  +3 -2      main.c                Modified
  +14 -6     main_menubar.c        Modified
  +1 -1      new_packet_list.c     Modified

Directory: /trunk/
  Changes    Path          Action
  +139 -68   file.c        Modified
  +20 -4     file.h        Modified