bugfix limits in transition luma
[melted] / src / modules / core / transition_luma.c
index bba678d..0243974 100644 (file)
@@ -78,6 +78,10 @@ static inline int dissolve_yuv( mlt_frame this, mlt_frame that, float weight, in
        mlt_frame_get_image( this, &p_dest, &format, &width, &height, 1 );
        mlt_frame_get_image( that, &p_src, &format, &width_src, &height_src, 0 );
 
+       // Pick the lesser of two evils ;-)
+       width_src = width_src > width ? width : width_src;
+       height_src = height_src > height ? height : height_src;
+       
        p = p_dest;
        limit = p_dest + height_src * width_src * 2;
 
@@ -92,7 +96,7 @@ static inline int dissolve_yuv( mlt_frame this, mlt_frame that, float weight, in
 
 // image processing functions
 
-static inline int32_t smoothstep( int32_t edge1, int32_t edge2, int32_t a )
+static inline int32_t smoothstep( int32_t edge1, int32_t edge2, uint32_t a )
 {
        if ( a < edge1 )
                return 0;
@@ -128,6 +132,10 @@ static void luma_composite( mlt_frame a_frame, mlt_frame b_frame, int luma_width
        mlt_frame_get_image( a_frame, &p_dest, &format_dest, &width_dest, &height_dest, 1 );
        mlt_frame_get_image( b_frame, &p_src, &format_src, &width_src, &height_src, 0 );
 
+       // Pick the lesser of two evils ;-)
+       width_src = width_src > width_dest ? width_dest : width_src;
+       height_src = height_src > height_dest ? height_dest : height_src;
+       
        stride_src = width_src * 2;
        stride_dest = width_dest * 2;
 
@@ -437,8 +445,9 @@ static int transition_get_image( mlt_frame a_frame, uint8_t **image, mlt_image_f
        float frame_delta = delta_calculate( transition, a_frame );
        
        float luma_softness = mlt_properties_get_double( properties, "softness" );
-       int progressive = mlt_properties_get_int( b_props, "progressive" ) ||
+       int progressive = 
                        mlt_properties_get_int( a_props, "consumer_progressive" ) ||
+                       mlt_properties_get_int( properties, "progressive" ) ||
                        mlt_properties_get_int( b_props, "luma.progressive" );
        int top_field_first =  mlt_properties_get_int( b_props, "top_field_first" );
        int reverse = mlt_properties_get_int( properties, "reverse" );