]> git.sesse.net Git - vlc/blobdiff - modules/misc/freetype.c
ALL: backport of 13001,13004,13005,13011,13012,13019,13020,13025 from trunk.
[vlc] / modules / misc / freetype.c
index babe744f278e434716a2380daa98c5817cbbced3..98fd7abba941ead2298a8e4e340b79ee2942ed06 100644 (file)
@@ -1,10 +1,10 @@
 /*****************************************************************************
  * freetype.c : Put text on the video, using freetype2
  *****************************************************************************
- * Copyright (C) 2002, 2003 VideoLAN
+ * Copyright (C) 2002 - 2005 the VideoLAN team
  * $Id$
  *
- * Authors: Sigmund Augdal <sigmunau@idi.ntnu.no>
+ * Authors: Sigmund Augdal Helberg <dnumgis@videolan.org>
  *          Gildas Bazin <gbazin@videolan.org>
  *
  * This program is free software; you can redistribute it and/or modify
@@ -34,7 +34,7 @@
 
 #include <vlc/vlc.h>
 #include <vlc/vout.h>
-#include "osd.h"
+#include "vlc_osd.h"
 #include "vlc_block.h"
 #include "vlc_filter.h"
 
@@ -70,9 +70,13 @@ typedef struct line_desc_t line_desc_t;
 static int  Create ( vlc_object_t * );
 static void Destroy( vlc_object_t * );
 
-static subpicture_t *RenderText( filter_t *, block_t * );
+/* The RenderText call maps to pf_render_string, defined in vlc_filter.h */
+static int RenderText( filter_t *, subpicture_region_t *,
+                       subpicture_region_t * );
 static line_desc_t *NewLine( byte_t * );
 
+static int SetFontSize( filter_t *, int );
+
 /*****************************************************************************
  * Module descriptor
  *****************************************************************************/
@@ -82,20 +86,50 @@ static line_desc_t *NewLine( byte_t * );
 #define FONTSIZE_LONGTEXT N_("The size of the fonts used by the osd module. " \
     "If set to something different than 0 this option will override the " \
     "relative font size " )
+#define OPACITY_TEXT N_("Opacity, 0..255")
+#define OPACITY_LONGTEXT N_("The opacity (inverse of transparency) of " \
+    "overlay text. 0 = transparent, 255 = totally opaque. " )
+#define COLOR_TEXT N_("Text Default Color")
+#define COLOR_LONGTEXT N_("The color of overlay text. 1 byte for each color, "\
+    "hexadecimal. #000000 = all colors off, 0xFF0000 = just Red, " \
+    "0xFFFFFF = all color on [White]" )
 #define FONTSIZER_TEXT N_("Font size")
 #define FONTSIZER_LONGTEXT N_("The size of the fonts used by the osd module" )
 
 static int   pi_sizes[] = { 20, 18, 16, 12, 6 };
 static char *ppsz_sizes_text[] = { N_("Smaller"), N_("Small"), N_("Normal"),
                                    N_("Large"), N_("Larger") };
+static int pi_color_values[] = {
+  0x00000000, 0x00808080, 0x00C0C0C0, 0x00FFFFFF, 0x00800000,
+  0x00FF0000, 0x00FF00FF, 0x00FFFF00, 0x00808000, 0x00008000, 0x00008080, 
+  0x0000FF00, 0x00800080, 0x00000080, 0x000000FF, 0x0000FFFF }; 
+
+static char *ppsz_color_descriptions[] = {
+  N_("Black"), N_("Gray"), N_("Silver"), N_("White"), N_("Maroon"),
+  N_("Red"), N_("Fuchsia"), N_("Yellow"), N_("Olive"), N_("Green"), N_("Teal"),
+  N_("Lime"), N_("Purple"), N_("Navy"), N_("Blue"), N_("Aqua") };
 
 vlc_module_begin();
-    set_description( _("freetype2 font renderer") );
+    set_shortname( _("Text renderer"));
+    set_description( _("Freetype2 font renderer") );
+    set_category( CAT_VIDEO );
+    set_subcategory( SUBCAT_VIDEO_SUBPIC );
 
     add_file( "freetype-font", DEFAULT_FONT, NULL, FONT_TEXT, FONT_LONGTEXT,
               VLC_FALSE );
+
     add_integer( "freetype-fontsize", 0, NULL, FONTSIZE_TEXT,
                  FONTSIZE_LONGTEXT, VLC_TRUE );
