From b51af279c4be77dd401b0ba910a34d9977b2453f Mon Sep 17 00:00:00 2001 From: Michele Calgaro Date: Thu, 25 Jun 2020 12:28:32 +0900 Subject: [PATCH] Fixed FTBFS with xine 1.2.10. The use of the xine scope plugin is currently disabled when xine >= 1.2.10 is detected. Additional info: The xine scope plugin is statically linked and therefore written by accessing internal xine data that are no longer publicly available from xine 1.2.10. To add back the scope plugin functionality, the plugin will have to be converted into a dynamically linked one and further work done to fix the logic of the plugin itself. Signed-off-by: Michele Calgaro --- src/app/xineEngine.cpp | 16 ++- src/app/xineScope.c | 302 +++++++++++++++++++++-------------------- 2 files changed, 168 insertions(+), 150 deletions(-) diff --git a/src/app/xineEngine.cpp b/src/app/xineEngine.cpp index 2e113c8..d89d300 100644 --- a/src/app/xineEngine.cpp +++ b/src/app/xineEngine.cpp @@ -146,16 +146,19 @@ VideoWindow::init() xine_osd_set_text_palette( m_osd, XINE_TEXTPALETTE_WHITE_BLACK_TRANSPARENT, XINE_OSD_TEXT1 ); } - #ifndef XINE_SAFE_MODE +#ifndef XINE_SAFE_MODE +#if XINE_MAJOR_VERSION < 1 || (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION < 2) || \ + (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION == 2 && XINE_SUB_VERSION < 10) debug() << "scope_plugin_new()\n"; m_scope = scope_plugin_new( m_xine, m_audioPort ); +#endif //FIXME this one seems to make seeking unstable for Codeine, perhaps xine_set_param( m_stream, XINE_PARAM_METRONOM_PREBUFFER, 6000 ); //less buffering, faster seeking.. // causes an abort currently //xine_trick_mode( m_stream, XINE_TRICK_MODE_SEEK_TO_TIME, 1 ); - #endif +#endif { @@ -266,11 +269,14 @@ VideoWindow::load( const KURL &url ) // FIXME leaves one erroneous buffer timerEvent( 0 ); +#if XINE_MAJOR_VERSION < 1 || (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION < 2) || \ + (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION == 2 && XINE_SUB_VERSION < 10) if( m_scope ) { xine_post_out_t *source = xine_get_audio_source( m_stream ); xine_post_in_t *target = (xine_post_in_t*)xine_post_input( m_scope, const_cast("audio in") ); xine_post_wire( source, target ); } +#endif #endif announceStateChange(); @@ -572,6 +578,8 @@ VideoWindow::scope() if( xine_get_status( m_stream ) != XINE_STATUS_PLAY ) return scope; +#if XINE_MAJOR_VERSION < 1 || (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION < 2) || \ + (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION == 2 && XINE_SUB_VERSION < 10) //prune the buffer list and update the m_current_vpts timestamp timerEvent( 0 ); @@ -618,6 +626,7 @@ VideoWindow::scope() m_current_vpts = best_node->vpts_end; m_current_vpts++; //FIXME needs to be done for some reason, or you get situations where it uses same buffer again and again } +#endif return scope; } @@ -625,6 +634,8 @@ VideoWindow::scope() void VideoWindow::timerEvent( TQTimerEvent* ) { +#if XINE_MAJOR_VERSION < 1 || (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION < 2) || \ + (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION == 2 && XINE_SUB_VERSION < 10) /// here we prune the buffer list regularly #ifndef XINE_SAFE_MODE MyNode * const first_node = myList->next; @@ -650,6 +661,7 @@ VideoWindow::timerEvent( TQTimerEvent* ) prev = node; } #endif +#endif } void diff --git a/src/app/xineScope.c b/src/app/xineScope.c index 740d574..cd3b4e1 100644 --- a/src/app/xineScope.c +++ b/src/app/xineScope.c @@ -1,148 +1,154 @@ -/* Author: Max Howell , (C) 2004 - Copyright: See COPYING file that comes with this distribution */ - -/* gcc doesn't like inline for me */ -#define inline -/* need access to port_ticket */ -#define XINE_ENGINE_INTERNAL - -#include "xineScope.h" -#include -#include - - -static MyNode theList; -static metronom_t theMetronom; -static int myChannels = 0; - -MyNode* const myList = &theList; -metronom_t* const myMetronom = &theMetronom; - - -/* defined in xineEngine.cpp */ -extern void _debug( const char * ); - - -/************************* -* post plugin functions * -*************************/ - -static int -scope_port_open( xine_audio_port_t *port_gen, xine_stream_t *stream, uint32_t bits, uint32_t rate, int mode ) -{ - _debug( "scope_port_open()\n" ); - - #define port ((post_audio_port_t*)port_gen) - - _x_post_rewire( (post_plugin_t*)port->post ); - _x_post_inc_usage( port ); - - port->stream = stream; - port->bits = bits; - port->rate = rate; - port->mode = mode; - - myChannels = _x_ao_mode2channels( mode ); - - return port->original_port->open( port->original_port, stream, bits, rate, mode ); -} - -static void -scope_port_close( xine_audio_port_t *port_gen, xine_stream_t *stream ) -{ - _debug( "scope_port_close()\n" ); - - port->stream = NULL; - port->original_port->close( port->original_port, stream ); - - _x_post_dec_usage( port ); -} - -static void -scope_port_put_buffer( xine_audio_port_t *port_gen, audio_buffer_t *buf, xine_stream_t *stream ) -{ - MyNode *new_node; - const int num_samples = buf->num_frames * myChannels; - - /* we are too simple to handle 8bit */ - /* what does it mean when stream == NULL? */ - if( port->bits == 8 ) { - port->original_port->put_buffer( port->original_port, buf, stream ); return; } - - /* I keep my own metronom because xine wouldn't for some reason */ - memcpy( myMetronom, stream->metronom, sizeof(metronom_t) ); - - new_node = malloc( sizeof(MyNode) ); - new_node->vpts = myMetronom->got_audio_samples( myMetronom, buf->vpts, buf->num_frames ); - new_node->num_frames = buf->num_frames; - new_node->mem = malloc( num_samples * 2 ); - memcpy( new_node->mem, buf->mem, num_samples * 2 ); - - { - int64_t - K = myMetronom->pts_per_smpls; /*smpls = 1<<16 samples*/ - K *= num_samples; - K /= (1<<16); - K += new_node->vpts; - - new_node->vpts_end = K; - } - - /* pass data to original port */ - port->original_port->put_buffer( port->original_port, buf, stream ); - - /* finally we should append the current buffer to the list - * NOTE this is thread-safe due to the way we handle the list in the GUI thread */ - new_node->next = myList->next; - myList->next = new_node; - - #undef port -} - -static void -scope_dispose( post_plugin_t *this ) -{ - free( this ); -} - - -/************************ -* plugin init function * -************************/ - -xine_post_t* -scope_plugin_new( xine_t *xine, xine_audio_port_t *audio_target ) -{ - if( audio_target == NULL ) - return NULL; - - post_plugin_t *post_plugin = xine_xmalloc( sizeof(post_plugin_t) ); - - { - post_plugin_t *this = post_plugin; - post_in_t *input; - post_out_t *output; - post_audio_port_t *port; - - _x_post_init( this, 1, 0 ); - - port = _x_post_intercept_audio_port( this, audio_target, &input, &output ); - port->new_port.open = scope_port_open; - port->new_port.close = scope_port_close; - port->new_port.put_buffer = scope_port_put_buffer; - - this->xine_post.audio_input[0] = &port->new_port; - this->xine_post.type = PLUGIN_POST; - - this->dispose = scope_dispose; - } - - /* code is straight from xine_init_post() - can't use that function as it only dlopens the plugins - and our plugin is statically linked in */ - - post_plugin->running_ticket = xine->port_ticket; - post_plugin->xine = xine; - - return &post_plugin->xine_post; -} +/* Author: Max Howell , (C) 2004 + Copyright: See COPYING file that comes with this distribution */ + +/* gcc doesn't like inline for me */ +#define inline +/* need access to port_ticket */ +#define XINE_ENGINE_INTERNAL + +#include "xineScope.h" +#include +#include + + +static MyNode theList; +static metronom_t theMetronom; +static int myChannels = 0; + +MyNode* const myList = &theList; +metronom_t* const myMetronom = &theMetronom; + + +/* defined in xineEngine.cpp */ +extern void _debug( const char * ); + + +/************************* +* post plugin functions * +*************************/ + +static int +scope_port_open( xine_audio_port_t *port_gen, xine_stream_t *stream, uint32_t bits, uint32_t rate, int mode ) +{ + _debug( "scope_port_open()\n" ); + + #define port ((post_audio_port_t*)port_gen) + + _x_post_rewire( (post_plugin_t*)port->post ); + _x_post_inc_usage( port ); + + port->stream = stream; + port->bits = bits; + port->rate = rate; + port->mode = mode; + + myChannels = _x_ao_mode2channels( mode ); + + return port->original_port->open( port->original_port, stream, bits, rate, mode ); +} + +static void +scope_port_close( xine_audio_port_t *port_gen, xine_stream_t *stream ) +{ + _debug( "scope_port_close()\n" ); + + port->stream = NULL; + port->original_port->close( port->original_port, stream ); + + _x_post_dec_usage( port ); +} + +static void +scope_port_put_buffer( xine_audio_port_t *port_gen, audio_buffer_t *buf, xine_stream_t *stream ) +{ +#if XINE_MAJOR_VERSION < 1 || (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION < 2) || \ + (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION == 2 && XINE_SUB_VERSION < 10) + MyNode *new_node; + const int num_samples = buf->num_frames * myChannels; + + /* we are too simple to handle 8bit */ + /* what does it mean when stream == NULL? */ + if( port->bits == 8 ) { + port->original_port->put_buffer( port->original_port, buf, stream ); return; } + + /* I keep my own metronom because xine wouldn't for some reason */ + memcpy( myMetronom, stream->metronom, sizeof(metronom_t) ); + + new_node = malloc( sizeof(MyNode) ); + new_node->vpts = myMetronom->got_audio_samples( myMetronom, buf->vpts, buf->num_frames ); + new_node->num_frames = buf->num_frames; + new_node->mem = malloc( num_samples * 2 ); + memcpy( new_node->mem, buf->mem, num_samples * 2 ); + + { + int64_t + K = myMetronom->pts_per_smpls; /*smpls = 1<<16 samples*/ + K *= num_samples; + K /= (1<<16); + K += new_node->vpts; + + new_node->vpts_end = K; + } + + /* pass data to original port */ + port->original_port->put_buffer( port->original_port, buf, stream ); + + /* finally we should append the current buffer to the list + * NOTE this is thread-safe due to the way we handle the list in the GUI thread */ + new_node->next = myList->next; + myList->next = new_node; +#endif + + #undef port +} + +static void +scope_dispose( post_plugin_t *this ) +{ + free( this ); +} + + +/************************ +* plugin init function * +************************/ + +xine_post_t* +scope_plugin_new( xine_t *xine, xine_audio_port_t *audio_target ) +{ + if( audio_target == NULL ) + return NULL; + + post_plugin_t *post_plugin = xine_xmalloc( sizeof(post_plugin_t) ); + + { + post_plugin_t *this = post_plugin; + post_in_t *input; + post_out_t *output; + post_audio_port_t *port; + + _x_post_init( this, 1, 0 ); + + port = _x_post_intercept_audio_port( this, audio_target, &input, &output ); + port->new_port.open = scope_port_open; + port->new_port.close = scope_port_close; + port->new_port.put_buffer = scope_port_put_buffer; + + this->xine_post.audio_input[0] = &port->new_port; + this->xine_post.type = PLUGIN_POST; + + this->dispose = scope_dispose; + } + +#if XINE_MAJOR_VERSION < 1 || (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION < 2) || \ + (XINE_MAJOR_VERSION == 1 && XINE_MINOR_VERSION == 2 && XINE_SUB_VERSION < 10) + /* code is straight from xine_init_post() + can't use that function as it only dlopens the plugins + and our plugin is statically linked in */ + + post_plugin->running_ticket = xine->port_ticket; + post_plugin->xine = xine; +#endif + + return &post_plugin->xine_post; +}