SMFSource: code readability

git-svn-id: svn://localhost/ardour2/branches/3.0@4975 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Hans Baier 2009-04-14 10:38:23 +00:00
parent 625c9ac228
commit ad2757c84f

View file

@ -171,9 +171,9 @@ SMFSource::write_unlocked (MidiRingBuffer<nframes_t>& source, sframes_t position
{ {
_write_data_count = 0; _write_data_count = 0;
nframes_t time; nframes_t event_time;
Evoral::EventType type; Evoral::EventType event_type;
uint32_t size; uint32_t event_size;
size_t buf_capacity = 4; size_t buf_capacity = 4;
uint8_t* buf = (uint8_t*)malloc(buf_capacity); uint8_t* buf = (uint8_t*)malloc(buf_capacity);
@ -185,36 +185,36 @@ SMFSource::write_unlocked (MidiRingBuffer<nframes_t>& source, sframes_t position
Evoral::MIDIEvent<nframes_t> ev; Evoral::MIDIEvent<nframes_t> ev;
while (true) { while (true) {
bool ret = source.peek_time(&time); bool ret = source.peek_time(&event_time);
g_debug ("time: %u, last_write_end: %lu, duration: %u", time, _last_write_end, duration); g_debug ("event_time: %u, last_write_end: %lu, duration: %u", event_time, _last_write_end, duration);
if (!ret || time > _last_write_end + duration) { if (!ret || event_time > _last_write_end + duration) {
if (!ret) g_debug ("peek failed"); if (!ret) g_debug ("peek failed");
if (time > _last_write_end + duration) g_debug ("time: %u > last_write_end: %lu + duration: %u", time, _last_write_end, duration); if (event_time > _last_write_end + duration) g_debug ("event_time: %u > last_write_end: %lu + duration: %u", event_time, _last_write_end, duration);
break; break;
} }
ret = source.read_prefix(&time, &type, &size); ret = source.read_prefix(&event_time, &event_type, &event_size);
if (!ret) { if (!ret) {
cerr << "ERROR: Unable to read event prefix, corrupt MIDI ring buffer" << endl; cerr << "ERROR: Unable to read event prefix, corrupt MIDI ring buffer" << endl;
break; break;
} }
if (size > buf_capacity) { if (event_size > buf_capacity) {
buf_capacity = size; buf_capacity = event_size;
buf = (uint8_t*)realloc(buf, size); buf = (uint8_t*)realloc(buf, event_size);
} }
ret = source.read_contents(size, buf); ret = source.read_contents(event_size, buf);
if (!ret) { if (!ret) {
cerr << "ERROR: Read time/size but not buffer, corrupt MIDI ring buffer" << endl; cerr << "ERROR: Read event time/size but not buffer, corrupt MIDI ring buffer" << endl;
break; break;
} }
assert(time >= position); assert(event_time >= position);
time -= position; event_time -= position;
ev.set(buf, size, time); ev.set(buf, event_size, event_time);
ev.set_event_type(EventTypeMap::instance().midi_event_type(ev.buffer()[0])); ev.set_event_type(EventTypeMap::instance().midi_event_type(ev.buffer()[0]));
if (!(ev.is_channel_event() || ev.is_smf_meta_event() || ev.is_sysex())) { if (!(ev.is_channel_event() || ev.is_smf_meta_event() || ev.is_sysex())) {
cerr << "SMFSource: WARNING: caller tried to write non SMF-Event of type " cerr << "SMFSource: WARNING: caller tried to write non SMF-Event of type "