Some more assert() debugging
authorRobin Gareus <robin@gareus.org>
Thu, 3 Aug 2017 21:37:38 +0000 (23:37 +0200)
committerRobin Gareus <robin@gareus.org>
Thu, 3 Aug 2017 21:46:05 +0000 (23:46 +0200)
libs/ardour/audio_port.cc
libs/ardour/session_ltc.cc
libs/backends/coreaudio/coreaudio_backend.cc
libs/backends/portaudio/portaudio_backend.cc

index f559578e4e366bc3c93978a609e2248c1c9a1d15..70943e35a308ef47a475372cb08b86d4fad821ef 100644 (file)
@@ -77,6 +77,7 @@ AudioBuffer&
 AudioPort::get_audio_buffer (pframes_t nframes)
 {
        /* caller must hold process lock */
+       assert (_port_handle);
        _buffer->set_data ((Sample *) port_engine.get_buffer (_port_handle, _cycle_nframes) +
                           _global_port_buffer_offset + _port_buffer_offset, nframes);
        return *_buffer;
@@ -86,6 +87,7 @@ Sample*
 AudioPort::engine_get_whole_audio_buffer ()
 {
        /* caller must hold process lock */
+       assert (_port_handle);
        return (Sample *) port_engine.get_buffer (_port_handle, _cycle_nframes);
 }
 
index 91e11c8a4cde9b3b00cbc3e0f0f3f06489f104d7..8acb16dde30ea7f9247aac42958c45f36bbe0596 100644 (file)
@@ -64,6 +64,7 @@ using namespace Timecode;
 void
 Session::ltc_tx_initialize()
 {
+       assert (!ltc_encoder && !ltc_enc_buf);
        ltc_enc_tcformat = config.get_timecode_format();
 
        ltc_tx_parse_offset();
@@ -93,11 +94,11 @@ void
 Session::ltc_tx_cleanup()
 {
        DEBUG_TRACE (DEBUG::LTC, "LTC TX cleanup\n");
+       ltc_tx_connections.drop_connections ();
        free(ltc_enc_buf);
        ltc_enc_buf = NULL;
        ltc_encoder_free(ltc_encoder);
        ltc_encoder = NULL;
-       ltc_tx_connections.drop_connections ();
 }
 
 void
index 0ef1e6332bad76cd2b9d1f1f51ed99ebc875744f..7137b1d65a4927ae9a2fb5fc79fbbb7e37a2b204 100644 (file)
@@ -1569,7 +1569,9 @@ CoreAudioBackend::n_physical_inputs () const
 void*
 CoreAudioBackend::get_buffer (PortEngine::PortHandle port, pframes_t nframes)
 {
-       if (!port || !valid_port (port)) return NULL;
+       assert (port);
+       assert (valid_port (port));
+       if (!port || !valid_port (port)) return NULL; // XXX remove me
        return static_cast<CoreBackendPort*>(port)->get_buffer (nframes);
 }
 
index c2402a994d706e85511b18b4215d9f376f0b6a9a..d6e220cf276664da33777a5059c0cb1944e8415c 100644 (file)
@@ -1816,7 +1816,9 @@ PortAudioBackend::n_physical_inputs () const
 void*
 PortAudioBackend::get_buffer (PortEngine::PortHandle port, pframes_t nframes)
 {
-       if (!port || !valid_port (port)) return NULL;
+       assert (port);
+       assert (valid_port (port));
+       if (!port || !valid_port (port)) return NULL; // XXX remove me
        return static_cast<PamPort*>(port)->get_buffer (nframes);
 }