summaryrefslogtreecommitdiff
path: root/src/wx/content_menu.cc
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2025-02-09 02:06:04 +0100
committerCarl Hetherington <cth@carlh.net>2025-02-09 17:29:16 +0100
commitd0308d53dd9f4d036d8c5fe8023920fcdfd43f39 (patch)
tree5b37c1db5dc50e541a542663390061f743fe815a /src/wx/content_menu.cc
parent8d0d9866ae3e0395d899705e27b3806a5de7ef0e (diff)
Remove unnecessary wx_ptr
It was only ever used for wxDialog subclasses, which can be stack-allocated.
Diffstat (limited to 'src/wx/content_menu.cc')
-rw-r--r--src/wx/content_menu.cc26
1 files changed, 13 insertions, 13 deletions
diff --git a/src/wx/content_menu.cc b/src/wx/content_menu.cc
index b2010d4bf..1106a10fc 100644
--- a/src/wx/content_menu.cc
+++ b/src/wx/content_menu.cc
@@ -249,8 +249,8 @@ ContentMenu::repeat ()
return;
}
- auto d = make_wx<RepeatDialog>(_parent);
- if (d->ShowModal() != wxID_OK) {
+ RepeatDialog dialog(_parent);
+ if (dialog.ShowModal() != wxID_OK) {
return;
}
@@ -259,7 +259,7 @@ ContentMenu::repeat ()
return;
}
- film->repeat_content (_content, d->number ());
+ film->repeat_content(_content, dialog.number());
_content.clear ();
_views.clear ();
@@ -475,8 +475,8 @@ ContentMenu::properties ()
{
auto film = _film.lock ();
DCPOMATIC_ASSERT (film);
- auto d = make_wx<ContentPropertiesDialog>(_parent, film, _content.front());
- d->ShowModal ();
+ ContentPropertiesDialog dialog(_parent, film, _content.front());
+ dialog.ShowModal();
}
@@ -486,26 +486,26 @@ ContentMenu::advanced ()
DCPOMATIC_ASSERT(!_content.empty());
auto content = _content.front();
- auto dialog = make_wx<ContentAdvancedDialog>(_parent, content);
+ ContentAdvancedDialog dialog(_parent, content);
- if (dialog->ShowModal() == wxID_CANCEL) {
+ if (dialog.ShowModal() == wxID_CANCEL) {
return;
}
if (content->video) {
- content->video->set_use(!dialog->ignore_video());
- content->video->set_burnt_subtitle_language(dialog->burnt_subtitle_language());
+ content->video->set_use(!dialog.ignore_video());
+ content->video->set_burnt_subtitle_language(dialog.burnt_subtitle_language());
}
auto ffmpeg = dynamic_pointer_cast<FFmpegContent>(content);
if (ffmpeg) {
- ffmpeg->set_filters(dialog->filters());
+ ffmpeg->set_filters(dialog.filters());
}
- if (dialog->video_frame_rate()) {
+ if (dialog.video_frame_rate()) {
auto film = _film.lock();
DCPOMATIC_ASSERT(film);
- content->set_video_frame_rate(film, *dialog->video_frame_rate());
+ content->set_video_frame_rate(film, *dialog.video_frame_rate());
} else {
content->unset_video_frame_rate();
}
@@ -570,7 +570,7 @@ ContentMenu::auto_crop ()
auto const crop = guess_crop_for_content ();
update_viewer (crop);
- _auto_crop_dialog.reset(_parent, crop);
+ _auto_crop_dialog.emplace(_parent, crop);
_auto_crop_dialog->Show ();
/* Update the dialog and view when the crop threshold changes */