provisional framework docs and corrections
[melted] / src / modules / sdl / consumer_sdl.c
index d35e4a7..589194a 100644 (file)
@@ -27,7 +27,7 @@
 #include <pthread.h>
 #include <SDL/SDL.h>
 #include <SDL/SDL_syswm.h>
-#include <sys/timeb.h>
+#include <sys/time.h>
 
 /** This classes definition.
 */
@@ -41,7 +41,7 @@ struct consumer_sdl_s
        mlt_deque queue;
        pthread_t thread;
        int running;
-       uint8_t audio_buffer[ 4096 * 6 ];
+       uint8_t audio_buffer[ 4096 * 19 ];
        int audio_avail;
        pthread_mutex_t audio_mutex;
        pthread_cond_t audio_cond;
@@ -102,8 +102,11 @@ mlt_consumer consumer_sdl_init( char *arg )
                // Default scaler (for now we'll use nearest)
                mlt_properties_set( this->properties, "rescale", "nearest" );
 
+               // Default buffer for low latency
+               mlt_properties_set_int( this->properties, "buffer", 8 );
+
                // Default progressive true
-               mlt_properties_set_int( this->properties, "progressive", 1 );
+               mlt_properties_set_int( this->properties, "progressive", 0 );
 
                // Get aspect ratio
                this->aspect_ratio = mlt_properties_get_double( this->properties, "aspect_ratio" );
@@ -114,9 +117,9 @@ mlt_consumer consumer_sdl_init( char *arg )
                        this->width = mlt_properties_get_int( this->properties, "width" );
                        this->height = mlt_properties_get_int( this->properties, "height" );
                }
-
+               
                // Default window size
-               this->window_width = (int)( (float)this->height * this->aspect_ratio ) + 1;
+               this->window_width = (int)( (float)this->height * 4.0/3.0 + 0.5);
                this->window_height = this->height;
                
                // Set the sdl flags
