ardour/livetrax
13
0
Fork 0
Code

Gracefully handle errors parsing midnam documents instead of crashing.

Browse Source 
git-svn-id: svn://localhost/ardour2/branches/3.0@13904 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
David Robillard 2013-01-19 21:31:21 +00:00
parent 444d89b669
commit 85d9fa3b25
1 changed files with 21 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
libs/ardour/midi_patch_manager.cc
View File

@ -30,6 +30,8 @@
#include "ardour/midi_patch_manager.h"
#include "ardour/midi_patch_search_path.h"
#include "i18n.h"
using namespace std;
using namespace ARDOUR;
using namespace MIDI;
@ -109,22 +111,28 @@ MidiPatchManager::refresh()
info << "Loading " << result.size() << " MIDI patches from " << search_path.to_string() << endmsg;
for (vector<std::string>::iterator i = result.begin(); i != result.end(); ++i) {
boost::shared_ptr<MIDINameDocument> document(new MIDINameDocument(*i));
boost::shared_ptr<MIDINameDocument> document;
try {
document = boost::shared_ptr<MIDINameDocument>(new MIDINameDocument(*i));
} catch (...) {
error << "Error parsing MIDI patch file " << *i << endmsg;
continue;
}
for (MIDINameDocument::MasterDeviceNamesList::const_iterator device =
document->master_device_names_by_model().begin();
device != document->master_device_names_by_model().end();
++device) {
//cerr << "got model " << device->first << endl;
// have access to the documents by model name
_documents[device->first] = document;
// build a list of all master devices from all documents
document->master_device_names_by_model().begin();
device != document->master_device_names_by_model().end();
++device) {
if (_documents.find(device->first) != _documents.end()) {
warning << string_compose(_("Duplicate MIDI device `%1' in `%2' ignored"),
device->first, *i)
<< endmsg;
continue;
}
_documents[device->first] = document;
_master_devices_by_model[device->first] = device->second;
_all_models.insert(device->first);
// make sure there are no double model names
// TODO: handle this gracefully.
assert(_documents.count(device->first) == 1);
assert(_master_devices_by_model.count(device->first) == 1);
_all_models.insert(device->first);
}
}