diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-05-11 21:05:56 +0200 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-05-11 21:05:56 +0200 |
| commit | 72f6f3de6a098cc74394e047aaa3fa5f9f44aa83 (patch) | |
| tree | 0f83898e07b18f18694a09e2209e8418454bab19 /src/wx/audio_panel.cc | |
| parent | a98f6e4f84d02b87ba05cecfcc3005858f274afa (diff) | |
Use case for property checks now that it's possible.
This would have prevented the bug fixed in 6bbf7dee.
Diffstat (limited to 'src/wx/audio_panel.cc')
| -rw-r--r-- | src/wx/audio_panel.cc | 27 |
1 files changed, 20 insertions, 7 deletions
diff --git a/src/wx/audio_panel.cc b/src/wx/audio_panel.cc index 1418f1ff9..41e073dd1 100644 --- a/src/wx/audio_panel.cc +++ b/src/wx/audio_panel.cc @@ -214,8 +214,9 @@ AudioPanel::film_changed (FilmProperty property) void AudioPanel::film_content_changed (int property) { - auto ac = _parent->selected_audio (); - if (property == AudioContentProperty::STREAMS) { + auto const ac = _parent->selected_audio(); + switch (property) { + case AudioContentProperty::STREAMS: if (ac.size() == 1) { _mapping->set (ac.front()->audio->mapping()); _mapping->set_input_channels (ac.front()->audio->channel_names ()); @@ -242,13 +243,17 @@ AudioPanel::film_content_changed (int property) setup_description (); setup_peak (); layout (); - } else if (property == AudioContentProperty::GAIN) { + break; + case AudioContentProperty::GAIN: /* This is a bit aggressive but probably not so bad */ _peak_cache.clear(); setup_peak (); - } else if (property == ContentProperty::VIDEO_FRAME_RATE) { + break; + case ContentProperty::VIDEO_FRAME_RATE: setup_description (); - } else if (property == AudioContentProperty::FADE_IN) { + break; + case AudioContentProperty::FADE_IN: + { set<Frame> check; for (auto i: ac) { check.insert (i->audio->fade_in().get()); @@ -262,7 +267,10 @@ AudioPanel::film_content_changed (int property) } else { _fade_in->clear (); } - } else if (property == AudioContentProperty::FADE_OUT) { + break; + } + case AudioContentProperty::FADE_OUT: + { set<Frame> check; for (auto i: ac) { check.insert (i->audio->fade_out().get()); @@ -276,7 +284,10 @@ AudioPanel::film_content_changed (int property) } else { _fade_out->clear (); } - } else if (property == AudioContentProperty::USE_SAME_FADES_AS_VIDEO) { + break; + } + case AudioContentProperty::USE_SAME_FADES_AS_VIDEO: + { set<bool> check; for (auto i: ac) { check.insert(i->audio->use_same_fades_as_video()); @@ -288,6 +299,8 @@ AudioPanel::film_content_changed (int property) _use_same_fades_as_video->set(false); } setup_sensitivity (); + break; + } } } |
