]> git.sesse.net Git - vlc/commitdiff
Merge branch '0.9.0-libass' of git://git.videolan.org/vlc
authorDerk-Jan Hartman <hartman@videolan.org>
Sat, 9 Aug 2008 00:07:55 +0000 (02:07 +0200)
committerDerk-Jan Hartman <hartman@videolan.org>
Sat, 9 Aug 2008 00:07:55 +0000 (02:07 +0200)
configure.ac
extras/contrib/src/Makefile
extras/contrib/src/Patches/libass_mplayer_updates.patch [new file with mode: 0644]
modules/codec/Modules.am
modules/codec/libass.c [new file with mode: 0644]

index b5eebe464d869c7dadd1182c60ffe877638a402e..b9bc30e749c95426710ff811cddd1997c1db58c8 100644 (file)
@@ -3766,6 +3766,27 @@ AS_IF( [test "${enable_csri}" = "yes"], [
       ])
   ])
 
+dnl
+dnl libass subtitle rendering module
+dnl
+AC_ARG_ENABLE(libass,
+  [  --enable-libass           Subtitle support using libass (default disabled)])
+AS_IF( [test "${enable_libass}" = "yes"], [
+  PKG_CHECK_MODULES(LIBASS,
+      libass >= 0.9.5,
+      [
+        VLC_ADD_LDFLAGS([libass],[$LIBASS_LIBS])
+        VLC_ADD_CFLAGS([libass],[$LIBASS_CFLAGS])
+        VLC_ADD_PLUGIN([libass])
+
+               AC_CHECK_HEADERS(fontconfig/fontconfig.h,
+                 [VLC_ADD_CPPFLAGS([libass],[-DHAVE_FONTCONFIG])
+                  VLC_ADD_LIBS([libass],[-lfontconfig])])
+      ],[
+        AC_MSG_WARN([LIBASS library not found])
+      ])
+  ])
+
 dnl
 dnl asa demuxer
 dnl
index 39fba3d6522fd8e706fc17ff9d533b2170d29fe2..aa76dd17814301d5aa75c42c22f39effa49fe304 100644 (file)
@@ -471,13 +471,13 @@ fontconfig: fontconfig-$(FONTCONFIG_VERSION).tar.gz Patches/fontconfig.patch
 .fontconfig: fontconfig .xml .freetype
 ifdef HAVE_WIN32
   ifdef HAVE_CYGWIN
-       (cd $<; autoreconf && LIBXML2_CFLAGS=`$(PREFIX)/bin/xml2-config --cflags` ./configure --target=$(HOST) --disable-pic --disable-shared --disable-docs --with-cache-dir=WINDOWSTEMPDIR --with-arch=i686 --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 && make && make install)
+       (cd $<; autoreconf && LIBXML2_CFLAGS=`$(PREFIX)/bin/xml2-config --cflags` ./configure --target=$(HOST) --disable-pic --disable-shared --disable-docs --with-cache-dir=WINDOWSTEMPDIR --with-arch=i686 --with-default-fonts=C:\\windows\fonts --with-add-fonts=C:\\winnt\fonts --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 && make && make install)
   else
-       (cd $<; autoreconf && $(HOSTCC) LIBXML2_CFLAGS=`$(PREFIX)/bin/xml2-config --cflags` ./configure $(HOSTCONF) --with-cache-dir=WINDOWSTEMPDIR --with-arch=i686 --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 --disable-docs && make && make install)
+       (cd $<; autoreconf && $(HOSTCC) LIBXML2_CFLAGS=`$(PREFIX)/bin/xml2-config --cflags` ./configure $(HOSTCONF) --with-cache-dir=WINDOWSTEMPDIR --with-arch=i686 --with-default-fonts=C:\\windows\fonts --with-add-fonts=C:\\winnt\fonts --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 --disable-docs && make && make install)
   endif
 else
   ifdef HAVE_DARWIN_OS
-       (cd $<; autoreconf && $(HOSTCC) LIBXML2_CFLAGS=`xml2-config --cflags` LIBXML2_LIBS=`xml2-config --libs` ./configure $(HOSTCONF) --with-cache-dir=/var/cache/fontconfig --with-confdir=/etc/fonts --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 --disable-docs && make && make install-exec && (cd fontconfig ; make install-data) && cp fontconfig.pc $(PKG_CONFIG_LIBDIR))
+       (cd $<; autoreconf && $(HOSTCC) LIBXML2_CFLAGS=`xml2-config --cflags` LIBXML2_LIBS=`xml2-config --libs` ./configure $(HOSTCONF) --with-cache-dir=/var/cache/fontconfig --with-default-fonts=/System/Library/Fonts --with-add-fonts=/Library/Fonts,~/Library/Fonts --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 --disable-docs && make && make install-exec && (cd fontconfig ; make install-data) && cp fontconfig.pc $(PKG_CONFIG_LIBDIR))
   else
        (cd $<; autoreconf && $(HOSTCC) LIBXML2_CFLAGS=`$(PREFIX)/bin/xml2-config --cflags` ./configure $(HOSTCONF) --prefix=$(PREFIX) --with-freetype-config=$(PREFIX)/bin/freetype-config --enable-libxml2 --disable-docs && make && make install)
   endif
@@ -2442,9 +2442,10 @@ DISTCLEAN_PKG += schroedinger-$(SCHROED_VERSION).tar.gz
 libass-$(ASS_VERSION).tar.bz2:
        $(WGET) $(ASS_URL)
 
+# mplayer patches up to August 8, 2008 are included
 libass: libass-$(ASS_VERSION).tar.bz2
        $(EXTRACT_BZ2)
-       cd $@; patch -p0 < ../Patches/libass-h-char-fix.patch
+       patch -p0 < Patches/libass_mplayer_updates.patch
 ifneq ($(HOST),$(BUILD))
        patch -p0 < Patches/libass-cross.patch
        cd $@; autoconf