@@ -260,6 +263,7 @@ static int consumer_play_audio( consumer_sdl this, mlt_frame frame, int init_aud
        if ( mlt_properties_get_int( properties, "audio_off" ) )
        {
                this->playing = 1;
+               init_audio = 1;
                return init_audio;
        }
 
@@ -268,16 +272,13 @@ static int consumer_play_audio( consumer_sdl this, mlt_frame frame, int init_aud
                SDL_AudioSpec request;
                SDL_AudioSpec got;
 
-               SDL_EnableKeyRepeat( SDL_DEFAULT_REPEAT_DELAY, SDL_DEFAULT_REPEAT_INTERVAL );
-               SDL_EnableUNICODE( 1 );
-
                // specify audio format
                memset( &request, 0, sizeof( SDL_AudioSpec ) );
                this->playing = 0;
                request.freq = frequency;
                request.format = AUDIO_S16;
                request.channels = channels;
-               request.samples = 2048;
+               request.samples = 1024;
                request.callback = sdl_fill_audio;
                request.userdata = (void *)this;
                if ( SDL_OpenAudio( &request, &got ) != 0 )
@@ -294,11 +295,11 @@ static int consumer_play_audio( consumer_sdl this, mlt_frame frame, int init_aud
 
        if ( init_audio == 0 )
        {
+               mlt_properties properties = mlt_frame_properties( frame );
                bytes = ( samples * channels * 2 );
                pthread_mutex_lock( &this->audio_mutex );
                while ( bytes > ( sizeof( this->audio_buffer) - this->audio_avail ) )
                        pthread_cond_wait( &this->audio_cond, &this->audio_mutex );
-               mlt_properties properties = mlt_frame_properties( frame );
                if ( mlt_properties_get_double( properties, "_speed" ) == 1 )
                        memcpy( &this->audio_buffer[ this->audio_avail ], pcm, bytes );
                else
@@ -315,7 +316,7 @@ static int consumer_play_audio( consumer_sdl this, mlt_frame frame, int init_aud
        return init_audio;
 }
 
-static int consumer_play_video( consumer_sdl this, mlt_frame frame, int64_t elapsed, int skip, int64_t playtime )
+static int consumer_play_video( consumer_sdl this, mlt_frame frame )
 {
        // Get the properties of this consumer
        mlt_properties properties = this->properties;
@@ -324,103 +325,47 @@ static int consumer_play_video( consumer_sdl this, mlt_frame frame, int64_t elap
        int width = this->width, height = this->height;
        uint8_t *image;
        int changed = 0;
-       int show = 1;
-
-       if ( mlt_properties_get_int( properties, "video_off" ) )
-       {
-               mlt_frame_close( frame );
-               return 0;
-       }
 
-       // Handle events
-       if ( this->sdl_screen != NULL )
+       if ( mlt_properties_get_int( properties, "video_off" ) == 0 )
        {
-               SDL_Event event;
-
-               changed = consumer_get_dimensions( &this->window_width, &this->window_height );
+               // Get the image, width and height
+               mlt_frame_get_image( frame, &image, &vfmt, &width, &height, 0 );
 
-               while ( SDL_PollEvent( &event ) )
+               // Handle events
+               if ( this->sdl_screen != NULL )
                {
-                       switch( event.type )
+                       SDL_Event event;
+       
+                       changed = consumer_get_dimensions( &this->window_width, &this->window_height );
+       
+                       while ( SDL_PollEvent( &event ) )
                        {
-                               case SDL_VIDEORESIZE:
-                                       this->window_width = event.resize.w;
-                                       this->window_height = event.resize.h;
-                                       changed = 1;
-                                       break;
-                               case SDL_KEYDOWN:
-                                       {
-                                               mlt_producer producer = mlt_properties_get_data( properties, "transport_producer", NULL );
-                                               char keyboard[ 2 ] = " ";
-                                               void (*callback)( mlt_producer, char * ) = mlt_properties_get_data( properties, "transport_callback", NULL );
-                                               if ( callback != NULL && producer != NULL && event.key.keysym.unicode < 0x80 && event.key.keysym.unicode > 0 )
+                               switch( event.type )
+                               {
+                                       case SDL_VIDEORESIZE:
+                                               this->window_width = event.resize.w;
+                                               this->window_height = event.resize.h;
+                                               changed = 1;
+                                               break;
+                                       case SDL_QUIT:
+                                               this->running = 0;
+                                               break;
+                                       case SDL_KEYDOWN:
                                                {
-                                                       keyboard[ 0 ] = ( char )event.key.keysym.unicode;
-                                                       callback( producer, keyboard );
+                                                       mlt_producer producer = mlt_properties_get_data( properties, "transport_producer", NULL );
+                                                       char keyboard[ 2 ] = " ";
+                                                       void (*callback)( mlt_producer, char * ) = mlt_properties_get_data( properties, "transport_callback", NULL );
+                                                       if ( callback != NULL && producer != NULL && event.key.keysym.unicode < 0x80 && event.key.keysym.unicode > 0 )
+                                                       {
+                                                               keyboard[ 0 ] = ( char )event.key.keysym.unicode;
+                                                               callback( producer, keyboard );
+                                                       }
                                                }
-                                       }
-                                       break;
-                       }
-               }
-       }
-
-       // Set skip
-       mlt_properties_set_int( mlt_frame_properties( frame ), "skip", skip );
-       mlt_properties_set_position( mlt_frame_properties( frame ), "playtime", playtime );
-
-       // Push this frame to the back of the queue
-       mlt_deque_push_back( this->queue, frame );
-       frame = NULL;
-
-       if ( this->playing )
-       {
-               // We might want to use an old frame if the current frame is skipped
-               mlt_frame candidate = NULL;
-
-               while ( frame == NULL && mlt_deque_count( this->queue ) )
-               {
-                       frame = mlt_deque_peek_front( this->queue );
-                       show = !mlt_properties_get_int( mlt_frame_properties( frame ), "skip" );
-                       playtime = mlt_properties_get_position( mlt_frame_properties( frame ), "playtime" );
-
-                       // Check if frame is in the future or past
-                       if ( !show )
-                       {
-                               frame = mlt_deque_pop_front( this->queue );
-                               mlt_frame_close( frame );
-                               frame = NULL;
-                       }
-                       else if ( playtime > elapsed )
-                       {
-                               // no frame to show or remove
-                               frame = NULL;
-                               break;
-                       }
-                       else if ( playtime < elapsed - 40 )
-                       {
-                               if ( candidate != NULL )
-                                       mlt_frame_close( candidate );
-                               candidate = mlt_deque_pop_front( this->queue );
-                               frame = NULL;
-                       }
-                       else
-                       {
-                               // Get the frame at the front of the queue
-                               frame = mlt_deque_pop_front( this->queue );
+                                               break;
+                               }
                        }
                }
-
-               if ( frame == NULL )
-                       frame = candidate;
-               else if ( candidate != NULL )
-                       mlt_frame_close( candidate );
-       }
-
-       if ( this->playing && frame != NULL )
-       {
-               // Get the image, width and height
-               mlt_frame_get_image( frame, &image, &vfmt, &width, &height, 0 );
-
+       
                if ( width != this->width || height != this->height )
                {
                        this->width = width;
@@ -430,12 +375,12 @@ static int consumer_play_video( consumer_sdl this, mlt_frame frame, int64_t elap
 
                if ( this->sdl_screen == NULL || changed )
                {
-                       double aspect_ratio = mlt_frame_get_aspect_ratio( frame );
-                       float display_aspect_ratio = (float)width / (float)height;
+                       double aspect_ratio = ( float )this->aspect_ratio * this->width / this->height;
+                       float display_aspect_ratio = ( float )width / height;
                        SDL_Rect rect;
-
-                       if ( mlt_properties_get_double( properties, "aspect_ratio" ) )
-                               aspect_ratio = mlt_properties_get_double( properties, "aspect_ratio" );
+                       if ( mlt_properties_get( properties, "rescale" ) != NULL &&
+                                       !strcmp( mlt_properties_get( properties, "rescale" ), "none" ) )
+                               aspect_ratio = mlt_frame_get_aspect_ratio( frame ) * mlt_properties_get_int(mlt_frame_properties(frame),"width") / mlt_properties_get_int(mlt_frame_properties(frame),"height");
 
                        if ( aspect_ratio == 1 )
                        {
@@ -445,11 +390,11 @@ static int consumer_play_video( consumer_sdl this, mlt_frame frame, int64_t elap
                        else if ( this->window_width < this->window_height * aspect_ratio )
                        {
                                rect.w = this->window_width;
-                               rect.h = this->window_width / aspect_ratio;
+                               rect.h = this->window_width / aspect_ratio + 1;
                        }
                        else
                        {
-                               rect.w = this->window_height * aspect_ratio;
+                               rect.w = this->window_height * aspect_ratio + 1;
                                rect.h = this->window_height;
                        }
 
@@ -494,17 +439,7 @@ static int consumer_play_video( consumer_sdl this, mlt_frame frame, int64_t elap
        }
 
        // Close the frame
-       if ( frame != NULL )
-               mlt_frame_close( frame );
-
-       if ( mlt_deque_count( this->queue ) && !skip )
-       {
-               // Tell the producers about our scale relative to the normalisation
-               frame = mlt_deque_peek_front( this->queue );
-               mlt_properties_set_double( mlt_frame_properties( frame ), "consumer_scale",
-                       ( double )height / mlt_properties_get_double( properties, "height" ) );
-               mlt_frame_get_image( frame, &image, &vfmt, &width, &height, 0 );
-       }
+       mlt_frame_close( frame );
 
        return 0;
 }
@@ -524,57 +459,92 @@ static void *consumer_thread( void *arg )
        int init_audio = 1;
 
        // Obtain time of thread start
-       struct timeb now;
+       struct timeval now;
        int64_t start = 0;
        int64_t elapsed = 0;
        int duration = 0;
        int64_t playtime = 0;
-       int skip = 0;
+       struct timespec tm;
+       mlt_frame next = NULL;
+       mlt_frame frame = NULL;
 
        if ( SDL_Init( SDL_INIT_VIDEO | SDL_INIT_AUDIO | SDL_INIT_NOPARACHUTE ) < 0 )
        {
                fprintf( stderr, "Failed to initialize SDL: %s\n", SDL_GetError() );
                return NULL;
        }
-       
+
+       SDL_EnableKeyRepeat( SDL_DEFAULT_REPEAT_DELAY, SDL_DEFAULT_REPEAT_INTERVAL );
+       SDL_EnableUNICODE( 1 );
+
        // Loop until told not to
        while( this->running )
        {
                // Get a frame from the attached producer
-               mlt_frame frame = mlt_consumer_get_frame( consumer );
+               frame = mlt_consumer_rt_frame( consumer );
 
                // Ensure that we have a frame
                if ( frame != NULL )
                {
-                       skip = 0;
-
-                       // SDL adapts display aspect, but set this so pixel aspect can be normalised
-//                     mlt_properties_set_double( mlt_frame_properties( frame ), "consumer_aspect_ratio",
-//                             mlt_frame_get_aspect_ratio( frame ) );
-                       
                        // Play audio
                        init_audio = consumer_play_audio( this, frame, init_audio, &duration );
 
                        if ( this->playing )
                        {
                                // Get the current time
-                               ftime( &now );
+                               gettimeofday( &now, NULL );
 
                                // Determine elapsed time
                                if ( start == 0 )
-                                       start = ( int64_t )now.time * 1000 + now.millitm;
+                                       start = ( int64_t )now.tv_sec * 1000000 + now.tv_usec;
                                else
-                                       elapsed = ( ( int64_t )now.time * 1000 + now.millitm ) - start;
+                                       elapsed = ( ( int64_t )now.tv_sec * 1000000 + now.tv_usec) - start;
+                       }
 
-                               skip = playtime < elapsed;
+                       // Set playtime for this frame
+                       mlt_properties_set_position( mlt_frame_properties( frame ), "playtime", playtime );
 
-                               if ( skip )
-                                       start = start + ( elapsed - playtime );
-                       }
+                       // Push this frame to the back of the queue
+                       mlt_deque_push_back( this->queue, frame );
+
+                       // Calculate the next playtime
+                       playtime += ( duration * 1000 );
+               }
 
-                       consumer_play_video( this, frame, elapsed, skip, playtime );
+               if ( this->playing )
+               {
+                       // Pop the next frame
+                       next = mlt_deque_pop_front( this->queue );
 
-                       playtime += duration;
+                       // See if we have to delay the display of the current frame
+                       if ( next != NULL && mlt_properties_get_int( mlt_frame_properties( next ), "rendered" ) == 1 )
+                       {
+                               mlt_position scheduled = mlt_properties_get_position( mlt_frame_properties( next ), "playtime" ) + 5000;
+                               if ( scheduled > elapsed && mlt_deque_count( this->queue ) > 25 )
+                               {
+                                       tm.tv_sec = ( scheduled - elapsed ) / 1000000;
+                                       tm.tv_nsec = ( ( scheduled - elapsed ) % 1000000 ) * 1000;
+                                       nanosleep( &tm, NULL );
+
+                                       // Show current frame
+                                       consumer_play_video( this, next );
+                               }
+                               else if ( scheduled > elapsed )
+                               {
+                                       // More time to kill
+                                       mlt_deque_push_front( this->queue, next );
+                               }
+                               else
+                               {
+                                       // Show current frame
+                                       consumer_play_video( this, next );
+                               }
+                       }
+                       else
+                       {
+                               // This is an unrendered frame - just close it
+                               mlt_frame_close( next );
+                       }
                }
        }
 
@@ -654,4 +624,3 @@ static void consumer_close( mlt_consumer parent )
        // Finally clean up this
        free( this );
 }
-