> Actually, there shouldn't *be* a "file_sel" variable; there should be
> separate variables for the current "File->Open" and "File->Save {As}"
> dialog boxes. I've checked in a change to do that.
...although we also have to arrange either to
1) nuke any open "File->Save {As}" dialog box in
"close_cap_file()"
or
2) gray out the "File->Open" and "File->Close" menu items (and
any other menu items that would cause the current capture
file to be closed) if there's currently a "File->Save {As}"
dialog box open
(this isn't a new problem).
That also raises another question:
Should we nag the user with a dialog box if they try to close (by
opening a new file, starting a new capture, closing a capture file,
exiting Ethereal, etc.) a live capture that they haven't saved?
Network Monitor does it, but, whilst it could be nice if you forgot to
save a capture you *are* interested in, it could just be a pain if
you're simply not interested in the current capture.