ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 6192] Change "File -> Save As" to only save the displayed

Date: Tue, 22 May 2012 15:45:48 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6192

Guy Harris <guy@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Platform|Other                       |All

--- Comment #7 from Guy Harris <guy@xxxxxxxxxxxx> 2012-05-22 15:45:48 PDT ---
OK, in the trunk we now have, in the File menu:

    Save As - always attempts to save all packets and, if the save succeeds,
makes the new file the current capture file;

    Export Specified Packets - lets the user specify which packets to write
out, defaults to displayed packets (and those the displayed packets depend on
for reassembly - that's new in the trunk), and never makes the new file the
current capture file.

Having "Save As" not save all packets would make it different from "Save As" in
other applications - it should be a "save all I have" operation, just as "Save"
is.  (Note that "Save" now means "save all the packets *AND* whatever changes
I've made to packet comments", and thus

    1) only asks for a file name if you're saving a temporary file - otherwise
it just saves the file out without asking the user anything

and

    2) can be active even if you've already done a Save, if you've edited
comments since then

making it behave the way "Save" acts in other applications.)

I'm inclined to retitle this to "have the 'save specified packets' operation
default to saving displayed packets" and close it.  "Save As" shouldn't be that
operation.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.