+
+    /* opacity valid on 0..255, with default 255 = fully opaque */
+    add_integer_with_range( "freetype-opacity", 255, 0, 255, NULL,
+        OPACITY_TEXT, OPACITY_LONGTEXT, VLC_TRUE );
+
+    /* hook to the color values list, with default 0x00ffffff = white */
+    add_integer( "freetype-color", 0x00FFFFFF, NULL, COLOR_TEXT,
+                 COLOR_LONGTEXT, VLC_FALSE );
+        change_integer_list( pi_color_values, ppsz_color_descriptions, 0 );
+
     add_integer( "freetype-rel-fontsize", 16, NULL, FONTSIZER_TEXT,
                  FONTSIZER_LONGTEXT, VLC_FALSE );
         change_integer_list( pi_sizes, ppsz_sizes_text, 0 );
@@ -105,32 +139,24 @@ vlc_module_begin();
     set_callbacks( Create, Destroy );
 vlc_module_end();
 
-/**
- * Private data in a subpicture. Describes a string.
- */
-typedef struct subpicture_data_t
-{
-    int            i_width;
-    int            i_height;
-    /** The string associated with this subpicture */
-    byte_t        *psz_text;
-    line_desc_t   *p_lines;
-
-} subpicture_data_t;
-
 struct line_desc_t
 {
     /** NULL-terminated list of glyphs making the string */
     FT_BitmapGlyph *pp_glyphs;
     /** list of relative positions for the glyphs */
     FT_Vector      *p_glyph_pos;
+
     int             i_height;
     int             i_width;
+    int             i_red, i_green, i_blue;
+    int             i_alpha;
+
     line_desc_t    *p_next;
 };
 
-static void Render    ( filter_t *, subpicture_t *, subpicture_data_t * );
-static void FreeString( subpicture_data_t * );
+static int Render( filter_t *, subpicture_region_t *, line_desc_t *, int, int);
+static void FreeLines( line_desc_t * );
+static void FreeLine( line_desc_t * );
 
 /*****************************************************************************
  * filter_sys_t: freetype local data
@@ -143,7 +169,12 @@ struct filter_sys_t
     FT_Library     p_library;   /* handle to library     */
     FT_Face        p_face;      /* handle to face object */
     vlc_bool_t     i_use_kerning;
-    uint8_t        pi_gamma[256];
+    uint8_t        i_font_opacity;
+    int            i_font_color;
+    int            i_font_size;
+
+    int            i_default_font_size;
+    int            i_display_height;
 };
 
 /*****************************************************************************
@@ -156,12 +187,11 @@ static int Create( vlc_object_t *p_this )
     filter_t *p_filter = (filter_t *)p_this;
     filter_sys_t *p_sys;
     char *psz_fontfile = NULL;
-    int i, i_error;
-    int i_fontsize = 0;
+    int i_error;
     vlc_value_t val;
 
     /* Allocate structure */
-    p_sys = malloc( sizeof( filter_sys_t ) );
+    p_filter->p_sys = p_sys = malloc( sizeof( filter_sys_t ) );
     if( !p_sys )
     {
         msg_Err( p_filter, "out of memory" );
@@ -169,17 +199,23 @@ static int Create( vlc_object_t *p_this )
     }
     p_sys->p_face = 0;
     p_sys->p_library = 0;
+    p_sys->i_font_size = 0;
+    p_sys->i_display_height = 0;
 
