/* We can't have 0 as a menu item ID on OS X */
int id = 1;
BOOST_FOREACH (shared_ptr<dcp::CPL> i, cpls) {
- wxMenuItem* item = _cpl_menu->AppendCheckItem (
+ wxMenuItem* item = _cpl_menu->AppendRadioItem (
id++,
wxString::Format (
"%s (%s)",
);
item->Check (dcp->cpl() && dcp->cpl() == i->id());
}
- } catch (dcp::DCPReadError) {
+ } catch (dcp::DCPReadError &) {
/* The DCP is probably missing */
- } catch (dcp::KDMDecryptionError) {
+ } catch (dcp::KDMDecryptionError &) {
/* We have an incorrect KDM */
- } catch (KDMError) {
+ } catch (KDMError &) {
/* We have an incorrect KDM */
}
} else {
}
try {
- shared_ptr<FFmpegContent> joined (new FFmpegContent (film, fc));
+ shared_ptr<FFmpegContent> joined (new FFmpegContent(fc));
film->remove_content (_content);
film->examine_and_add_content (joined);
} catch (JoinError& e) {
if (r == wxID_OK) {
if (dc) {
- content.push_back (shared_ptr<DCPContent> (new DCPContent (film, path)));
+ content.push_back (shared_ptr<DCPContent>(new DCPContent(path)));
} else {
- content = content_factory (film, path);
+ content = content_factory (path);
}
}
void
ContentMenu::properties ()
{
- ContentPropertiesDialog* d = new ContentPropertiesDialog (_parent, _content.front ());
+ shared_ptr<Film> film = _film.lock ();
+ DCPOMATIC_ASSERT (film);
+ ContentPropertiesDialog* d = new ContentPropertiesDialog (_parent, film, _content.front());
d->ShowModal ();
d->Destroy ();
}