small round of compiler warning fixes
[ardour.git] / libs / ardour / midi_ring_buffer.cc
index 902f6866f17443c2e4909999aaeb95a43515b0e7..555ac6fb96c6422975655688ee8f9eaf3092d9c0 100644 (file)
@@ -17,6 +17,8 @@
 */
 
 #include "pbd/compose.h"
+#include "pbd/enumwriter.h"
+#include "pbd/error.h"
 
 #include "ardour/debug.h"
 #include "ardour/midi_ring_buffer.h"
 #include "ardour/event_type_map.h"
 
 using namespace std;
-using namespace ARDOUR;
 using namespace PBD;
 
+namespace ARDOUR {
+
 /** Read a block of MIDI events from this buffer into a MidiBuffer.
  *
  * Timestamps of events returned are relative to start (i.e. event with stamp 0
@@ -41,83 +44,43 @@ MidiRingBuffer<T>::read(MidiBuffer& dst, framepos_t start, framepos_t end, frame
        }
 
        T                 ev_time;
-       Evoral::EventType ev_type;
        uint32_t          ev_size;
-
-       /* If we see the end of a loop during this read, we must write the events after it
-          to the MidiBuffer with adjusted times.  The situation is as follows:
-
-          session frames----------------------------->
-
-                    |                            |                    |
-               start_of_loop                   start              end_of_loop
-
-          The MidiDiskstream::read method which will have happened before this checks for
-          loops ending, and helpfully inserts a magic LoopEvent into the ringbuffer.  After this,
-          the MidiDiskstream continues to write events with their proper session frame times,
-          so after the LoopEvent event times will go backwards (ie non-monotonically).
-
-          Once we hit end_of_loop, we need to fake it to make it look as though the loop has been
-          immediately repeated.  Say that an event E after the end_of_loop in the ringbuffer
-          has time E_t, which is a time in session frames.  Its offset from the start
-          of the loop will be E_t - start_of_loop.  Its `faked' time will therefore be
-          end_of_loop + E_t - start_of_loop.  And so its port-buffer-relative time (for
-          writing to the MidiBuffer) will be end_of_loop + E_t - start_of_loop - start.
-
-          The subtraction of start is already taken care of, so if we see a LoopEvent, we'll
-          set up loop_offset to equal end_of_loop - start_of_loop, so that given an event
-          time E_t in the ringbuffer we can get the port-buffer-relative time as
-          E_t + offset - start.
-       */
-
-       frameoffset_t loop_offset = 0;
-
-       size_t count = 0;
-
-       const size_t prefix_size = sizeof(T) + sizeof(Evoral::EventType) + sizeof(uint32_t);
+       size_t            count = 0;
+       const size_t      prefix_size = sizeof(T) + sizeof(Evoral::EventType) + sizeof(uint32_t);
 
        while (this->read_space() >= prefix_size) {
 
                uint8_t peekbuf[prefix_size];
-               bool success;
 
-               success = this->peek (peekbuf, prefix_size);
                /* this cannot fail, because we've already verified that there
                   is prefix_space to read
                */
-               assert (success);
+               this->peek (peekbuf, prefix_size);
 
-               ev_time = *((T*) peekbuf);
-               ev_type = *((Evoral::EventType*)(peekbuf + sizeof (T)));
-               ev_size = *((uint32_t*)(peekbuf + sizeof(T) + sizeof (Evoral::EventType)));
+               ev_time = *(reinterpret_cast<T*>((uintptr_t)peekbuf));
+               ev_size = *(reinterpret_cast<uint32_t*>((uintptr_t)(peekbuf + sizeof(T) + sizeof (Evoral::EventType))));
 
-               if (ev_time + loop_offset >= end) {
+               if (ev_time >= end) {
                        DEBUG_TRACE (DEBUG::MidiDiskstreamIO, string_compose ("MRB event @ %1 past end @ %2\n", ev_time, end));
                        break;
-               } else if (ev_time + loop_offset < start) {
+               } else if (ev_time < start) {
                        DEBUG_TRACE (DEBUG::MidiDiskstreamIO, string_compose ("MRB event @ %1 before start @ %2\n", ev_time, start));
                        break;
                } else {
                        DEBUG_TRACE (DEBUG::MidiDiskstreamIO, string_compose ("MRB event @ %1 in range %2 .. %3\n", ev_time, start, end));
                }
 
-               assert(ev_time >= start);
-
                ev_time -= start;
                ev_time += offset;
 
-               // This event marks a loop end (i.e. the next event's timestamp
-               // will be non-monotonic). Don't write it into the buffer - the
-               // significance of this event ends here.
-               
-               if (ev_type == LoopEventType) {
-                       assert (ev_size == sizeof (framepos_t));
-                       framepos_t loop_start;
-                       read_contents (ev_size, (uint8_t *) &loop_start);
-                       loop_offset = ev_time - loop_start;
-                       _tracker.resolve_notes (dst, ev_time);
-                       continue;
-               }
+               /* we're good to go ahead and read the data now but since we
+                * have the prefix data already, just skip over that
+                */
+               this->increment_read_ptr (prefix_size);
+
+               uint8_t status;
+               bool r = this->peek (&status, sizeof(uint8_t)); 
+               assert (r); // If this failed, buffer is corrupt, all hope is lost
 
                /* lets see if we are going to be able to write this event into dst.
                 */
@@ -127,33 +90,13 @@ MidiRingBuffer<T>::read(MidiBuffer& dst, framepos_t start, framepos_t end, frame
                                DEBUG_TRACE (DEBUG::MidiDiskstreamIO, string_compose ("MidiRingBuffer: overflow in destination MIDI buffer, stopped after %1 events\n", count));
                                break;
                        }
-                       cerr << "MRB: Unable to reserve space in buffer, event skipped";
-                       this->increment_read_ptr (prefix_size + ev_size); // Advance read pointer to next event
+                       error << "MRB: Unable to reserve space in buffer, event skipped" << endmsg;
+                       this->increment_read_ptr (ev_size); // Advance read pointer to next event
                        continue;
                }
 
-               /* we're good to go ahead and read the data now but since we
-                * have the prefix data already, just skip over that
-                */
-               this->increment_read_ptr (prefix_size);
-               ev_time += loop_offset;
-
-               uint8_t status;
-               success = this->peek (&status, sizeof(uint8_t));
-               assert(success); // If this failed, buffer is corrupt, all hope is lost
-
-               // Ignore event if it doesn't match channel filter
-               if (is_channel_event(status) && get_channel_mode() == FilterChannels) {
-                       const uint8_t channel = status & 0x0F;
-                       if (!(get_channel_mask() & (1L << channel))) {
-                               // cerr << "MRB skipping event due to channel mask" << endl;
-                               this->increment_read_ptr (ev_size); // Advance read pointer to next event
-                               continue;
-                       }
-               }
-
                // write MIDI buffer contents
-               success = read_contents (ev_size, write_loc);
+               bool success = read_contents (ev_size, write_loc);
 
 #ifndef NDEBUG
                if (DEBUG::MidiDiskstreamIO && PBD::debug_bits) {
@@ -177,10 +120,7 @@ MidiRingBuffer<T>::read(MidiBuffer& dst, framepos_t start, framepos_t end, frame
                        } else if (is_note_off(write_loc[0])) {
                                _tracker.remove (write_loc[1], write_loc[0] & 0xf);
                        }
-                       
-                       if (is_channel_event(status) && get_channel_mode() == ForceChannel) {
-                               write_loc[0] = (write_loc[0] & 0xF0) | (get_channel_mask() & 0x0F);
-                       }
+
                        ++count;
                } else {
                        cerr << "WARNING: error reading event contents from MIDI ring" << endl;
@@ -190,6 +130,36 @@ MidiRingBuffer<T>::read(MidiBuffer& dst, framepos_t start, framepos_t end, frame
        return count;
 }
 
+template<typename T>
+void
+MidiRingBuffer<T>::flush (framepos_t /*start*/, framepos_t end)
+{
+       const size_t prefix_size = sizeof(T) + sizeof(Evoral::EventType) + sizeof(uint32_t);
+
+       while (this->read_space() >= prefix_size) {
+               uint8_t  peekbuf[prefix_size];
+               bool     success;
+               uint32_t ev_size;
+               T        ev_time;
+
+               success = this->peek (peekbuf, prefix_size);
+               /* this cannot fail, because we've already verified that there
+                  is prefix_space to read
+               */
+               assert (success);
+
+               ev_time = *(reinterpret_cast<T*>((uintptr_t)peekbuf));
+               
+               if (ev_time >= end) {
+                       break;
+               }
+
+               ev_size = *(reinterpret_cast<uint32_t*>((uintptr_t)(peekbuf + sizeof(T) + sizeof (Evoral::EventType))));
+               this->increment_read_ptr (prefix_size);
+               this->increment_read_ptr (ev_size);
+       }
+}
+
 template<typename T>
 void
 MidiRingBuffer<T>::dump(ostream& str)
@@ -268,5 +238,20 @@ MidiRingBuffer<T>::dump(ostream& str)
        delete [] buf;
 }
 
+template<typename T>
+void
+MidiRingBuffer<T>::reset_tracker ()
+{
+       _tracker.reset ();
+}
+
+template<typename T>
+void
+MidiRingBuffer<T>::loop_resolve (MidiBuffer& dst, framepos_t t)
+{
+       _tracker.resolve_notes (dst, t);
+}
+
 template class MidiRingBuffer<framepos_t>;
 
+}  // namespace ARDOUR