-    for( i = 0; i < 256; i++ )
-    {
-        p_sys->pi_gamma[i] = (uint8_t)( pow( (double)i * 255.0f, 0.5f ) );
-    }
-
-    var_Create( p_filter, "freetype-font", VLC_VAR_STRING | VLC_VAR_DOINHERIT );
+    var_Create( p_filter, "freetype-font",
+                VLC_VAR_STRING | VLC_VAR_DOINHERIT );
     var_Create( p_filter, "freetype-fontsize",
                 VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
     var_Create( p_filter, "freetype-rel-fontsize",
                 VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
+    var_Create( p_filter, "freetype-opacity",
+                VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
+    var_Get( p_filter, "freetype-opacity", &val );
+    p_sys->i_font_opacity = __MAX( __MIN( val.i_int, 255 ), 0 );
+    var_Create( p_filter, "freetype-color",
+                VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
+    var_Get( p_filter, "freetype-color", &val );
+    p_sys->i_font_color = __MAX( __MIN( val.i_int, 0xFFFFFF ), 0 );
 
     /* Look what method was requested */
     var_Get( p_filter, "freetype-font", &val );
@@ -229,32 +265,11 @@ static int Create( vlc_object_t *p_this )
     p_sys->i_use_kerning = FT_HAS_KERNING( p_sys->p_face );
 
     var_Get( p_filter, "freetype-fontsize", &val );
-    if( val.i_int )
-    {
-        i_fontsize = val.i_int;
-    }
-    else
-    {
-        var_Get( p_filter, "freetype-rel-fontsize", &val );
-        i_fontsize = (int)p_filter->fmt_out.video.i_height / val.i_int;
-    }
-    if( i_fontsize <= 0 )
-    {
-        msg_Warn( p_filter, "Invalid fontsize, using 12" );
-        i_fontsize = 12;
-    }
-    msg_Dbg( p_filter, "Using fontsize: %i", i_fontsize);
-
-    i_error = FT_Set_Pixel_Sizes( p_sys->p_face, 0, i_fontsize );
-    if( i_error )
-    {
-        msg_Err( p_filter, "couldn't set font size to %d", i_fontsize );
-        goto error;
-    }
+    p_sys->i_default_font_size = val.i_int;
+    if( SetFontSize( p_filter, 0 ) != VLC_SUCCESS ) goto error;
 
     if( psz_fontfile ) free( psz_fontfile );
-    p_filter->pf_render_string = RenderText;
-    p_filter->p_sys = p_sys;
+    p_filter->pf_render_text = RenderText;
     return VLC_SUCCESS;
 
  error:
@@ -274,6 +289,7 @@ static void Destroy( vlc_object_t *p_this )
 {
     filter_t *p_filter = (filter_t *)p_this;
     filter_sys_t *p_sys = p_filter->p_sys;
+
     FT_Done_Face( p_sys->p_face );
     FT_Done_FreeType( p_sys->p_library );
     free( p_sys );
@@ -284,122 +300,161 @@ static void Destroy( vlc_object_t *p_this )
  *****************************************************************************
  * This function merges the previously rendered freetype glyphs into a picture
  *****************************************************************************/
-static void Render( filter_t *p_filter, subpicture_t *p_spu,
-                    subpicture_data_t *p_string )
+static int Render( filter_t *p_filter, subpicture_region_t *p_region,
+                   line_desc_t *p_line, int i_width, int i_height )
 {
-    filter_sys_t *p_sys = p_filter->p_sys;
-    line_desc_t *p_line;
-    uint8_t *p_y, *p_u, *p_v, *p_a;
+    static uint8_t pi_gamma[16] =
+        {0x00, 0x52, 0x84, 0x96, 0xb8, 0xca, 0xdc, 0xee, 0xff,
+          0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff};
+
+    uint8_t *p_dst;
     video_format_t fmt;
     int i, x, y, i_pitch;
+    uint8_t i_y; /* YUV values, derived from incoming RGB */
+    int8_t i_u, i_v;
+    subpicture_region_t *p_region_tmp;
 
     /* Create a new subpicture region */
     memset( &fmt, 0, sizeof(video_format_t) );
-    fmt.i_chroma = VLC_FOURCC('Y','U','V','A');
-    fmt.i_aspect = VOUT_ASPECT_FACTOR;
-    fmt.i_width = fmt.i_visible_width = p_string->i_width + 2;
-    fmt.i_height = fmt.i_visible_height = p_string->i_height + 2;
+    fmt.i_chroma = VLC_FOURCC('Y','U','V','P');
+    fmt.i_aspect = 0;
+    fmt.i_width = fmt.i_visible_width = i_width + 4;
+    fmt.i_height = fmt.i_visible_height = i_height + 4;
     fmt.i_x_offset = fmt.i_y_offset = 0;
-    p_spu->p_region = p_spu->pf_create_region( VLC_OBJECT(p_filter), &fmt );
-    if( !p_spu->p_region )
+    p_region_tmp = spu_CreateRegion( p_filter, &fmt );
+    if( !p_region_tmp )
     {
         msg_Err( p_filter, "cannot allocate SPU region" );
-        return;
+        return VLC_EGENERIC;
     }
 
-    p_spu->p_region->i_x = p_spu->p_region->i_y = 0;
-    p_y = p_spu->p_region->picture.Y_PIXELS;
-    p_u = p_spu->p_region->picture.U_PIXELS;
-    p_v = p_spu->p_region->picture.V_PIXELS;
-    p_a = p_spu->p_region->picture.A_PIXELS;
-    i_pitch = p_spu->p_region->picture.Y_PITCH;
+    p_region->fmt = p_region_tmp->fmt;
+    p_region->picture = p_region_tmp->picture;
+    free( p_region_tmp );
+
+    /* Calculate text color components */
+    i_y = (uint8_t)(( 66 * p_line->i_red  + 129 * p_line->i_green +
+                      25 * p_line->i_blue + 128) >> 8) +  16;
+    i_u = (int8_t)(( -38 * p_line->i_red  -  74 * p_line->i_green +
+                     112 * p_line->i_blue + 128) >> 8) + 128;
+    i_v = (int8_t)(( 112 * p_line->i_red  -  94 * p_line->i_green -
+                      18 * p_line->i_blue + 128) >> 8) + 128;
+
+    /* Build palette */
+    fmt.p_palette->i_entries = 16;
+    for( i = 0; i < 8; i++ )
+    {
+        fmt.p_palette->palette[i][0] = 0;
+        fmt.p_palette->palette[i][1] = 0x80;
+        fmt.p_palette->palette[i][2] = 0x80;
+        fmt.p_palette->palette[i][3] = pi_gamma[i];
+        fmt.p_palette->palette[i][3] =
+            (int)fmt.p_palette->palette[i][3] * (255 - p_line->i_alpha) / 255;
+    }
+    for( i = 8; i < fmt.p_palette->i_entries; i++ )
+    {
+        fmt.p_palette->palette[i][0] = i * 16 * i_y / 256;
+        fmt.p_palette->palette[i][1] = i_u;
+        fmt.p_palette->palette[i][2] = i_v;
+        fmt.p_palette->palette[i][3] = pi_gamma[i];
+        fmt.p_palette->palette[i][3] =
+            (int)fmt.p_palette->palette[i][3] * (255 - p_line->i_alpha) / 255;
+    }
 
-    /* Initialize the region pixels (only the alpha will be changed later) */
-    memset( p_y, 0x00, i_pitch * p_spu->p_region->fmt.i_height );
-    memset( p_u, 0x80, i_pitch * p_spu->p_region->fmt.i_height );
-    memset( p_v, 0x80, i_pitch * p_spu->p_region->fmt.i_height );
-    memset( p_a, 0x00, i_pitch * p_spu->p_region->fmt.i_height );
+    p_dst = p_region->picture.Y_PIXELS;
+    i_pitch = p_region->picture.Y_PITCH;
 
-#define pi_gamma p_sys->pi_gamma
+    /* Initialize the region pixels */
+    memset( p_dst, 0, i_pitch * p_region->fmt.i_height );
 
-    for( p_line = p_string->p_lines; p_line != NULL; p_line = p_line->p_next )
+    for( ; p_line != NULL; p_line = p_line->p_next )
     {
         int i_glyph_tmax = 0;
-        int i_bitmap_offset, i_offset;
+        int i_bitmap_offset, i_offset, i_align_offset = 0;
         for( i = 0; p_line->pp_glyphs[i] != NULL; i++ )
         {
             FT_BitmapGlyph p_glyph = p_line->pp_glyphs[ i ];
             i_glyph_tmax = __MAX( i_glyph_tmax, p_glyph->top );
         }
 
+        if( p_line->i_width < i_width )
+        {
+            if( p_region->i_text_align == SUBPICTURE_ALIGN_RIGHT )
+            {
+                i_align_offset = i_width - p_line->i_width;
+            }
+            else if( p_region->i_text_align != SUBPICTURE_ALIGN_LEFT )
+            {
+                i_align_offset = ( i_width - p_line->i_width ) / 2;
+            }
+        }
+
         for( i = 0; p_line->pp_glyphs[i] != NULL; i++ )
         {
             FT_BitmapGlyph p_glyph = p_line->pp_glyphs[ i ];
 
             i_offset = ( p_line->p_glyph_pos[ i ].y +
-                i_glyph_tmax - p_glyph->top + 1 ) *
-                i_pitch + p_line->p_glyph_pos[ i ].x + p_glyph->left + 1;
+                i_glyph_tmax - p_glyph->top + 2 ) *
+                i_pitch + p_line->p_glyph_pos[ i ].x + p_glyph->left + 2 +
+                i_align_offset;
 
             for( y = 0, i_bitmap_offset = 0; y < p_glyph->bitmap.rows; y++ )
             {
                 for( x = 0; x < p_glyph->bitmap.width; x++, i_bitmap_offset++ )
                 {
-                    if( !pi_gamma[p_glyph->bitmap.buffer[i_bitmap_offset]] )
-                        continue;
-
-                    i_offset -= i_pitch;
-                    p_a[i_offset + x] = ((uint16_t)p_a[i_offset + x] +
-                      pi_gamma[p_glyph->bitmap.buffer[i_bitmap_offset]])/2;
-                    i_offset += i_pitch; x--;
-                    p_a[i_offset + x] = ((uint16_t)p_a[i_offset + x] +
-                      pi_gamma[p_glyph->bitmap.buffer[i_bitmap_offset]])/2;
-                    x += 2;
-                    p_a[i_offset + x] = ((uint16_t)p_a[i_offset + x] +
-                      pi_gamma[p_glyph->bitmap.buffer[i_bitmap_offset]])/2;
-                    i_offset += i_pitch; x--;
-                    p_a[i_offset + x] = ((uint16_t)p_a[i_offset + x] +
-                      pi_gamma[p_glyph->bitmap.buffer[i_bitmap_offset]])/2;
-                    i_offset -= i_pitch;
+                    if( p_glyph->bitmap.buffer[i_bitmap_offset] )
+                        p_dst[i_offset+x] =
+                         ((int)p_glyph->bitmap.buffer[i_bitmap_offset] + 8)/16;
                 }
                 i_offset += i_pitch;
             }
+        }
+    }
 
-            i_offset = ( p_line->p_glyph_pos[ i ].y +
-                i_glyph_tmax - p_glyph->top ) *
-                i_pitch + p_line->p_glyph_pos[ i ].x + p_glyph->left + 1;
+    /* Outlining (find something better than nearest neighbour filtering ?) */
+    if( 1 )
+    {
+        uint8_t *p_dst = p_region->picture.Y_PIXELS;
+        uint8_t *p_top = p_dst; /* Use 1st line as a cache */
+        uint8_t left, current;
 
-            for( y = 0, i_bitmap_offset = 0; y < p_glyph->bitmap.rows; y++ )
+        for( y = 1; y < (int)fmt.i_height - 1; y++ )
+        {
+            if( y > 1 ) memcpy( p_top, p_dst, fmt.i_width );
+            p_dst += p_region->picture.Y_PITCH;
+            left = 0;
+
+            for( x = 1; x < (int)fmt.i_width - 1; x++ )
             {
-               for( x = 0; x < p_glyph->bitmap.width; x++, i_bitmap_offset++ )
-               {
-                   p_y[i_offset + x] =
-                       pi_gamma[p_glyph->bitmap.buffer[i_bitmap_offset]];
-               }
-               i_offset += i_pitch;
+                current = p_dst[x];
+                p_dst[x] = ( 8 * (int)p_dst[x] + left + p_dst[x+1] + p_top[x -1]+ p_top[x] + p_top[x+1] +
+                             p_dst[x -1 + p_region->picture.Y_PITCH ] + p_dst[x + p_region->picture.Y_PITCH] + p_dst[x + 1 + p_region->picture.Y_PITCH]) / 16;
+                left = current;
             }
-
-#undef pi_gamma
         }
+        memset( p_top, 0, fmt.i_width );
     }
+
+    return VLC_SUCCESS;
 }
 
 /**
- * This function receives a string and creates a subpicture for it. It
- * also calculates the size needed for this string, and renders the
+ * This function renders a text subpicture region into another one.
+ * It also calculates the size needed for this string, and renders the
  * needed glyphs into memory. It is used as pf_add_string callback in
  * the vout method by this module
  */
-static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
+static int RenderText( filter_t *p_filter, subpicture_region_t *p_region_out,
+                       subpicture_region_t *p_region_in )
 {
     filter_sys_t *p_sys = p_filter->p_sys;
-    subpicture_t *p_subpic = 0;
-    subpicture_data_t *p_string = 0;
-    line_desc_t  *p_line = 0, *p_next = 0;
+    line_desc_t  *p_lines = 0, *p_line = 0, *p_next = 0, *p_prev = 0;
     int i, i_pen_y, i_pen_x, i_error, i_glyph_index, i_previous;
-    uint32_t *psz_unicode, *psz_unicode_orig = 0, i_char;
+    uint32_t *psz_unicode, *psz_unicode_orig = 0, i_char, *psz_line_start;
     int i_string_length;
     char *psz_string;
     vlc_iconv_t iconv_handle = (vlc_iconv_t)(-1);
+    int i_font_color, i_font_alpha, i_font_size, i_red, i_green, i_blue;
 
     FT_BBox line;
     FT_BBox glyph_size;
@@ -407,41 +462,31 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
     FT_Glyph tmp_glyph;
 
     /* Sanity check */
-    if( !p_block ) return NULL;
-    psz_string = p_block->p_buffer;
-    if( !psz_string || !*psz_string ) goto error;
-
-    result.x = 0;
-    result.y = 0;
-    line.xMin = 0;
-    line.xMax = 0;
-    line.yMin = 0;
-    line.yMax = 0;
-
-    /* Create and initialize a subpicture */
-    p_subpic = p_filter->pf_sub_buffer_new( p_filter );
-    if( !p_subpic ) goto error;
-
-    p_subpic->i_start = p_block->i_pts;
-    p_subpic->i_stop = p_block->i_pts + p_block->i_length;
-    p_subpic->b_ephemer = (p_block->i_length == 0);
-    p_subpic->b_absolute = VLC_FALSE;
-
-    /* Create and initialize private data for the subpicture */
-    p_string = malloc( sizeof(subpicture_data_t) );
-    if( !p_string )
-    {
-        msg_Err( p_filter, "Out of memory" );
-        goto error;
-    }
-    p_string->p_lines = 0;
-    p_string->psz_text = strdup( psz_string );
+    if( !p_region_in || !p_region_out ) return VLC_EGENERIC;
+    psz_string = p_region_in->psz_text;
+    if( !psz_string || !*psz_string ) return VLC_EGENERIC;
+
+    i_font_color = __MAX( __MIN( p_region_in->i_text_color, 0xFFFFFF ), 0 );
+    if( i_font_color == 0xFFFFFF ) i_font_color = p_sys->i_font_color;
+
+    i_font_alpha = __MAX( __MIN( p_region_in->i_text_alpha, 255 ), 0 );
+    if( !i_font_alpha ) i_font_alpha = 255 - p_sys->i_font_opacity;
+
+    i_font_size  = __MAX( __MIN( p_region_in->i_text_size, 255 ), 0 );
+    SetFontSize( p_filter, i_font_size );
+
+    i_red   = ( i_font_color & 0x00FF0000 ) >> 16;
+    i_green = ( i_font_color & 0x0000FF00 ) >>  8;
+    i_blue  =   i_font_color & 0x000000FF;
+
+    result.x =  result.y = 0;
+    line.xMin = line.xMax = line.yMin = line.yMax = 0;
 
     psz_unicode = psz_unicode_orig =
         malloc( ( strlen(psz_string) + 1 ) * sizeof(uint32_t) );
     if( psz_unicode == NULL )
     {
-        msg_Err( p_filter, "Out of memory" );
+        msg_Err( p_filter, "out of memory" );
         goto error;
     }
 #if defined(WORDS_BIGENDIAN)
@@ -451,7 +496,7 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
 #endif
     if( iconv_handle == (vlc_iconv_t)-1 )
     {
-        msg_Warn( p_filter, "Unable to do convertion" );
+        msg_Warn( p_filter, "unable to do conversion" );
         goto error;
     }
 
@@ -470,7 +515,7 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
 
         if( i_in_bytes )
         {
-            msg_Warn( p_filter, "Failed to convert string to unicode (%s), "
+            msg_Warn( p_filter, "failed to convert string to unicode (%s), "
                       "bytes left %d", strerror(errno), i_in_bytes );
             goto error;
         }
@@ -493,17 +538,15 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
 
     /* Calculate relative glyph positions and a bounding box for the
      * entire string */
-    p_line = NewLine( psz_string );
-    if( p_line == NULL )
+    if( !(p_line = NewLine( psz_string )) )
     {
-        msg_Err( p_filter, "Out of memory" );
+        msg_Err( p_filter, "out of memory" );
         goto error;
     }
-    p_string->p_lines = p_line;
-    i_pen_x = 0;
-    i_pen_y = 0;
-    i_previous = 0;
-    i = 0;
+    p_lines = p_line;
+    i_pen_x = i_pen_y = 0;
+    i_previous = i = 0;
+    psz_line_start = psz_unicode;
 
 #define face p_sys->p_face
 #define glyph face->glyph
@@ -518,29 +561,31 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
 
         if( i_char == '\n' )
         {
-            p_next = NewLine( psz_string );
-            if( p_next == NULL )
+            psz_line_start = psz_unicode;
+            if( !(p_next = NewLine( psz_string )) )
             {
-                msg_Err( p_filter, "Out of memory" );
+                msg_Err( p_filter, "out of memory" );
                 goto error;
             }
             p_line->p_next = p_next;
             p_line->i_width = line.xMax;
             p_line->i_height = face->size->metrics.height >> 6;
             p_line->pp_glyphs[ i ] = NULL;
+            p_line->i_alpha = i_font_alpha;
+            p_line->i_red = i_red;
+            p_line->i_green = i_green;
+            p_line->i_blue = i_blue;
+            p_prev = p_line;
             p_line = p_next;
             result.x = __MAX( result.x, line.xMax );
             result.y += face->size->metrics.height >> 6;
             i_pen_x = 0;
-            line.xMin = 0;
-            line.xMax = 0;
-            line.yMin = 0;
-            line.yMax = 0;
+            i_previous = i = 0;
+            line.xMin = line.xMax = line.yMin = line.yMax = 0;
             i_pen_y += face->size->metrics.height >> 6;
 #if 0
             msg_Dbg( p_filter, "Creating new line, i is %d", i );
 #endif
-            i = 0;
             continue;
         }
 
@@ -569,13 +614,44 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
             goto error;
         }
         FT_Glyph_Get_CBox( tmp_glyph, ft_glyph_bbox_pixels, &glyph_size );
-        i_error = FT_Glyph_To_Bitmap( &tmp_glyph, ft_render_mode_normal,
-                                      NULL, 1 );
-        if( i_error ) continue;
+        i_error = FT_Glyph_To_Bitmap( &tmp_glyph, ft_render_mode_normal, 0, 1);
+        if( i_error )
+        {
+            FT_Done_Glyph( tmp_glyph );
+            continue;
+        }
         p_line->pp_glyphs[ i ] = (FT_BitmapGlyph)tmp_glyph;
 
         /* Do rest */
-        line.xMax = p_line->p_glyph_pos[i].x + glyph_size.xMax - glyph_size.xMin;
+        line.xMax = p_line->p_glyph_pos[i].x + glyph_size.xMax -
+            glyph_size.xMin + ((FT_BitmapGlyph)tmp_glyph)->left;
+        if( line.xMax > (int)p_filter->fmt_out.video.i_visible_width - 20 )
+        {
+            p_line->pp_glyphs[ i ] = NULL;
+            FreeLine( p_line );
+            p_line = NewLine( psz_string );
+            if( p_prev ) p_prev->p_next = p_line;
+            else p_lines = p_line;
+
+            while( psz_unicode > psz_line_start && *psz_unicode != ' ' )
+            {
+                psz_unicode--;
+            }
+            if( psz_unicode == psz_line_start )
+            {
+                msg_Warn( p_filter, "unbreakable string" );
+                goto error;
+            }
+            else
+            {
+                *psz_unicode = '\n';
+            }
+            psz_unicode = psz_line_start;
+            i_pen_x = 0;
+            i_previous = i = 0;
+            line.xMin = line.xMax = line.yMin = line.yMax = 0;
+            continue;
+        }
         line.yMax = __MAX( line.yMax, glyph_size.yMax );
         line.yMin = __MIN( line.yMin, glyph_size.yMin );
 
@@ -587,59 +663,62 @@ static subpicture_t *RenderText( filter_t *p_filter, block_t *p_block )
     p_line->i_width = line.xMax;
     p_line->i_height = face->size->metrics.height >> 6;
     p_line->pp_glyphs[ i ] = NULL;
+    p_line->i_alpha = i_font_alpha;
+    p_line->i_red = i_red;
+    p_line->i_green = i_green;
+    p_line->i_blue = i_blue;
     result.x = __MAX( result.x, line.xMax );
     result.y += line.yMax - line.yMin;
-    p_string->i_height = result.y;
-    p_string->i_width = result.x;
 
 #undef face
 #undef glyph
 
-    Render( p_filter, p_subpic, p_string );
-    FreeString( p_string );
-    block_Release( p_block );
+    p_region_out->i_x = p_region_in->i_x;
+    p_region_out->i_y = p_region_in->i_y;
+
+    Render( p_filter, p_region_out, p_lines, result.x, result.y );
+
     if( psz_unicode_orig ) free( psz_unicode_orig );
-    return p_subpic;
+    FreeLines( p_lines );
+    return VLC_SUCCESS;
 
  error:
-    FreeString( p_string );
-    if( p_subpic ) p_filter->pf_sub_buffer_del( p_filter, p_subpic );
-    block_Release( p_block );
     if( psz_unicode_orig ) free( psz_unicode_orig );
-    return NULL;
+    FreeLines( p_lines );
+    return VLC_EGENERIC;
 }
 
-static void FreeString( subpicture_data_t *p_string )
+static void FreeLine( line_desc_t *p_line )
 {
     unsigned int i;
+    for( i = 0; p_line->pp_glyphs[ i ] != NULL; i++ )
+    {
+        FT_Done_Glyph( (FT_Glyph)p_line->pp_glyphs[ i ] );
+    }
+    free( p_line->pp_glyphs );
+    free( p_line->p_glyph_pos );
+    free( p_line );
+}
+
+static void FreeLines( line_desc_t *p_lines )
+{
     line_desc_t *p_line, *p_next;
 
-    if( !p_string ) return;
+    if( !p_lines ) return;
 
-    for( p_line = p_string->p_lines; p_line != NULL; p_line = p_next )
+    for( p_line = p_lines; p_line != NULL; p_line = p_next )
     {
         p_next = p_line->p_next;
-        for( i = 0; p_line->pp_glyphs[ i ] != NULL; i++ )
-        {
-            FT_Done_Glyph( (FT_Glyph)p_line->pp_glyphs[ i ] );
-        }
-        free( p_line->pp_glyphs );
-        free( p_line->p_glyph_pos );
-        free( p_line );
+        FreeLine( p_line );
     }
-
-    free( p_string->psz_text );
-    free( p_string );
 }
 
 static line_desc_t *NewLine( byte_t *psz_string )
 {
     int i_count;
     line_desc_t *p_line = malloc( sizeof(line_desc_t) );
-    if( !p_line )
-    {
-        return NULL;
-    }
+
+    if( !p_line ) return NULL;
     p_line->i_height = 0;
     p_line->i_width = 0;
     p_line->p_next = NULL;
@@ -648,15 +727,15 @@ static line_desc_t *NewLine( byte_t *psz_string )
      * sure the string is utf8. Better be safe than sorry. */
     i_count = strlen( psz_string );
 
-    p_line->pp_glyphs = malloc( sizeof(FT_BitmapGlyph)
-                                * ( i_count + 1 ) );
+    p_line->pp_glyphs = malloc( sizeof(FT_BitmapGlyph) * ( i_count + 1 ) );
     if( p_line->pp_glyphs == NULL )
     {
         free( p_line );
         return NULL;
     }
-    p_line->p_glyph_pos = malloc( sizeof( FT_Vector )
-                                  * i_count + 1 );
+    p_line->pp_glyphs[0] = NULL;
+
+    p_line->p_glyph_pos = malloc( sizeof( FT_Vector ) * i_count + 1 );
     if( p_line->p_glyph_pos == NULL )
     {
         free( p_line->pp_glyphs );
@@ -666,3 +745,48 @@ static line_desc_t *NewLine( byte_t *psz_string )
 
     return p_line;
 }
+
+static int SetFontSize( filter_t *p_filter, int i_size )
+{
+    filter_sys_t *p_sys = p_filter->p_sys;
+
+    if( i_size && i_size == p_sys->i_font_size ) return VLC_SUCCESS;
+
+    if( !i_size )
+    {
+        vlc_value_t val;
+
+        if( !p_sys->i_default_font_size &&
+            p_sys->i_display_height == (int)p_filter->fmt_out.video.i_height )
+            return VLC_SUCCESS;
+
+        if( p_sys->i_default_font_size )
+        {
+            i_size = p_sys->i_default_font_size;
+        }
+        else
+        {
+            var_Get( p_filter, "freetype-rel-fontsize", &val );
+            i_size = (int)p_filter->fmt_out.video.i_height / val.i_int;
+            p_filter->p_sys->i_display_height =
+                p_filter->fmt_out.video.i_height;
+        }
+        if( i_size <= 0 )
+        {
+            msg_Warn( p_filter, "Invalid fontsize, using 12" );
+            i_size = 12;
+        }
+
+        msg_Dbg( p_filter, "Using fontsize: %i", i_size );
+    }
+
+    p_sys->i_font_size = i_size;
+
+    if( FT_Set_Pixel_Sizes( p_sys->p_face, 0, i_size ) )
+    {
+        msg_Err( p_filter, "couldn't set font size to %d", i_size );
+        return VLC_EGENERIC;
+    }
+
+    return VLC_SUCCESS;
+}