diff options
| author | Ahmad Fatoum <ahmad@a3f.at> | 2017-04-14 11:33:57 +0200 |
|---|---|---|
| committer | Peter Wu <peter@lekensteyn.nl> | 2017-04-17 10:51:10 +0000 |
| commit | b478df61f54f08e0279d83e2cc11313fbdf28758 (patch) | |
| tree | 99585c43e985315c471038eb5a95e3368e9e4a73 /epan/dissectors/packet-snort.c | |
| parent | f63ad23ef9036a60e78e5efd45936aae1705d5ac (diff) | |
Qt: Provide both file save and open preferences
This is a breaking change.
prefs_register_filename_preference hasn't been differentiating
between files to be saved and ones to be opened.
On GTK, a neutral dialog is used, so no problems there.
On Qt, a save dialog has been always used, even in dissectors that
were reading configuration files without modification.
prefs_register_filename_preference now takes an argument to indicate
whether UI could be a save dialog with a warning on overwriting
a file, or whether it's a general purpose open file dialog.
Qt now does this. Previously no warning was shown on overwriting a file,
so it may be used for opening files too without irritating the user.
This has been changed, as non-destructive reads should now use
the open dialog.
Dissectors were changed accordingly.
Change-Id: I9087fefa5ee7ca58de0775d4fe2c0fdcfa3a3018
Reviewed-on: https://code.wireshark.org/review/21086
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Diffstat (limited to 'epan/dissectors/packet-snort.c')
| -rw-r--r-- | epan/dissectors/packet-snort.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-snort.c b/epan/dissectors/packet-snort.c index d6c8894099..691245dd7c 100644 --- a/epan/dissectors/packet-snort.c +++ b/epan/dissectors/packet-snort.c @@ -1471,11 +1471,11 @@ proto_register_snort(void) prefs_register_filename_preference(snort_module, "binary", "Snort binary", "The name of the snort binary file to run", - &pref_snort_binary_filename); + &pref_snort_binary_filename, FALSE); prefs_register_filename_preference(snort_module, "config", "Configuration filename", "The name of the file containing the snort IDS configuration. Typically snort.conf", - &pref_snort_config_filename); + &pref_snort_config_filename, FALSE); prefs_register_bool_preference(snort_module, "show_rule_set_stats", "Show rule stats in protocol tree", |