X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;ds=inline;f=libs%2Fevoral%2Fevoral%2FEventRingBuffer.hpp;h=df9e6aa9ee87060b6af1f5143480f8f3cd508e18;hb=19395ac258bd48595790bcb6fa8b206c43a64796;hp=92d366387f50687f5dd5e32d118429edc983b58b;hpb=166ef64e3db4ab72b7b1e7455234e2b9ceddf6d8;p=ardour.git

diff --git a/libs/evoral/evoral/EventRingBuffer.hpp b/libs/evoral/evoral/EventRingBuffer.hpp
index 92d366387f..df9e6aa9ee 100644
--- a/libs/evoral/evoral/EventRingBuffer.hpp
+++ b/libs/evoral/evoral/EventRingBuffer.hpp
@@ -1,15 +1,15 @@
 /* This file is part of Evoral.
- * Copyright (C) 2008 Dave Robillard <http://drobilla.net>
- * 
+ * Copyright (C) 2008 David Robillard <http://drobilla.net>
+ *
  * Evoral is free software; you can redistribute it and/or modify it under the
  * terms of the GNU General Public License as published by the Free Software
  * Foundation; either version 2 of the License, or (at your option) any later
  * version.
- * 
+ *
  * Evoral is distributed in the hope that it will be useful, but WITHOUT ANY
  * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
  * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for details.
- * 
+ *
  * You should have received a copy of the GNU General Public License along
  * with this program; if not, write to the Free Software Foundation, Inc.,
  * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
@@ -18,76 +18,112 @@
 #ifndef EVORAL_EVENT_RING_BUFFER_HPP
 #define EVORAL_EVENT_RING_BUFFER_HPP
 
-#include <glib.h>
-#include "evoral/RingBuffer.hpp"
+#include <iostream>
+
+#include "pbd/ringbufferNPT.h"
+
+#include "evoral/visibility.h"
 #include "evoral/EventSink.hpp"
 #include "evoral/types.hpp"
 
-#include <iostream>
 using namespace std;
 
 namespace Evoral {
 
-
 /** A RingBuffer of events (generic time-stamped binary "blobs").
  *
  * This packs a timestamp, size, and size bytes of data flat into the buffer.
  * Useful for MIDI events, OSC messages, etc.
+ *
+ * Note: the uint8_t template argument to RingBufferNPT<> indicates "byte
+ * oriented data", not anything particular linked to MIDI or any other
+ * possible interpretation of uint8_t.
  */
-template<typename T>
-class EventRingBuffer : public Evoral::RingBuffer<uint8_t>, public Evoral::EventSink<T> {
+template<typename Time>
+class /*LIBEVORAL_API*/ EventRingBuffer : public PBD::RingBufferNPT<uint8_t>, public Evoral::EventSink<Time> {
 public:
 
 	/** @param capacity Ringbuffer capacity in bytes.
 	 */
-	EventRingBuffer(size_t capacity) : RingBuffer<uint8_t>(capacity)
+	EventRingBuffer(size_t capacity) : PBD::RingBufferNPT<uint8_t>(capacity)
 	{}
 
-	size_t capacity() const { return _size; }
-	
-	bool peek_time(T* time);
+	inline size_t capacity() const { return bufsize(); }
 
-	uint32_t write(T  time, EventType  type, uint32_t  size, const uint8_t* buf);
-	bool     read (T* time, EventType* type, uint32_t* size,       uint8_t* buf);
-};
+	/** Peek at the ringbuffer (read w/o advancing read pointer).
+	 * @return how much has been peeked (wraps around if read exceeds
+	 * the end of the buffer):
+	 * <pre>
+	 * |===========--------------R=============================|
+	 *            read-pointer---^
+	 * </pre>
+	 */
+	inline bool peek (uint8_t*, size_t size);
 
+	inline uint32_t write(Time  time, EventType  type, uint32_t  size, const uint8_t* buf);
+	inline bool     read (Time* time, EventType* type, uint32_t* size,       uint8_t* buf);
+};
 
-template<typename T>
+template<typename Time>
 inline bool
-EventRingBuffer<T>::peek_time(T* time)
+EventRingBuffer<Time>::peek (uint8_t* buf, size_t size)
 {
-	bool success = RingBuffer<uint8_t>::full_peek(sizeof(T), (uint8_t*)time);
-	return success;
-}
+	PBD::RingBufferNPT<uint8_t>::rw_vector vec;
+
+	get_read_vector (&vec);
 
+	if (vec.len[0] + vec.len[1] < size) {
+		return false;
+	}
 
-template<typename T>
+	if (vec.len[0] > 0) {
+		memcpy (buf, vec.buf[0], min (vec.len[0], size));
+	}
+
+	if (vec.len[0] < size) {
+		if (vec.len[1]) {
+			memcpy (buf + vec.len[0], vec.buf[1], size - vec.len[0]);
+		}
+	}
+
+	return true;
+}
+
+template<typename Time>
 inline bool
-EventRingBuffer<T>::read(T* time, EventType* type, uint32_t* size, uint8_t* buf)
+EventRingBuffer<Time>::read(Time* time, EventType* type, uint32_t* size, uint8_t* buf)
 {
-	bool success = RingBuffer<uint8_t>::full_read(sizeof(T), (uint8_t*)time);
-	if (success)
-		success = RingBuffer<uint8_t>::full_read(sizeof(EventType), (uint8_t*)type);
-	if (success)
-		success = RingBuffer<uint8_t>::full_read(sizeof(uint32_t), (uint8_t*)size);
-	if (success)
-		success = RingBuffer<uint8_t>::full_read(*size, buf);
-	
-	return success;
+	if (PBD::RingBufferNPT<uint8_t>::read ((uint8_t*)time, sizeof (Time)) != sizeof (Time)) {
+		return false;
+	}
+
+	if (PBD::RingBufferNPT<uint8_t>::read ((uint8_t*)type, sizeof(EventType)) != sizeof (EventType)) {
+		return false;
+	}
+
+	if (PBD::RingBufferNPT<uint8_t>::read ((uint8_t*)size, sizeof(uint32_t)) != sizeof (uint32_t)) {
+		return false;
+	}
+
+	if (PBD::RingBufferNPT<uint8_t>::read (buf, *size) != *size) {
+		return false;
+	}
+
+	return true;
 }
 
 
-template<typename T>
+template<typename Time>
 inline uint32_t
-EventRingBuffer<T>::write(T time, EventType type, uint32_t size, const uint8_t* buf)
+EventRingBuffer<Time>::write(Time time, EventType type, uint32_t size, const uint8_t* buf)
 {
-	if (write_space() < (sizeof(T) + sizeof(EventType) + sizeof(uint32_t) + size)) {
+	if (!buf || write_space() < (sizeof(Time) + sizeof(EventType) + sizeof(uint32_t) + size)) {
 		return 0;
 	} else {
-		RingBuffer<uint8_t>::write(sizeof(T), (uint8_t*)&time);
-		RingBuffer<uint8_t>::write(sizeof(EventType), (uint8_t*)&type);
-		RingBuffer<uint8_t>::write(sizeof(uint32_t), (uint8_t*)&size);
-		RingBuffer<uint8_t>::write(size, buf);
+		PBD::RingBufferNPT<uint8_t>::write ((uint8_t*)&time, sizeof(Time));
+		PBD::RingBufferNPT<uint8_t>::write ((uint8_t*)&type, sizeof(EventType));
+		PBD::RingBufferNPT<uint8_t>::write ((uint8_t*)&size, sizeof(uint32_t));
+		PBD::RingBufferNPT<uint8_t>::write (buf, size);
 		return size;
 	}
 }