X-Git-Url: http://research.m1stereo.tv/gitweb?a=blobdiff_plain;f=src%2Fmodules%2Fgtk2%2Fproducer_pango.c;h=4756213eef8e22deed81af2c990fb3b08c85e3cc;hb=70933f360aa7d16e5140267ce2f716de924464c7;hp=52bf2ff20bb12454769a6ffd752b64c900429b61;hpb=6fbad87cfc97a367c4e47216b83c9e433253aa9e;p=melted

diff --git a/src/modules/gtk2/producer_pango.c b/src/modules/gtk2/producer_pango.c
index 52bf2ff..4756213 100644
--- a/src/modules/gtk2/producer_pango.c
+++ b/src/modules/gtk2/producer_pango.c
@@ -18,7 +18,7 @@
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  */
 
-#include "producer_pango.h"
+#include <framework/mlt_producer.h>
 #include <framework/mlt_frame.h>
 #include <framework/mlt_geometry.h>
 #include <stdlib.h>
@@ -30,6 +30,15 @@
 #include <pthread.h>
 #include <ctype.h>
 
+typedef struct producer_pango_s *producer_pango;
+
+typedef enum
+{
+	pango_align_left = 0,
+	pango_align_center,
+	pango_align_right
+} pango_align;
+
 static pthread_mutex_t pango_mutex = PTHREAD_MUTEX_INITIALIZER;
 
 struct producer_pango_s
@@ -433,8 +442,8 @@ static void refresh_image( mlt_frame frame, int width, int height )
 		else if ( strcmp( interps, "hyper" ) == 0 )
 			interp = GDK_INTERP_HYPER;
 
-//		fprintf( stderr, "SCALING PANGO from %dx%d to %dx%d was %dx%d\n", gdk_pixbuf_get_width( pixbuf ), gdk_pixbuf_get_height( pixbuf ), width, height, this->width, this->height );
-			
+// fprintf(stderr,"%s: scaling from %dx%d to %dx%d\n", __FILE__, this->width, this->height, width, height);
+
 		// Note - the original pixbuf is already safe and ready for destruction
 		pixbuf = gdk_pixbuf_scale_simple( pixbuf, width, height, interp );
 
@@ -544,7 +553,7 @@ static int producer_get_frame( mlt_producer producer, mlt_frame_ptr frame, int i
 	producer_pango this = producer->child;
 
 	// Generate a frame
-	*frame = mlt_frame_init( );
+	*frame = mlt_frame_init( MLT_PRODUCER_SERVICE( producer ) );
 
 	// Obtain properties of frame and producer
 	mlt_properties properties = MLT_FRAME_PROPERTIES( *frame );
@@ -629,7 +638,7 @@ static GdkPixbuf *pango_get_pixbuf( const char *markup, const char *text, const
 	pango_layout_set_width( layout, -1 ); // set wrapping constraints
 	pango_font_description_set_weight( desc, ( PangoWeight ) weight  );
 	if ( size != 0 )
-		pango_font_description_set_size( desc, PANGO_SCALE * size );
+		pango_font_description_set_absolute_size( desc, PANGO_SCALE * size );
 	pango_layout_set_font_description( layout, desc );
 //	pango_layout_set_spacing( layout, space );
 	pango_layout_set_alignment( layout, ( PangoAlignment ) align  );
@@ -653,6 +662,19 @@ static GdkPixbuf *pango_get_pixbuf( const char *markup, const char *text, const
 	}
 	pango_layout_get_pixel_size( layout, &w, &h );
 
+	// Interpret size property as an absolute pixel height and compensate for
+	// freetype's "interpretation" of our absolute size request. This gives
+	// precise control over compositing and better quality by reducing scaling
+	// artifacts with composite geometries that constrain the dimensions.
+	// If you do not want this, then put the size in the font property or in
+	// the pango markup.
+	if ( size != 0 )
+	{
+		pango_font_description_set_absolute_size( desc, PANGO_SCALE * size * size/h );
+		pango_layout_set_font_description( layout, desc );
+		pango_layout_get_pixel_size( layout, &w, &h );
+	}
+
 	pixbuf = gdk_pixbuf_new( GDK_COLORSPACE_RGB, TRUE /* has alpha */, 8, w + 2 * pad, h + 2 * pad );
 	pango_draw_background( pixbuf, bg );