X-Git-Url: http://research.m1stereo.tv/gitweb?a=blobdiff_plain;f=src%2Fmodules%2Fcore%2Fproducer_colour.c;h=415b595eed325fb018affda975f003e64e2ad532;hb=521423f4869254f213455a239688159c283594b0;hp=6ff180ff966cb08f972ac60c462e2f0b8613f020;hpb=6820dfa615e2f55eb4ef17327c2cd293605c3389;p=melted

diff --git a/src/modules/core/producer_colour.c b/src/modules/core/producer_colour.c
index 6ff180f..415b595 100644
--- a/src/modules/core/producer_colour.c
+++ b/src/modules/core/producer_colour.c
@@ -48,9 +48,7 @@ mlt_producer producer_colour_init( char *colour )
 		producer->close = producer_close;
 
 		// Set the default properties
-		if ( colour == NULL )
-			colour = "0x000000ff";
-		mlt_properties_set( properties, "resource", colour );
+		mlt_properties_set( properties, "resource", colour == NULL ? "0x000000ff" : colour );
 		
 		return producer;
 	}
@@ -124,7 +122,6 @@ static int producer_get_image( mlt_frame frame, uint8_t **buffer, mlt_image_form
 	// See if we need to regenerate
 	if ( *width != current_width || *height != current_height )
 	{
-	fprintf( stderr, "%dx%d\n", *width, *height );
 		// Allocate the image
 		size = *width * *height * 2;
 		image = mlt_pool_alloc( size );
@@ -137,14 +134,16 @@ static int producer_get_image( mlt_frame frame, uint8_t **buffer, mlt_image_form
 		// Color the image
 		rgba_color color = parse_color( mlt_properties_get( producer_props, "resource" ) );
 		uint8_t y, u, v;
-		int i;
+		int i = 0;
 		RGB2YUV( color.r, color.g, color.b, y, u, v );
-		color.r = y;
-		color.g = u;
-		color.b = y;
-		color.a = v;
-		for ( i = 0; i < size; i += 4 )
-			memcpy( &image[ i ], &color, 4 );
+
+		while ( i < size )
+		{
+			image[ i ++ ] = y;
+			image[ i ++ ] = u;
+			image[ i ++ ] = y;
+			image[ i ++ ] = v;
+		}
 	}
 
 	// Update the frame
@@ -194,8 +193,10 @@ static int producer_get_frame( mlt_producer producer, mlt_frame_ptr frame, int i
 
 		// Set producer-specific frame properties
 		mlt_properties_set_int( properties, "progressive", 1 );
+		mlt_properties_set_int( properties, "aspect_ratio", 1 );
 
 		// colour is an alias for resource
+		// CY: Do we really need this?
 		if ( mlt_properties_get( producer_props, "colour" ) != NULL )
 			mlt_properties_set( producer_props, "resource", mlt_properties_get( producer_props, "colour" ) );