[vlc-commits] Subtitles: hide {y:*} in microDVD

Jean-Baptiste Kempf git at videolan.org
Thu Mar 17 00:04:37 CET 2011


vlc/vlc-1.1 | branch: master | Jean-Baptiste Kempf <jb at videolan.org> | Wed Mar 16 23:49:28 2011 +0100| [66dd0616d4c454c207e2b9db84e140c27df0e417] | committer: Jean-Baptiste Kempf

Subtitles: hide {y:*} in microDVD

This seems to be the latest fashion in poland, for {y:i} (italic) and
{y:b} (bold), but I don't know how to correctly support it:
 - should we remove the tags in the demuxer like the MPL2 one (bad
   solution for MPL2 too)?
 - should we replace it by <i> in codecs?
(cherry picked from commit 195b652da5facd60397504f706a44d97bd70eb90)

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc/vlc-1.1.git/?a=commit;h=66dd0616d4c454c207e2b9db84e140c27df0e417
---

 modules/codec/subtitles/subsdec.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/modules/codec/subtitles/subsdec.c b/modules/codec/subtitles/subsdec.c
index bfdbb13..d67145d 100644
--- a/modules/codec/subtitles/subsdec.c
+++ b/modules/codec/subtitles/subsdec.c
@@ -973,7 +973,7 @@ static char *CreateHtmlSubtitle( int *pi_align, char *psz_subtitle )
             /* Hide {\stupidity} */
             psz_subtitle = strchr( psz_subtitle, '}' ) + 1;
         }
-        else if( psz_subtitle[0] == '{' && psz_subtitle[1] == 'Y'
+        else if( psz_subtitle[0] == '{' && ( psz_subtitle[1] == 'Y' || psz_subtitle[1] == 'y' )
                 && psz_subtitle[2] == ':' && strchr( psz_subtitle, '}' ) )
         {
             /* Hide {Y:stupidity} */



More information about the vlc-commits mailing list