reorganized consumer_westley.

[melted] / src / modules / resample / filter_resample.c

diff --git a/src/modules/resample/filter_resample.c b/src/modules/resample/filter_resample.c
index 57f9aad..2c848a8 100644 (file)
--- a/src/modules/resample/filter_resample.c
+++ b/src/modules/resample/filter_resample.c
@@ -58,11 +58,9 @@ static int resample_get_audio( mlt_frame frame, int16_t **buffer, mlt_audio_form
        // Duplicate channels as necessary
        if ( channels_avail < *channels )
        {
-               int size = *channels * *samples * 2;
+               int size = *channels * *samples * sizeof( int16_t );
                int16_t *new_buffer = mlt_pool_alloc( size );
                
-               mlt_properties_set_data( properties, "audio", new_buffer, size, ( mlt_destructor )mlt_pool_release, NULL );
-               
                // Duplicate the existing channels
                for ( i = 0; i < *samples; i++ )
                {
@@ -74,10 +72,31 @@ static int resample_get_audio( mlt_frame frame, int16_t **buffer, mlt_audio_form
                        }
                }
                
+               // Update the audio buffer now - destroys the old
+               mlt_properties_set_data( properties, "audio", new_buffer, size, ( mlt_destructor )mlt_pool_release, NULL );
+               
+               *buffer = new_buffer;
+       }
+       else if ( channels_avail > *channels )
+       {
+               // Nasty hack for ac3 5.1 audio - may be a cause of failure?
+               int size = *channels * *samples * sizeof( int16_t );
+               int16_t *new_buffer = mlt_pool_alloc( size );
+               
+               // Drop all but the first *channels
+               for ( i = 0; i < *samples; i++ )
+               {
+                       new_buffer[ ( i * *channels ) + 0 ] = (*buffer)[ ( i * channels_avail ) + 2 ];
+                       new_buffer[ ( i * *channels ) + 1 ] = (*buffer)[ ( i * channels_avail ) + 3 ];
+               }
+
+               // Update the audio buffer now - destroys the old
+               mlt_properties_set_data( properties, "audio", new_buffer, size, ( mlt_destructor )mlt_pool_release, NULL );
+               
                *buffer = new_buffer;
        }
 
-       // Return now if now work to do
+       // Return now if no work to do
        if ( output_rate == *frequency )
                return 0;
 
@@ -130,24 +149,27 @@ static int resample_get_audio( mlt_frame frame, int16_t **buffer, mlt_audio_form
 
 static mlt_frame filter_process( mlt_filter this, mlt_frame frame )
 {
-       mlt_properties properties = mlt_filter_properties( this );
-       mlt_properties frame_props = mlt_frame_properties( frame );
-
-       // Propogate the frequency property if supplied
-       if ( mlt_properties_get( properties, "frequency" ) != NULL )
-               mlt_properties_set_int( frame_props, "resample.frequency", mlt_properties_get_int( properties, "frequency" ) );
-
-       // Propogate the other properties
-       mlt_properties_set_int( frame_props, "resample.channels", mlt_properties_get_int( properties, "channels" ) );
-       mlt_properties_set_data( frame_props, "resample.state", mlt_properties_get_data( properties, "state", NULL ), 0, NULL, NULL );
-       mlt_properties_set_data( frame_props, "resample.input_buffer", mlt_properties_get_data( properties, "input_buffer", NULL ), 0, NULL, NULL );
-       mlt_properties_set_data( frame_props, "resample.output_buffer", mlt_properties_get_data( properties, "output_buffer", NULL ), 0, NULL, NULL );
+       if ( frame->get_audio != NULL )
+       {
+               mlt_properties properties = mlt_filter_properties( this );
+               mlt_properties frame_props = mlt_frame_properties( frame );
+
+               // Propogate the frequency property if supplied
+               if ( mlt_properties_get( properties, "frequency" ) != NULL )
+                       mlt_properties_set_int( frame_props, "resample.frequency", mlt_properties_get_int( properties, "frequency" ) );
+
+               // Propogate the other properties
+               mlt_properties_set_int( frame_props, "resample.channels", mlt_properties_get_int( properties, "channels" ) );
+               mlt_properties_set_data( frame_props, "resample.state", mlt_properties_get_data( properties, "state", NULL ), 0, NULL, NULL );
+               mlt_properties_set_data( frame_props, "resample.input_buffer", mlt_properties_get_data( properties, "input_buffer", NULL ), 0, NULL, NULL );
+               mlt_properties_set_data( frame_props, "resample.output_buffer", mlt_properties_get_data( properties, "output_buffer", NULL ), 0, NULL, NULL );
        
-       // Backup the original get_audio (it's still needed)
-       mlt_properties_set_data( frame_props, "resample.get_audio", frame->get_audio, 0, NULL, NULL );
+               // Backup the original get_audio (it's still needed)
+               mlt_properties_set_data( frame_props, "resample.get_audio", frame->get_audio, 0, NULL, NULL );
 
-       // Override the get_audio method
-       frame->get_audio = resample_get_audio;
+               // Override the get_audio method
+               frame->get_audio = resample_get_audio;
+       }
 
        return frame;
 }