diff options
author | Guy Harris <guy@alum.mit.edu> | 2004-01-31 03:22:42 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2004-01-31 03:22:42 +0000 |
commit | 1980fa5dbf6e3aa953d8bd09d69e0700b74a56ad (patch) | |
tree | 1dda390f197eb0fd4dc3bf2e9b4e5b36a8162189 /gtk/proto_dlg.c | |
parent | 61e26a56bada54f651cb5c4110cae4b8f6548863 (diff) |
Pass ESD_BTN_OK rather than NULL as a second argument to
"simple_dialog()"; NULL might be #defined to be a pointer expression on
some platforms, causing compiler warnings (and, on platforms where a
null pointer doesn't have all its bits 0, possibly causing misbehavior,
although I don't think there are any such platforms on which Ethereal
runs).
Don't allow 0 as button mask argument to "simple_dialog()".
Squelch a compiler warning.
Report fatal problems as errors, not warnings.
Report file I/O errors with "file_open_error_message()".
Report file write errors (including those reported by "close()", e.g.
some errors writing to an NFS server) when saving raw packet data to a
file.
svn path=/trunk/; revision=9915
Diffstat (limited to 'gtk/proto_dlg.c')
-rw-r--r-- | gtk/proto_dlg.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gtk/proto_dlg.c b/gtk/proto_dlg.c index 3f7fa24ea5..bca706f678 100644 --- a/gtk/proto_dlg.c +++ b/gtk/proto_dlg.c @@ -1,6 +1,6 @@ /* proto_dlg.c * - * $Id: proto_dlg.c,v 1.30 2004/01/21 21:19:33 ulfl Exp $ + * $Id: proto_dlg.c,v 1.31 2004/01/31 03:22:41 guy Exp $ * * Laurent Deniel <laurent.deniel@free.fr> * @@ -426,7 +426,7 @@ proto_save_cb(GtkWidget *save_bt _U_, gpointer parent_w) /* Create the directory that holds personal configuration files, if necessary. */ if (create_persconffile_dir(&pf_dir_path) == -1) { - simple_dialog(ESD_TYPE_WARN, NULL, + simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "Can't create directory\n\"%s\"\nfor disabled protocols file: %s.", pf_dir_path, strerror(errno)); g_free(pf_dir_path); @@ -438,7 +438,7 @@ proto_save_cb(GtkWidget *save_bt _U_, gpointer parent_w) save_disabled_protos_list(&pf_path, &pf_save_errno); if (pf_path != NULL) { - simple_dialog(ESD_TYPE_WARN, NULL, + simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "Could not save to your disabled protocols file\n\"%s\": %s.", pf_path, strerror(pf_save_errno)); g_free(pf_path); |