Allow the user to save either all of the current capture, or only the

packets that are currently being displayed from that capture.

Centralize the code to control whether "File:Save" and "File:Save As"
are enabled (and *always* have "File:Save As" enabled if you have a
capture; "File:Save" is enabled only if you have a live capture you've
not yet saved, although it does the same thing as "File:Save As").

Have the "save_file" member of a "capture_file" structure represent
*only* the file currently being *written* to by a capture, and, if there
is no capture currently in progress, have it be NULL; the name of the
file currently being *displayed" is in the "filename" member, and an
"is_tempfile" member indicates whether it's a temporary file for a live
capture or not.

Have "close_cap_file()" delete the current capture file if it's a
temporary capture file that hasn't been saved (in its entirety - saving
selected frames doesn't count).  Do the same (if there *is* a current
capture file) when exiting.

The "Ready to load or capture" message is the only statusbar message in
the "main" context; "close_cap_file()" should never pop it, it should
only pop whatever message exists in the "file" context, and thus has no
need to take, as an argument, the context for the message it should pop.

Update the man page to reflect the new behavior of "File:Save" and
"File:Save As", and to reflect recent changes to "Display:Match Selected".

svn path=/trunk/; revision=1170

1 files changed, 12 insertions, 3 deletions

diff --git a/doc/ethereal.pod.template b/doc/ethereal.pod.template
index 52e7a2c994..c376c104e0 100644
--- a/doc/ethereal.pod.template
+++ b/doc/ethereal.pod.template
@@ -177,6 +177,12 @@ allows a filter to be specified; when the capture file is read, the
 filter is applied to all packets read from the file, and packets not
 matching the filter are discarded.
 
+=item File:Save, File:Save As
+
+Save the current capture, or the packets currently displayed from that
+capture, to a file.  A check box lets you select whether to save all
+packets, or just those that have passed the current display filter.
+
 =item File:Print
 
 Prints, for all the packets in the current capture, the packet number,
@@ -217,9 +223,12 @@ live capture is in progress.
 =item Display:Match Selected
 
 Creates and applies a display filter based on the data that is currently
-highlighted in the protocol tree. The display filter is based on absolute
-offset within the packet, so could be unreliable if the packet contains
-protocols with variable-length headers, like source-routed token-ring.
+highlighted in the protocol tree.  If that data is a field that can be
+tested in a display filter expression, the display filter will test that
+field; otherwise, the display filter will be based on absolute offset
+within the packet, and so could be unreliable if the packet contains
+protocols with variable-length headers, such as a source-routed
+token-ring packet.
 
 =item Display:Colorize Display