diff --git a/extras/contrib/src/Patches/libass_mplayer_updates.patch b/extras/contrib/src/Patches/libass_mplayer_updates.patch
new file mode 100644 (file)
index 0000000..8954a69
--- /dev/null
@@ -0,0 +1,273 @@
+diff -ruN libass-0.9.5/libass/ass.c libass-0.9.5-patched/libass/ass.c
+--- libass/libass/ass.c        2008-05-22 20:01:18.000000000 +0200
++++ libass-0.9.5-patched/libass/ass.c  2008-08-08 23:59:21.000000000 +0200
+@@ -846,16 +846,22 @@
+               char* ip;
+               char* op;
+               size_t rc;
++                int clear = 0;
+               
+-              outbuf = malloc(size);
++              outbuf = malloc(osize);
+               ip = data;
+               op = outbuf;
+               
+-              while (ileft) {
++              while (1) {
++                        if(ileft)
+                       rc = iconv(icdsc, &ip, &ileft, &op, &oleft);
++                        else {// clear the conversion state and leave
++                            clear = 1;
++                            rc = iconv(icdsc, NULL, NULL, &op, &oleft);
++                        }
+                       if (rc == (size_t)(-1)) {
+                               if (errno == E2BIG) {
+-                                      int offset = op - outbuf;
++                                      size_t offset = op - outbuf;
+                                       outbuf = (char*)realloc(outbuf, osize + size);
+                                       op = outbuf + offset;
+                                       osize += size;
+@@ -864,7 +870,9 @@
+                                       mp_msg(MSGT_ASS, MSGL_WARN, MSGTR_LIBASS_ErrorRecodingFile);
+                                       return NULL;
+                               }
+-                      }
++                      } else
++                            if( clear )
++                                break;
+               }
+               outbuf[osize - oleft - 1] = 0;
+       }
+diff -ruN libass-0.9.5/libass/ass_bitmap.c libass-0.9.5-patched/libass/ass_bitmap.c
+--- libass/libass/ass_bitmap.c 2008-05-22 20:01:18.000000000 +0200
++++ libass-0.9.5-patched/libass/ass_bitmap.c   2008-08-09 00:04:36.000000000 +0200
+@@ -274,9 +274,10 @@
+       resize_tmp(priv, (*bm_g)->w, (*bm_g)->h);
+       
+       if (be) {
+-              blur((*bm_g)->buffer, priv->tmp, (*bm_g)->w, (*bm_g)->h, (*bm_g)->w, (int*)priv->gt2, priv->g_r, priv->g_w);
+               if (*bm_o)
+                       blur((*bm_o)->buffer, priv->tmp, (*bm_o)->w, (*bm_o)->h, (*bm_o)->w, (int*)priv->gt2, priv->g_r, priv->g_w);
++              else
++                      blur((*bm_g)->buffer, priv->tmp, (*bm_g)->w, (*bm_g)->h, (*bm_g)->w, (int*)priv->gt2, priv->g_r, priv->g_w);
+       }
+       if (*bm_o)
+diff -ruN libass-0.9.5/libass/ass_fontconfig.c libass-0.9.5-patched/libass/ass_fontconfig.c
+--- libass/libass/ass_fontconfig.c     2008-05-22 20:01:18.000000000 +0200
++++ libass-0.9.5-patched/libass/ass_fontconfig.c       2008-08-08 23:59:21.000000000 +0200
+@@ -129,6 +129,8 @@
+               goto error;
+       fset = FcFontSort(priv->config, pat, FcTrue, NULL, &result);
++        if(!fset)
++            goto error;
+       for (curf = 0; curf < fset->nfont; ++curf) {
+               FcPattern* curp = fset->fonts[curf];
+@@ -351,12 +353,15 @@
+       FcPattern* pattern;
+       FcFontSet* fset;
+       FcBool res;
++        int face_index, num_faces = 1;
+-      rc = FT_New_Memory_Face(ftlibrary, (unsigned char*)data, data_size, 0, &face);
++      for (face_index = 0; face_index < num_faces; ++face_index) {
++        rc = FT_New_Memory_Face(ftlibrary, (unsigned char*)data, data_size, 0, &face);
+       if (rc) {
+               mp_msg(MSGT_ASS, MSGL_WARN, MSGTR_LIBASS_ErrorOpeningMemoryFont, name);
+               return;
+       }
++        num_faces = face->num_faces;
+       pattern = FcFreeTypeQueryFace(face, (unsigned char*)name, 0, FcConfigGetBlanks(priv->config));
+       if (!pattern) {
+@@ -380,6 +385,7 @@
+       }
+       FT_Done_Face(face);
++        }
+ #endif
+ }
+@@ -419,7 +425,8 @@
+       for (i = 0; i < library->num_fontdata; ++i)
+               process_fontdata(priv, library, ftlibrary, i);
+-      if (FcDirCacheValid((const FcChar8 *)dir) == FcFalse)
++      if(dir) {
++        if (FcDirCacheValid((const FcChar8 *)dir) == FcFalse)
+       {
+               mp_msg(MSGT_ASS, MSGL_INFO, MSGTR_LIBASS_UpdatingFontCache);
+               if (FcGetVersion() >= 20390 && FcGetVersion() < 20400)
+@@ -457,6 +464,7 @@
+       if (!rc) {
+               mp_msg(MSGT_ASS, MSGL_WARN, MSGTR_LIBASS_FcConfigAppFontAddDirFailed);
+       }
++        }
+       priv->family_default = family ? strdup(family) : 0;
+       priv->path_default = path ? strdup(path) : 0;
+diff -ruN libass-0.9.5/libass/ass_render.c libass-0.9.5-patched/libass/ass_render.c
+--- libass/libass/ass_render.c 2008-05-22 20:01:18.000000000 +0200
++++ libass-0.9.5-patched/libass/ass_render.c   2008-08-09 00:04:28.000000000 +0200
+@@ -149,8 +149,8 @@
+               EVENT_HSCROLL, // "Banner" transition effect, text_width is unlimited
+               EVENT_VSCROLL // "Scroll up", "Scroll down" transition effects
+               } evt_type;
+-      int pos_x, pos_y; // position
+-      int org_x, org_y; // origin
++      double pos_x, pos_y; // position
++      double org_x, org_y; // origin
+       char have_origin; // origin is explicitly defined; if 0, get_base_point() is used
+       double scale_x, scale_y;
+       double hspacing; // distance between letters, in pixels
+@@ -161,6 +161,7 @@
+       uint32_t fade; // alpha from \fad
+       char be; // blur edges
+       int shadow;
++      int drawing_mode; // not implemented; when != 0 text is discarded, except for style override tags
+       effect_t effect_type;
+       int effect_timing;
+@@ -456,19 +457,19 @@
+ /**
+  * \brief Mapping between script and screen coordinates
+  */
+-static int x2scr(int x) {
++static int x2scr(double x) {
+       return x*frame_context.orig_width_nocrop / frame_context.track->PlayResX +
+               FFMAX(global_settings->left_margin, 0);
+ }
+ /**
+  * \brief Mapping between script and screen coordinates
+  */
+-static int y2scr(int y) {
++static int y2scr(double y) {
+       return y * frame_context.orig_height_nocrop / frame_context.track->PlayResY +
+               FFMAX(global_settings->top_margin, 0);
+ }
+ // the same for toptitles
+-static int y2scr_top(int y) {
++static int y2scr_top(double y) {
+       if (global_settings->use_margins)
+               return y * frame_context.orig_height_nocrop / frame_context.track->PlayResY;
+       else
+@@ -476,7 +477,7 @@
+                       FFMAX(global_settings->top_margin, 0);
+ }
+ // the same for subtitles
+-static int y2scr_sub(int y) {
++static int y2scr_sub(double y) {
+       if (global_settings->use_margins)
+               return y * frame_context.orig_height_nocrop / frame_context.track->PlayResY +
+                       FFMAX(global_settings->top_margin, 0) +
+@@ -679,11 +680,11 @@
+  * \param pwr multiplier for some tag effects (comes from \t tags)
+  */
+ static char* parse_tag(char* p, double pwr) {
+-#define skip_all(x) if (*p == (x)) ++p; else { \
+-      while ((*p != (x)) && (*p != '}') && (*p != 0)) {++p;} }
++#define skip_to(x) while ((*p != (x)) && (*p != '}') && (*p != 0)) { ++p;}
+ #define skip(x) if (*p == (x)) ++p; else { return p; }
+       
+-      skip_all('\\');
++      skip_to('\\');
++      skip('\\');
+       if ((*p == '}') || (*p == 0))
+               return p;
+@@ -727,7 +728,7 @@
+       } else if (mystrcmp(&p, "move")) {
+               int x1, x2, y1, y2;
+               long long t1, t2, delta_t, t;
+-              int x, y;
++              double x, y;
+               double k;
+               skip('(');
+               x1 = strtol(p, &p, 10);
+@@ -787,7 +788,7 @@
+       } else if (mystrcmp(&p, "fn")) {
+               char* start = p;
+               char* family;
+-              skip_all('\\');
++              skip_to('\\');
+               if (p > start) {
+                       family = malloc(p - start + 1);
+                       strncpy(family, start, p - start);
+@@ -888,6 +889,7 @@
+               render_context.org_x = v1;
+               render_context.org_y = v2;
+               render_context.have_origin = 1;
++              render_context.detect_collisions = 0;
+       } else if (mystrcmp(&p, "t")) {
+               double v[3];
+               int v1, v2;
+@@ -928,7 +930,8 @@
+               }
+               while (*p == '\\')
+                       p = parse_tag(p, k); // maybe k*pwr ? no, specs forbid nested \t's 
+-              skip_all(')'); // FIXME: better skip(')'), but much more tags support required
++              skip_to(')'); // in case there is some unknown tag or a comment
++              skip(')');
+       } else if (mystrcmp(&p, "clip")) {
+               int x0, y0, x1, y1;
+               int res = 1;
+@@ -1026,12 +1029,19 @@
+                       render_context.shadow = val;
+               else
+                       render_context.shadow = render_context.style->Shadow;
++      } else if (mystrcmp(&p, "pbo")) {
++              (void)strtol(p, &p, 10); // ignored
++      } else if (mystrcmp(&p, "p")) {
++              int val;
++              if (!mystrtoi(&p, 10, &val))
++                      val = 0;
++              render_context.drawing_mode = !!val;
+       }
+       return p;
+ #undef skip
+-#undef skip_all
++#undef skip_to
+ }
+ /**
+@@ -1071,7 +1081,7 @@
+                       p += 2;
+                       *str = p;
+                       return '\n';
+-              } else if (*(p+1) == 'n') {
++              } else if ((*(p+1) == 'n') || (*(p+1) == 'h')) {
+                       p += 2;
+                       *str = p;
+                       return ' ';
+@@ -1201,6 +1211,7 @@
+       render_context.clip_y1 = frame_context.track->PlayResY;
+       render_context.detect_collisions = 1;
+       render_context.fade = 0;
++      render_context.drawing_mode = 0;
+       render_context.effect_type = EF_NONE;
+       render_context.effect_timing = 0;
+       render_context.effect_skip_timing = 0;
+@@ -1748,7 +1759,9 @@
+       while (1) {
+               // get next char, executing style override
+               // this affects render_context
+-              code = get_next_char(&p);
++              do {
++                      code = get_next_char(&p);
++              } while (code && render_context.drawing_mode); // skip everything in drawing mode
+               
+               // face could have been changed in get_next_char
+               if (!render_context.font) {
+@@ -1934,7 +1947,7 @@
+       if (render_context.evt_type == EVENT_POSITIONED) {
+               int base_x = 0;
+               int base_y = 0;
+-              mp_msg(MSGT_ASS, MSGL_DBG2, "positioned event at %d, %d\n", render_context.pos_x, render_context.pos_y);
++              mp_msg(MSGT_ASS, MSGL_DBG2, "positioned event at %f, %f\n", render_context.pos_x, render_context.pos_y);
+               get_base_point(bbox, alignment, &base_x, &base_y);
+               device_x = x2scr(render_context.pos_x) - base_x;
+               device_y = y2scr(render_context.pos_y) - base_y;
index 5a4c2e0fdbe6957304ee41c666e8dd2ae378cfdf..a1e4d24ba8f42f0323c280a727de771bb8accf99 100644 (file)
@@ -37,3 +37,4 @@ SOURCES_fluidsynth = fluidsynth.c
 SOURCES_cc = cc.c cc.h
 SOURCES_kate = kate.c
 SOURCES_schroedinger = schroedinger.c
+SOURCES_libass = libass.c
diff --git a/modules/codec/libass.c b/modules/codec/libass.c
new file mode 100644 (file)
index 0000000..02d58a3
--- /dev/null
@@ -0,0 +1,770 @@
+/*****************************************************************************
+ * SSA/ASS subtitle decoder using libass.
+ *****************************************************************************
+ * Copyright (C) 2008 the VideoLAN team
+ * $Id$
+ *
+ * Authors: Laurent Aimar <fenrir@videolan.org>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
+ *****************************************************************************/
+
+/*****************************************************************************
+ * Preamble
+ *****************************************************************************/
+
+#ifdef HAVE_CONFIG_H
+#   include "config.h"
+#endif
+
+#include <string.h>
+#include <limits.h>
+#include <assert.h>
+#include <math.h>
+
+#include <vlc_common.h>
+#include <vlc_plugin.h>
+#include <vlc_vout.h>
+#include <vlc_codec.h>
+#include <vlc_osd.h>
+#include <vlc_input.h>
+
+#include <ass/ass.h>
+
+/*****************************************************************************
+ * Module descriptor
+ *****************************************************************************/
+static int  Create ( vlc_object_t * );
+static void Destroy( vlc_object_t * );
+
+vlc_module_begin();
+    set_shortname( N_("Subtitles (advanced)"));
+    set_description( N_("Subtitle renderers using libass") );
+    set_capability( "decoder", 100 );
+    set_category( CAT_INPUT );
+    set_subcategory( SUBCAT_INPUT_SCODEC );
+    set_callbacks( Create, Destroy );
+vlc_module_end();
+
+/*****************************************************************************
+ * Local prototypes
+ *****************************************************************************/
+static subpicture_t *DecodeBlock( decoder_t *, block_t ** );
+static void DestroySubpicture( subpicture_t * );
+static void PreRender( video_format_t *, spu_t *, subpicture_t * );
+static void UpdateRegions( video_format_t *, spu_t *,
+                           subpicture_t *, mtime_t );
+
+/* Yes libass sux with threads */
+typedef struct 
+{
+    vlc_object_t   *p_libvlc;
+
+    vlc_mutex_t     *p_lock;
+    int             i_refcount;
+    ass_library_t   *p_library;
+    ass_renderer_t  *p_renderer;
+    video_format_t  fmt;
+} ass_handle_t;
+static ass_handle_t *AssHandleYield( decoder_t *p_dec );
+static void AssHandleRelease( ass_handle_t * );
+
+/* */
+struct decoder_sys_t
+{
+    /* decoder_sys_t is shared between decoder and spu units */
+    vlc_mutex_t  lock;
+    int          i_refcount;
+
+    /* */
+    ass_handle_t *p_ass;
+
+    /* */
+    ass_track_t  *p_track;
+
+    /* */
+    subpicture_t *p_spu_final;
+};
+static void DecSysRelease( decoder_sys_t *p_sys );
+static void DecSysYield( decoder_sys_t *p_sys );
+
+struct subpicture_sys_t
+{
+    decoder_sys_t *p_dec_sys;
+    void *p_subs_data;
+    int i_subs_len;
+};
+
+typedef struct
+{
+    int x0;
+    int y0;
+    int x1;
+    int y1;
+} rectangle_t;
+
+static int BuildRegions( spu_t *p_spu, rectangle_t *p_region, int i_max_region, ass_image_t *p_img_list, int i_width, int i_height );
+static void SubpictureReleaseRegions( spu_t *p_spu, subpicture_t *p_subpic );
+static void RegionDraw( subpicture_region_t *p_region, ass_image_t *p_img );
+
+//#define DEBUG_REGION
+
+/*****************************************************************************
+ * Create: Open libass decoder.
+ *****************************************************************************/
+static int Create( vlc_object_t *p_this )
+{
+    decoder_t *p_dec = (decoder_t *)p_this;
+    decoder_sys_t *p_sys;
+    ass_track_t *p_track;
+
+    if( p_dec->fmt_in.i_codec != VLC_FOURCC('s','s','a',' ') )
+        return VLC_EGENERIC;
+
+    p_dec->pf_decode_sub = DecodeBlock;
+
+    p_dec->p_sys = p_sys = malloc( sizeof( decoder_sys_t ) );
+    if( !p_sys )
+        return VLC_ENOMEM;
+
+    /* */
+    p_sys->p_ass = AssHandleYield( p_dec );
+    if( !p_sys->p_ass )
+    {
+        free( p_sys );
+        return VLC_EGENERIC;
+    }
+    vlc_mutex_init( &p_sys->lock );
+    p_sys->i_refcount = 1;
+
+    /* Add a track */
+    vlc_mutex_lock( p_sys->p_ass->p_lock );
+    p_sys->p_track = p_track = ass_new_track( p_sys->p_ass->p_library );
+    if( !p_track )
+    {
+        vlc_mutex_unlock( p_sys->p_ass->p_lock );
+        DecSysRelease( p_sys );
+        return VLC_EGENERIC;
+    }
+    ass_process_codec_private( p_track, p_dec->fmt_in.p_extra, p_dec->fmt_in.i_extra );
+    vlc_mutex_unlock( p_sys->p_ass->p_lock );
+
+    return VLC_SUCCESS;
+}
+
+/*****************************************************************************
+ * Destroy: finish
+ *****************************************************************************/
+static void Destroy( vlc_object_t *p_this )
+{
+    decoder_t *p_dec = (decoder_t *)p_this;
+
+    DecSysRelease( p_dec->p_sys );
+}
+
+static void DecSysYield( decoder_sys_t *p_sys )
+{
+    vlc_mutex_lock( &p_sys->lock );
+    p_sys->i_refcount++;
+    vlc_mutex_unlock( &p_sys->lock );
+}
+static void DecSysRelease( decoder_sys_t *p_sys )
+{
+    /* */
+    vlc_mutex_lock( &p_sys->lock );
+    p_sys->i_refcount--;
+    if( p_sys->i_refcount > 0 )
+    {
+        vlc_mutex_unlock( &p_sys->lock );
+        return;
+    }
+    vlc_mutex_unlock( &p_sys->lock );
+    vlc_mutex_destroy( &p_sys->lock );
+
+    vlc_mutex_lock( p_sys->p_ass->p_lock );
+    if( p_sys->p_track )
+        ass_free_track( p_sys->p_track );
+    vlc_mutex_unlock( p_sys->p_ass->p_lock );
+
+    AssHandleRelease( p_sys->p_ass );
+    free( p_sys );
+}
+
+/****************************************************************************
+ * DecodeBlock:
+ ****************************************************************************/
+static subpicture_t *DecodeBlock( decoder_t *p_dec, block_t **pp_block )
+{
+    decoder_sys_t *p_sys = p_dec->p_sys;
+
+    subpicture_t *p_spu = NULL;
+    block_t *p_block;
+
+    if( !pp_block || *pp_block == NULL )
+        return NULL;
+
+    p_block = *pp_block;
+    if( p_block->i_flags & (BLOCK_FLAG_DISCONTINUITY|BLOCK_FLAG_CORRUPTED) )
+    {
+        msg_Dbg( p_dec, "Resetting libass track after time discontinuity" );
+        /* We need to reset our tracks for the time discontinuity to be
+         * handled */
+        vlc_mutex_lock( p_sys->p_ass->p_lock );
+        if( p_sys->p_track )
+            ass_free_track( p_sys->p_track );
+
+        p_sys->p_track = ass_new_track( p_sys->p_ass->p_library );
+        if( p_sys->p_track  )
+            ass_process_codec_private( p_sys->p_track,
+                                       p_dec->fmt_in.p_extra, p_dec->fmt_in.i_extra );
+        vlc_mutex_unlock( p_sys->p_ass->p_lock );
+
+        block_Release( p_block );
+        return NULL;
+    }
+    if( p_block->i_rate != 0 )
+        p_block->i_length = p_block->i_length * p_block->i_rate / INPUT_RATE_DEFAULT;
+    *pp_block = NULL;
+
+    if( p_block->i_buffer == 0 || p_block->p_buffer[0] == '\0' )
+    {
+        block_Release( p_block );
+        return NULL;
+    }
+
+    p_spu = p_dec->pf_spu_buffer_new( p_dec );
+    if( !p_spu )
+    {
+        msg_Warn( p_dec, "can't get spu buffer" );
+        block_Release( p_block );
+        return NULL;
+    }
+
+    p_spu->p_sys = malloc( sizeof( subpicture_sys_t ));
+    if( !p_spu->p_sys )
+    {
+        p_dec->pf_spu_buffer_del( p_dec, p_spu );
+        block_Release( p_block );
+        return NULL;
+    }
+
+    p_spu->p_sys->i_subs_len = p_block->i_buffer;
+    p_spu->p_sys->p_subs_data = malloc( p_block->i_buffer );
+    if( !p_spu->p_sys->p_subs_data )
+    {
+        free( p_spu->p_sys );
+        p_dec->pf_spu_buffer_del( p_dec, p_spu );
+        block_Release( p_block );
+        return NULL;
+    }
+    memcpy( p_spu->p_sys->p_subs_data, p_block->p_buffer,
+            p_block->i_buffer );
+
+    p_spu->i_x = 0;
+    p_spu->i_y = 0;
+    p_spu->i_start = p_block->i_pts;
+    p_spu->i_stop = p_block->i_pts + p_block->i_length;
+    p_spu->b_ephemer = true;
+    p_spu->b_absolute = true;
+    p_spu->b_pausable = true; /* ? */
+
+    vlc_mutex_lock( p_sys->p_ass->p_lock );
+    if( p_sys->p_track )
+    {
+        ass_process_chunk( p_sys->p_track, p_spu->p_sys->p_subs_data, p_spu->p_sys->i_subs_len,
+                           p_spu->i_start / 1000, (p_spu->i_stop-p_spu->i_start) / 1000 );
+    }
+    vlc_mutex_unlock( p_sys->p_ass->p_lock );
+
+    p_spu->pf_pre_render = PreRender;
+    p_spu->pf_update_regions = UpdateRegions;
+    p_spu->pf_destroy = DestroySubpicture;
+    p_spu->p_sys->p_dec_sys = p_sys;
+
+    DecSysYield( p_sys );
+
+    block_Release( p_block );
+
+    return p_spu;
+}
+
+/****************************************************************************
+ *
+ ****************************************************************************/
+static void DestroySubpicture( subpicture_t *p_subpic )
+{
+    DecSysRelease( p_subpic->p_sys->p_dec_sys );
+
+    free( p_subpic->p_sys->p_subs_data );
+    free( p_subpic->p_sys );
+}
+
+static void PreRender( video_format_t *p_fmt, spu_t *p_spu,
+                       subpicture_t *p_subpic )
+{
+    decoder_sys_t *p_dec_sys = p_subpic->p_sys->p_dec_sys;
+
+    p_dec_sys->p_spu_final = p_subpic;
+    VLC_UNUSED(p_fmt);
+    VLC_UNUSED(p_spu);
+}
+
+static void UpdateRegions( video_format_t *p_fmt, spu_t *p_spu,
+                           subpicture_t *p_subpic, mtime_t i_ts )
+{
+    decoder_sys_t *p_sys = p_subpic->p_sys->p_dec_sys;
+    ass_handle_t *p_ass = p_sys->p_ass;
+
+    video_format_t fmt;
+    bool b_fmt_changed;
+
+    if( p_subpic != p_sys->p_spu_final )
+    {
+        SubpictureReleaseRegions( p_spu, p_subpic );
+        return;
+    }
+
+    vlc_mutex_lock( p_ass->p_lock );
+
+    /* */
+    fmt = *p_fmt;
+    fmt.i_chroma = VLC_FOURCC('R','G','B','A');
+    fmt.i_width = fmt.i_visible_width;
+    fmt.i_height = fmt.i_visible_height;
+    fmt.i_bits_per_pixel = 0;
+    fmt.i_x_offset = fmt.i_y_offset = 0;
+    fmt.i_sar_num = 1;
+    fmt.i_sar_den = 1;
+
+    b_fmt_changed = memcmp( &fmt, &p_ass->fmt, sizeof(fmt) ) != 0;
+    if( b_fmt_changed )
+    {
+        ass_set_frame_size( p_ass->p_renderer, fmt.i_width, fmt.i_height );
+        ass_set_aspect_ratio( p_ass->p_renderer, 1.0 ); // TODO ?
+
+        p_ass->fmt = fmt;
+    }
+
+    /* */
+    int i_changed;
+    ass_image_t *p_img = ass_render_frame( p_ass->p_renderer, p_sys->p_track, i_ts/1000, &i_changed );
+
+    if( !i_changed && !b_fmt_changed )
+    {
+        vlc_mutex_unlock( p_ass->p_lock );
+        return;
+    }
+
+    /* */
+    p_subpic->i_original_picture_height = fmt.i_height;
+    p_subpic->i_original_picture_width = fmt.i_width;
+    SubpictureReleaseRegions( p_spu, p_subpic );
+
+    /* XXX to improve efficiency we merge regions that are close minimizing
+     * the lost surface.
+     * libass tends to create a lot of small regions and thus spu engine
+     * reinstanciate a lot the scaler, and as we do not support subpel blending
+     * it looks ugly (text unaligned).
+     */
+    const int i_max_region = 4;
+    rectangle_t region[i_max_region];
+    const int i_region = BuildRegions( p_spu, region, i_max_region, p_img, fmt.i_width, fmt.i_height );
+
+    if( i_region <= 0 )
+    {
+        vlc_mutex_unlock( p_ass->p_lock );
+        return;
+    }
+
+    /* Allocate the regions and draw them */
+    subpicture_region_t *pp_region[i_max_region];
+    subpicture_region_t **pp_region_last = &p_subpic->p_region;
+
+    for( int i = 0; i < i_region; i++ )
+    {
+        subpicture_region_t *r;
+        video_format_t fmt_region;
+
+        /* */
+        fmt_region = fmt;
+        fmt_region.i_width =
+        fmt_region.i_visible_width  = region[i].x1 - region[i].x0;
+        fmt_region.i_height =
+        fmt_region.i_visible_height = region[i].y1 - region[i].y0;
+
+        pp_region[i] = r = p_subpic->pf_create_region( VLC_OBJECT(p_spu), &fmt_region );
+        if( !r )
+            break;
+        r->i_x = region[i].x0;
+        r->i_y = region[i].y0;
+        r->i_align = SUBPICTURE_ALIGN_TOP | SUBPICTURE_ALIGN_LEFT;
+
+        /* */
+        RegionDraw( r, p_img );
+
+        /* */
+        *pp_region_last = r;
+        pp_region_last = &r->p_next;
+    }
+    vlc_mutex_unlock( p_ass->p_lock );
+}
+
+static rectangle_t r_create( int x0, int y0, int x1, int y1 )
+{
+    rectangle_t r = { x0, y0, x1, y1 };
+    return r;
+}
+static rectangle_t r_img( const ass_image_t *p_img )
+{
+    return r_create( p_img->dst_x, p_img->dst_y, p_img->dst_x+p_img->w, p_img->dst_y+p_img->h );
+}
+static void r_add( rectangle_t *r, const rectangle_t *n )
+{
+    r->x0 = __MIN( r->x0, n->x0 );
+    r->y0 = __MIN( r->y0, n->y0 );
+    r->x1 = __MAX( r->x1, n->x1 );
+    r->y1 = __MAX( r->y1, n->y1 );
+}
+static int r_surface( const rectangle_t *r )
+{
+    return (r->x1-r->x0) * (r->y1-r->y0);
+}
+static bool r_overlap( const rectangle_t *a, const rectangle_t *b, int i_dx, int i_dy )
+{
+    return  __MAX(a->x0-i_dx, b->x0) < __MIN( a->x1+i_dx, b->x1 ) &&
+            __MAX(a->y0-i_dy, b->y0) < __MIN( a->y1+i_dy, b->y1 );
+}
+
+static int BuildRegions( spu_t *p_spu, rectangle_t *p_region, int i_max_region, ass_image_t *p_img_list, int i_width, int i_height )
+{
+    ass_image_t *p_tmp;
+    int i_count;
+
+    VLC_UNUSED(p_spu);
+
+#ifdef DEBUG_REGION
+    int64_t i_ck_start = mdate();
+#endif
+
+    for( p_tmp = p_img_list, i_count = 0; p_tmp != NULL; p_tmp = p_tmp->next )
+        i_count++;
+    if( i_count <= 0 )
+        return 0;
+
+    ass_image_t **pp_img = calloc( i_count, sizeof(*pp_img) );
+    if( !pp_img )
+        return 0;
+
+    for( p_tmp = p_img_list, i_count = 0; p_tmp != NULL; p_tmp = p_tmp->next, i_count++ )
+        pp_img[i_count] = p_tmp;
+
+    /* */
+    const int i_w_inc = __MAX( ( i_width + 49 ) / 50, 32 );
+    const int i_h_inc = __MAX( ( i_height + 99 ) / 100, 32 );
+    int i_maxh = i_w_inc;
+    int i_maxw = i_h_inc;
+    int i_region;
+    rectangle_t region[i_max_region+1];
+
+    i_region = 0;
+    for( int i_used = 0; i_used < i_count; )
+    {
+        int n;
+        for( n = 0; n < i_count; n++ )
+        {
+            if( pp_img[n] )
+                break;
+        }
+        assert( i_region < i_max_region + 1 );
+        region[i_region++] = r_img( pp_img[n] );
+        pp_img[n] = NULL; i_used++;
+
+        bool b_ok;
+        do {
+            b_ok = false;
+            for( n = 0; n < i_count; n++ )
+            {
+                ass_image_t *p_img = pp_img[n];
+                if( !p_img )
+                    continue;
+                rectangle_t r = r_img( p_img );
+
+                int k;
+                int i_best = -1;
+                int i_best_s = INT_MAX;
+                for( k = 0; k < i_region; k++ )
+                {
+                    if( !r_overlap( &region[k], &r, i_maxw, i_maxh ) )
+                        continue;
+                    int s = r_surface( &r );
+                    if( s < i_best_s )
+                    {
+                        i_best_s = s;
+                        i_best = k;
+                    }
+                }
+                if( i_best >= 0 )
+                {
+                    r_add( &region[i_best], &r );
+                    pp_img[n] = NULL; i_used++;
+                    b_ok = true;
+                }
+            }
+        } while( b_ok );
+
+        if( i_region > i_max_region )
+        {
+            int i_best_i = -1;
+            int i_best_j = -1;
+            int i_best_ds = INT_MAX;
+
+            /* merge best */
+            for( int i = 0; i < i_region; i++ )
+            {
+                for( int j = i+1; j < i_region; j++ )
+                {
+                    rectangle_t n = region[i];
+                    r_add( &n, &region[j] );
+                    int ds = r_surface( &n ) - r_surface( &region[i] ) - r_surface( &region[j] );
+
+                    if( ds < i_best_ds )
+                    {
+                        i_best_i = i;
+                        i_best_j = j;
+                        i_best_ds = ds;
+                    }
+                }
+            }
+#ifdef DEBUG_REGION
+            msg_Err( p_spu, "Merging %d and %d", i_best_i, i_best_j );
+#endif
+            r_add( &region[i_best_i], &region[i_best_j] );
+
+            if( i_best_j+1 < i_region )
+                memmove( &region[i_best_j], &region[i_best_j+1], sizeof(*region) * ( i_region - (i_best_j+1)  ) );
+            i_region--;
+        }
+    }
+
+    /* */
+    for( int n = 0; n < i_region; n++ )
+        p_region[n] = region[n];
+
+#ifdef DEBUG_REGION
+    int64_t i_ck_time = mdate() - i_ck_start;
+    msg_Err( p_spu, "ASS: %d objects merged into %d region in %d micros", i_count, i_region, (int)(i_ck_time) );
+#endif
+
+    free( pp_img );
+
+    return i_region;
+}
+
+static void RegionDraw( subpicture_region_t *p_region, ass_image_t *p_img )
+{
+    const plane_t *p = &p_region->picture.p[0];
+    const int i_x = p_region->i_x;
+    const int i_y = p_region->i_y;
+    const int i_width  = p_region->fmt.i_width;
+    const int i_height = p_region->fmt.i_height;
+
+    memset( p->p_pixels, 0x00, p->i_pitch * p->i_lines );
+    for( ; p_img != NULL; p_img = p_img->next )
+    {
+        if( p_img->dst_x < i_x || p_img->dst_x + p_img->w > i_x + i_width ||
+            p_img->dst_y < i_y || p_img->dst_y + p_img->h > i_y + i_height )
+            continue;
+
+        const int r = (p_img->color >> 24)&0xff;
+        const int g = (p_img->color >> 16)&0xff;
+        const int b = (p_img->color >>  8)&0xff;
+        const int a = (p_img->color      )&0xff;
+        int x, y;
+
+        for( y = 0; y < p_img->h; y++ )
+        {
+            for( x = 0; x < p_img->w; x++ )
+            {
+                const int alpha = p_img->bitmap[y*p_img->stride+x];
+                const int an = (255 - a) * alpha / 255;
+
+                uint8_t *p_rgba = &p->p_pixels[(y+p_img->dst_y-i_y) * p->i_pitch + 4 * (x+p_img->dst_x-i_x)];
+
+                /* Native endianness, but RGBA ordering */
+                p_rgba[0] = ( p_rgba[0] * (255-an) + r * an ) / 255;
+                p_rgba[1] = ( p_rgba[1] * (255-an) + g * an ) / 255;
+                p_rgba[2] = ( p_rgba[2] * (255-an) + b * an ) / 255;
+                p_rgba[3] = 255 - ( 255 - p_rgba[3] ) * ( 255 - an ) / 255;
+            }
+        }
+    }
+
+#ifdef DEBUG_REGION
+    /* XXX Draw a box for debug */
+#define P(x,y) ((uint32_t*)&p->p_pixels[(y)*p->i_pitch + 4*(x)])
+    for( int y = 0; y < p->i_lines; y++ )
+        *P(0,y) = *P(p->i_visible_pitch/4-1,y) = 0xff000000;
+    for( int x = 0; x < p->i_visible_pitch; x++ )
+        *P(x/4,0) = *P(x/4,p->i_visible_lines-1) = 0xff000000;
+#undef P
+#endif
+}
+
+
+static void SubpictureReleaseRegions( spu_t *p_spu, subpicture_t *p_subpic )
+{
+    while( p_subpic->p_region )
+    {
+        subpicture_region_t *p_region = p_subpic->p_region;
+        p_subpic->p_region = p_region->p_next;
+        spu_DestroyRegion( p_spu, p_region );
+    }
+    p_subpic->p_region = NULL;
+}
+
+/* */
+static ass_handle_t *AssHandleYield( decoder_t *p_dec )
+{
+    vlc_mutex_t *p_lock = var_AcquireMutex( "libass" );
+    if( !p_lock )
+        return NULL;
+
+    ass_handle_t *p_ass = NULL;
+    ass_library_t *p_library = NULL;
+    ass_renderer_t *p_renderer = NULL;
+    vlc_value_t val;
+
+    var_Create( p_dec->p_libvlc, "libass-handle", VLC_VAR_ADDRESS );
+    if( var_Get( p_dec->p_libvlc, "libass-handle", &val ) )
+        val.p_address = NULL;
+
+    if( val.p_address )
+    {
+        p_ass = val.p_address;
+
+        p_ass->i_refcount++;
+
+        vlc_mutex_unlock( p_lock );
+        return p_ass;
+    }
+
+    /* */
+    p_ass = malloc( sizeof(*p_ass) );
+    if( !p_ass )
+        goto error;
+
+    /* */
+    p_ass->p_libvlc = VLC_OBJECT(p_dec->p_libvlc);
+    p_ass->p_lock = p_lock;
+    p_ass->i_refcount = 1;
+
+    /* Create libass library */
+    p_ass->p_library = p_library = ass_library_init();
+    if( !p_library )
+        goto error;
+
+    /* load attachments */
+    input_attachment_t  **pp_attachments;
+    int                   i_attachments;
+
+    if( decoder_GetInputAttachments( p_dec, &pp_attachments, &i_attachments ))
+    {
+        i_attachments = 0;
+        pp_attachments = NULL;
+    }
+    for( int k = 0; k < i_attachments; k++ )
+    {
+        input_attachment_t *p_attach = pp_attachments[k];
+
+        if( !strcasecmp( p_attach->psz_mime, "application/x-truetype-font" ) )
+        {
+            msg_Dbg( p_dec, "adding embedded font %s", p_attach->psz_name );
+
+            ass_add_font( p_ass->p_library, p_attach->psz_name, p_attach->p_data, p_attach->i_data );
+        }
+        vlc_input_attachment_Delete( p_attach );
+    }
+    free( pp_attachments );
+
+    char *psz_font_dir = config_GetCacheDir();
+    if( !psz_font_dir )
+        goto error;
+    ass_set_fonts_dir( p_library, psz_font_dir );
+    free( psz_font_dir );
+
+    ass_set_extract_fonts( p_library, true );
+    ass_set_style_overrides( p_library, NULL );
+
+    /* Create the renderer */
+    p_ass->p_renderer = p_renderer = ass_renderer_init( p_library );
+    if( !p_renderer )
+        goto error;
+
+    ass_set_use_margins( p_renderer, false);
+    //if( false )
+    //    ass_set_margins( p_renderer, int t, int b, int l, int r);
+    ass_set_hinting( p_renderer, ASS_HINTING_NATIVE ); // No idea
+    ass_set_font_scale( p_renderer, 1.0 );
+    ass_set_line_spacing( p_renderer, 0.0 );
+
+    const char *psz_font = NULL; /* We don't ship a default font with VLC */
+    const char *psz_family = "Arial"; /* Use Arial if we can't find anything more suitable */
+#ifdef HAVE_FONTCONFIG
+    ass_set_fonts( p_renderer, psz_font, psz_family );  // setup default font/family
+#else
+    /* FIXME you HAVE to give him a font if no fontconfig */
+    ass_set_fonts_nofc( p_renderer, psz_font, psz_family );
+#endif
+    memset( &p_ass->fmt, 0, sizeof(p_ass->fmt) );
+
+    /* */
+    val.p_address = p_ass;
+    var_Set( p_dec->p_libvlc, "libass-handle", val );
+
+    /* */
+    vlc_mutex_unlock( p_ass->p_lock );
+    return p_ass;
+
+error:
+    if( p_renderer )
+        ass_renderer_done( p_renderer );
+    if( p_library )
+        ass_library_done( p_library );
+
+    free( p_ass );
+    vlc_mutex_unlock( p_lock );
+    return NULL;
+}
+static void AssHandleRelease( ass_handle_t *p_ass )
+{
+    vlc_mutex_lock( p_ass->p_lock );
+    p_ass->i_refcount--;
+    if( p_ass->i_refcount > 0 )
+    {
+        vlc_mutex_unlock( p_ass->p_lock );
+        return;
+    }
+
+    ass_renderer_done( p_ass->p_renderer );
+    ass_library_done( p_ass->p_library );
+
+    vlc_value_t val;
+    val.p_address = NULL;
+    var_Set( p_ass->p_libvlc, "libass-handle", val );
+
+    vlc_mutex_unlock( p_ass->p_lock );
+    free( p_ass );
+}
+