#include "mackie_control_exception.h"
#include "controls.h"
-#include <midi++/types.h>
-#include <midi++/port.h>
+#include "midi++/types.h"
+#include "midi++/port.h"
+#include "midi++/manager.h"
#include <sigc++/sigc++.h>
#include <boost/shared_array.hpp>
#include "i18n.h"
-#define _XOPEN_SOURCE 600 // force XSI for non-GNU strerror_r()
-
#include <sstream>
+
#include <cstring>
#include <cerrno>
-
using namespace std;
using namespace Mackie;
-SurfacePort::SurfacePort( MIDI::Port & port, int number )
-: _port( port ), _number( number ), _active( false )
+SurfacePort::SurfacePort()
+ : _input_port (0), _output_port (0), _number (0), _active (false)
+{
+}
+
+/** @param input_port Input MIDI::Port; this object takes responsibility for removing it from
+ * the MIDI::Manager and destroying it.
+ * @param output_port Output MIDI::Port; responsibility similarly taken.
+ */
+SurfacePort::SurfacePort (MIDI::Port & input_port, MIDI::Port & output_port, int number)
+ : _input_port (&input_port), _output_port (&output_port), _number (number), _active (false)
{
}
SurfacePort::~SurfacePort()
{
- //cout << "~SurfacePort::SurfacePort()" << endl;
+#ifdef PORT_DEBUG
+ cout << "~SurfacePort::SurfacePort()" << endl;
+#endif
// make sure another thread isn't reading or writing as we close the port
- Glib::RecMutex::Lock lock( _rwlock );
+ Glib::RecMutex::Lock lock (_rwlock);
_active = false;
- //cout << "~SurfacePort::SurfacePort() finished" << endl;
+
+ MIDI::Manager* mm = MIDI::Manager::instance ();
+
+ if (_input_port) {
+ mm->remove_port (_input_port);
+ delete _input_port;
+ }
+
+ if (_output_port) {
+ mm->remove_port (_output_port);
+ delete _output_port;
+ }
+
+#ifdef PORT_DEBUG
+ cout << "~SurfacePort::SurfacePort() finished" << endl;
+#endif
}
+// wrapper for one day when strerror_r is working properly
+string fetch_errmsg (int error_number)
+{
+ char * msg = strerror (error_number);
+ return msg;
+}
+
MidiByteArray SurfacePort::read()
{
const int max_buf_size = 512;
// check active. Mainly so that the destructor
// doesn't destroy the mutex while it's still locked
- if ( !active() ) return retval;
+ if (!active()) {
+ return retval;
+ }
// return nothing read if the lock isn't acquired
- Glib::RecMutex::Lock lock( _rwlock, Glib::TRY_LOCK );
+#if 0
+ Glib::RecMutex::Lock lock (_rwlock, Glib::TRY_LOCK);
- if ( !lock.locked() )
- {
- //cout << "SurfacePort::read not locked" << endl;
+ if (!lock.locked()) {
+ cout << "SurfacePort::read not locked" << endl;
return retval;
}
// check active again - destructor sequence
- if ( !active() ) return retval;
+ if (!active()) return retval;
+#endif
// read port and copy to return value
- int nread = port().read( buf, sizeof (buf) );
+ int nread = input_port().read (buf, sizeof (buf));
if (nread >= 0) {
- retval.copy( nread, buf );
- if ((size_t) nread == sizeof (buf))
- {
+ retval.copy (nread, buf);
+ if ((size_t) nread == sizeof (buf)) {
+#ifdef PORT_DEBUG
+ cout << "SurfacePort::read recursive" << endl;
+#endif
retval << read();
}
- }
- else
- {
- if ( errno != EAGAIN )
- {
+ } else {
+ if (errno != EAGAIN) {
ostringstream os;
- os << "Surface: error reading from port: " << port().name() << ": " << errno;
+ os << "Surface: error reading from port: " << input_port().name();
+ os << ": " << errno << fetch_errmsg (errno);
- char buf[512];
- int result = strerror_r( errno, buf, 512 );
- if (!result) {
- os << " " << buf;
- }
-
cout << os.str() << endl;
inactive_event();
- throw MackieControlException( os.str() );
+ throw MackieControlException (os.str());
}
}
+#ifdef PORT_DEBUG
+ cout << "SurfacePort::read: " << retval << endl;
+#endif
return retval;
}
-void SurfacePort::write( const MidiByteArray & mba )
+void SurfacePort::write (const MidiByteArray & mba)
{
- //if ( mba[0] == 0xf0 ) cout << "SurfacePort::write: " << mba << endl;
- //cout << "SurfacePort::write: " << mba << endl;
+ if (mba.empty()) {
+ return;
+ }
+
+#ifdef PORT_DEBUG
+ cout << "SurfacePort::write: " << mba << " to " << output_port().name() << endl;
+#endif
// check active before and after lock - to make sure
// that the destructor doesn't destroy the mutex while
// it's still in use
- if ( !active() ) return;
- Glib::RecMutex::Lock lock( _rwlock );
- if ( !active() ) return;
-
- int count = port().write( mba.bytes().get(), mba.size() );
- if ( count != (int)mba.size() )
- {
- if ( errno != EAGAIN )
- {
+ if (!active()) return;
+ Glib::RecMutex::Lock lock (_rwlock);
+ if (!active()) return;
+
+ int count = output_port().write (mba.bytes().get(), mba.size(), 0);
+ if (count != (int)mba.size()) {
+ if (errno == 0) {
+ cout << "port overflow on " << output_port().name() << ". Did not write all of " << mba << endl;
+ } else if (errno != EAGAIN) {
ostringstream os;
- os << "Surface: couldn't write to port " << port().name() << ": " << errno;
- char buf[512];
- int result = strerror_r( errno, buf, 512 );
-
- if (!result) {
- os << " " << buf;
- }
-
- cout << os.str();
+ os << "Surface: couldn't write to port " << output_port().name();
+ os << ", error: " << fetch_errmsg (errno) << "(" << errno << ")";
+
+ cout << os.str() << endl;
inactive_event();
- throw MackieControlException( os.str() );
}
}
- //if ( mba[0] == 0xf0 ) cout << "SurfacePort::write " << count << endl;
+#ifdef PORT_DEBUG
+ cout << "SurfacePort::wrote " << count << endl;
+#endif
}
-void SurfacePort::write_sysex( const MidiByteArray & mba )
+void SurfacePort::write_sysex (const MidiByteArray & mba)
{
+ if (mba.empty()) {
+ return;
+ }
+
MidiByteArray buf;
buf << sysex_hdr() << mba << MIDI::eox;
- write( buf );
+ write (buf);
}
-void SurfacePort::write_sysex( MIDI::byte msg )
+void SurfacePort::write_sysex (MIDI::byte msg)
{
MidiByteArray buf;
buf << sysex_hdr() << msg << MIDI::eox;
- write( buf );
+ write (buf);
}
-// This should be moved to midi++ at some point
-ostream & operator << ( ostream & os, const MIDI::Port & port )
-{
- os << "device: " << port.device();
- os << "; ";
- os << "name: " << port.name();
- os << "; ";
- os << "type: " << port.type();
- os << "; ";
- os << "mode: " << port.mode();
- os << "; ";
- os << "ok: " << port.ok();
- os << "; ";
- os << "number: " << port.number();
- os << "; ";
- return os;
-}
-
-ostream & Mackie::operator << ( ostream & os, const SurfacePort & port )
+ostream & Mackie::operator << (ostream & os, const SurfacePort & port)
{
os << "{ ";
- os << "device: " << port.port().device();
+ os << "name: " << port.input_port().name() << " " << port.output_port().name();
os << "; ";
- os << "name: " << port.port().name();
- os << "; ";
- os << "number: " << port.number();
os << " }";
return os;
}
+