Remove unused variables.
[ardour.git] / libs / ardour / export_profile_manager.cc
index 893788809fa23003ac1ce4f219e1ea6d0d4447ff..7eaa7c0632a1185ff4486d16dc7d0dee747f446d 100644 (file)
@@ -29,6 +29,7 @@
 
 #include "ardour/export_profile_manager.h"
 #include "ardour/export_format_specification.h"
+#include "ardour/export_formats_search_path.h"
 #include "ardour/export_timespan.h"
 #include "ardour/export_channel_configuration.h"
 #include "ardour/export_filename.h"
@@ -60,22 +61,14 @@ ExportProfileManager::ExportProfileManager (Session & s, std::string xml_node_na
 
   , format_list (new FormatList ())
 {
-
        /* Initialize path variables */
 
        export_config_dir = user_config_directory();
        export_config_dir /= "export";
-       search_path += export_config_dir;
-
-       search_path += ardour_search_path().add_subdirectory_to_paths("export");
 
-       sys::path sys_export = ardour_module_directory();
-       sys_export /= "export";
-
-       search_path += sys_export;
+       search_path += export_formats_search_path();
 
        info << string_compose (_("Searching for export formats in %1"), search_path.to_string()) << endmsg;
-       cerr << string_compose (_("Searching for export formats in %1"), search_path.to_string()) << endl;
 
        /* create export config directory if necessary */
 
@@ -131,7 +124,7 @@ ExportProfileManager::prepare_for_export ()
                     format_it != formats.end() && filename_it != filenames.end();
                     ++format_it, ++filename_it) {
 
-                       FilenamePtr filename = (*filename_it)->filename;
+                       ExportFilenamePtr filename = (*filename_it)->filename;
 //                     filename->include_timespan = (ts_list->size() > 1); Disabled for now...
 
                        boost::shared_ptr<BroadcastInfo> b;
@@ -150,7 +143,7 @@ ExportProfileManager::prepare_for_export ()
 }
 
 bool
-ExportProfileManager::load_preset (PresetPtr preset)
+ExportProfileManager::load_preset (ExportPresetPtr preset)
 {
        bool ok = true;
 
@@ -181,10 +174,27 @@ ExportProfileManager::load_presets ()
        }
 }
 
