diff options
author | Michael Mann <mmann78@netscape.net> | 2017-01-04 10:57:25 -0500 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-01-07 15:35:18 +0000 |
commit | 165c8b04227df05a0ae329e92adff7bd314b9d4b (patch) | |
tree | b632e9e30ca7c3ca472d0e8e3926ec53efe244fc /ui/qt/preference_editor_frame.cpp | |
parent | fb9ac503c16e88408a53e808f17f19f63787f2de (diff) |
Refactor range preference.
Refactored so that all handling of ranges in struct preference
can be internal to prefs.c
Change-Id: I68577909f9c07b23a16ab3443a523355d4645314
Reviewed-on: https://code.wireshark.org/review/19577
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'ui/qt/preference_editor_frame.cpp')
-rw-r--r-- | ui/qt/preference_editor_frame.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/ui/qt/preference_editor_frame.cpp b/ui/qt/preference_editor_frame.cpp index 845920680e..fa08823f7c 100644 --- a/ui/qt/preference_editor_frame.cpp +++ b/ui/qt/preference_editor_frame.cpp @@ -109,7 +109,7 @@ void PreferenceEditorFrame::editPreference(preference *pref, pref_module *module break; case PREF_RANGE: wmem_free(NULL, new_range_); - new_range_ = range_copy(NULL, pref->stashed_val.range); + new_range_ = range_copy(NULL, prefs_get_stashed_range(pref)); connect(ui->preferenceLineEdit, SIGNAL(textEdited(QString)), this, SLOT(rangeLineEditTextEdited(QString))); show = true; @@ -203,11 +203,7 @@ void PreferenceEditorFrame::on_buttonBox_accepted() } break; case PREF_RANGE: - if (!ranges_are_equal(pref_->stashed_val.range, new_range_)) { - wmem_free(wmem_epan_scope(), pref_->stashed_val.range); - pref_->stashed_val.range = range_copy(wmem_epan_scope(), new_range_); - apply = true; - } + apply = prefs_set_stashed_range(pref_, new_range_); break; default: break; |