-ExportProfileManager::PresetPtr
+std::string
+ExportProfileManager::preset_filename (std::string const & preset_name)
+{
+       string safe_name = legalize_for_path (preset_name);
+       return export_config_dir.to_string() + "/" + safe_name + export_preset_suffix;
+}
+
+ExportPresetPtr
+ExportProfileManager::new_preset (string const & name)
+{
+       // Generate new ID and do regular save
+       string filename = preset_filename (name);
+       current_preset.reset (new ExportPreset (filename, session));
+       preset_list.push_back (current_preset);
+       return save_preset (name);
+}
+
+ExportPresetPtr
 ExportProfileManager::save_preset (string const & name)
 {
-       string filename = export_config_dir.to_string() + "/" + name + export_preset_suffix;
+       string filename = preset_filename (name);
 
        if (!current_preset) {
                current_preset.reset (new ExportPreset (filename, session));
@@ -231,7 +241,7 @@ ExportProfileManager::remove_preset ()
 void
 ExportProfileManager::load_preset_from_disk (PBD::sys::path const & path)
 {
-       PresetPtr preset (new ExportPreset (path.to_string(), session));
+       ExportPresetPtr preset (new ExportPreset (path.to_string(), session));
 
        /* Handle id to filename mapping and don't add duplicates to list */
 
@@ -351,9 +361,9 @@ ExportProfileManager::init_timespans (XMLNodeList nodes)
        if (timespans.empty()) {
                TimespanStatePtr state (new TimespanState (session_range, selection_range, ranges));
                timespans.push_back (state);
-               
+
                // Add session as default selection
-               TimespanPtr timespan = handler->add_timespan();
+               ExportTimespanPtr timespan = handler->add_timespan();
                timespan->set_name (session_range->name());
                timespan->set_range_id ("session");
                timespan->set_range (session_range->start(), session_range->end());
@@ -381,7 +391,7 @@ ExportProfileManager::deserialize_timespan (XMLNode & root)
                        if ((!id.compare ("session") && *it == session_range.get()) ||
                            (!id.compare ("selection") && *it == selection_range.get()) ||
                            (!id.compare ((*it)->id().to_s()))) {
-                               TimespanPtr timespan = handler->add_timespan();
+                               ExportTimespanPtr timespan = handler->add_timespan();
                                timespan->set_name ((*it)->name());
                                timespan->set_range_id (id);
                                timespan->set_range ((*it)->start(), (*it)->end());
@@ -463,15 +473,15 @@ ExportProfileManager::init_channel_configs (XMLNodeList nodes)
        if (nodes.empty()) {
                ChannelConfigStatePtr config (new ChannelConfigState (handler->add_channel_config()));
                channel_configs.push_back (config);
-               
+
                // Add master outs as default
                IO* master_out = session.master_out()->output().get();
                if (!master_out) { return false; }
-               
+
                for (uint32_t n = 0; n < master_out->n_ports().n_audio(); ++n) {
                        PortExportChannel * channel = new PortExportChannel ();
                        channel->add_port (master_out->audio (n));
-                       
+
                        ExportChannelPtr chan_ptr (channel);
                        config->config->register_channel (chan_ptr);
                }
@@ -510,7 +520,7 @@ ExportProfileManager::remove_format_state (FormatStatePtr state)
 }
 
 sys::path
-ExportProfileManager::save_format_to_disk (FormatPtr format)
+ExportProfileManager::save_format_to_disk (ExportFormatSpecPtr format)
 {
        // TODO filename character stripping
 
@@ -519,6 +529,10 @@ ExportProfileManager::save_format_to_disk (FormatPtr format)
        string new_name = format->name();
        new_name += export_format_suffix;
 
+        /* make sure its legal for the filesystem */
+
+        new_name = legalize_for_path (new_name);
+
        sys::path new_path (export_config_dir);
        new_path /= new_name;
 
@@ -563,7 +577,7 @@ ExportProfileManager::save_format_to_disk (FormatPtr format)
 }
 
 void
-ExportProfileManager::remove_format_profile (FormatPtr format)
+ExportProfileManager::remove_format_profile (ExportFormatSpecPtr format)
 {
        for (FormatList::iterator it = format_list->begin(); it != format_list->end(); ++it) {
                if (*it == format) {
@@ -581,10 +595,10 @@ ExportProfileManager::remove_format_profile (FormatPtr format)
        FormatListChanged ();
 }
 
-ExportProfileManager::FormatPtr
-ExportProfileManager::get_new_format (FormatPtr original)
+ExportFormatSpecPtr
+ExportProfileManager::get_new_format (ExportFormatSpecPtr original)
 {
-       FormatPtr format;
+       ExportFormatSpecPtr format;
        if (original) {
                format.reset (new ExportFormatSpecification (*original));
        } else {
@@ -616,7 +630,7 @@ ExportProfileManager::init_formats (XMLNodeList nodes)
        }
 
        if (formats.empty ()) {
-               FormatStatePtr format (new FormatState (format_list, FormatPtr ()));
+               FormatStatePtr format (new FormatState (format_list, ExportFormatSpecPtr ()));
                formats.push_back (format);
                return false;
        }
@@ -668,7 +682,7 @@ void
 ExportProfileManager::load_format_from_disk (PBD::sys::path const & path)
 {
        XMLTree const tree (path.to_string());
-       FormatPtr format = handler->add_format (*tree.root());
+       ExportFormatSpecPtr format = handler->add_format (*tree.root());
 
        /* Handle id to filename mapping and don't add duplicates to list */
 
@@ -705,7 +719,7 @@ ExportProfileManager::init_filenames (XMLNodeList nodes)
        filenames.clear ();
 
        for (XMLNodeList::const_iterator it = nodes.begin(); it != nodes.end(); ++it) {
-               FilenamePtr filename = handler->add_filename();
+               ExportFilenamePtr filename = handler->add_filename();
                filename->set_state (**it);
                filenames.push_back (FilenameStatePtr (new FilenameState (filename)));
        }
@@ -724,15 +738,21 @@ ExportProfileManager::get_warnings ()
 {
        boost::shared_ptr<Warnings> warnings (new Warnings ());
 
-       assert (!channel_configs.empty ());
-
-       ChannelConfigStatePtr channel_config_state = channel_configs.front();
+       ChannelConfigStatePtr channel_config_state;
+       if (!channel_configs.empty ()) {
+               channel_config_state = channel_configs.front();
+       }
+       
        TimespanStatePtr timespan_state = timespans.front();
 
        /*** Check "global" config ***/
 
        TimespanListPtr timespans = timespan_state->timespans;
-       ChannelConfigPtr channel_config = channel_config_state->config;
+
+       ExportChannelConfigPtr channel_config;
+       if (channel_config_state) {
+               channel_config = channel_config_state->config;
+       }
 
        /* Check Timespans are not empty */
 
@@ -740,22 +760,27 @@ ExportProfileManager::get_warnings ()
                warnings->errors.push_back (_("No timespan has been selected!"));
        }
 
-       /* Check channel config ports */
-
-       if (!channel_config->all_channels_have_ports ()) {
-               warnings->warnings.push_back (_("Some channels are empty"));
+       if (channel_config_state == 0) {
+               warnings->errors.push_back (_("No channels have been selected!"));
+       } else {
+               /* Check channel config ports */
+               if (!channel_config->all_channels_have_ports ()) {
+                       warnings->warnings.push_back (_("Some channels are empty"));
+               }
        }
 
        /*** Check files ***/
 
-       FormatStateList::const_iterator format_it;
-       FilenameStateList::const_iterator filename_it;
-       for (format_it = formats.begin(), filename_it = filenames.begin();
-            format_it != formats.end() && filename_it != filenames.end();
-            ++format_it, ++filename_it) {
+       if (channel_config_state) {
+               FormatStateList::const_iterator format_it;
+               FilenameStateList::const_iterator filename_it;
+               for (format_it = formats.begin(), filename_it = filenames.begin();
+                    format_it != formats.end() && filename_it != filenames.end();
+                    ++format_it, ++filename_it) {
                        check_config (warnings, timespan_state, channel_config_state, *format_it, *filename_it);
+               }
        }
-
+       
        return warnings;
 }
 
@@ -767,9 +792,9 @@ ExportProfileManager::check_config (boost::shared_ptr<Warnings> warnings,
                                    FilenameStatePtr filename_state)
 {
        TimespanListPtr timespans = timespan_state->timespans;
-       ChannelConfigPtr channel_config = channel_config_state->config;
-       FormatPtr format = format_state->format;
-       FilenamePtr filename = filename_state->filename;
+       ExportChannelConfigPtr channel_config = channel_config_state->config;
+       ExportFormatSpecPtr format = format_state->format;
+       ExportFilenamePtr filename = filename_state->filename;
 
        /* Check format and maximum channel count */
        if (!format || !format->type()) {
@@ -792,7 +817,7 @@ ExportProfileManager::check_config (boost::shared_ptr<Warnings> warnings,
 
 //     filename->include_timespan = (timespans->size() > 1); Disabled for now...
 
-       for (std::list<TimespanPtr>::iterator timespan_it = timespans->begin(); timespan_it != timespans->end(); ++timespan_it) {
+       for (std::list<ExportTimespanPtr>::iterator timespan_it = timespans->begin(); timespan_it != timespans->end(); ++timespan_it) {
                filename->set_timespan (*timespan_it);
 
                if (channel_config->get_split()) {
@@ -819,7 +844,7 @@ ExportProfileManager::check_config (boost::shared_ptr<Warnings> warnings,
 }
 
 bool
-ExportProfileManager::check_format (FormatPtr format, uint32_t channels)
+ExportProfileManager::check_format (ExportFormatSpecPtr format, uint32_t channels)
 {
        switch (format->type()) {
          case ExportFormatBase::T_Sndfile:
@@ -831,7 +856,7 @@ ExportProfileManager::check_format (FormatPtr format, uint32_t channels)
 }
 
 bool
-ExportProfileManager::check_sndfile_format (FormatPtr format, unsigned int channels)
+ExportProfileManager::check_sndfile_format (ExportFormatSpecPtr format, unsigned int channels)
 {
        SF_INFO sf_info;
        sf_info.channels = channels;