Hi,
it would be nice if libass was parsing [V4+ Styles] case
insensitive.

When I have [v4+ Styles] in ass file (some program saves it
as this - I 
know of one - Subtitle Processor), then Mplayer with libass
support turned 
on doesn't display subtitles.

It is obvious from source code that performed comparision is
case 
sensitive.

http://svn.mplayerhq.hu/mplayer
/trunk/libass/ass.c?revision=25610&view=markup
... if (!strncmp(str, "[V4+ Styles]", 12)) ...


Of course, manual editing of ass files (s/v4+ Styles/V4+
Styles/) is 
possible, however it should not be much work to make it case
insensitive, 
and I think that some "problem" tolerance in
decoder is always welcome.


Tom

_______________________________________________
MPlayer-users mailing list
MPlayer-usersmplayerhq.hu
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-use
rs

On Saturday 29 March 2008 09:57:45 Tomáš Jirka wrote:
> Hi,
> it would be nice if libass was parsing [V4+ Styles]
case insensitive.
>
> When I have [v4+ Styles] in ass file (some program
saves it as this - I
> know of one - Subtitle Processor), then Mplayer with
libass support turned
> on doesn't display subtitles.
>
> It is obvious from source code that performed
comparision is case
> sensitive.
>
> http://svn.mplayerhq.hu/mplayer/t
runk/libass/ass.c?revision=25610&view=mark
>up ... if (!strncmp(str, "[V4+ Styles]", 12))
...
>
>
> Of course, manual editing of ass files (s/v4+
Styles/V4+ Styles/) is
> possible, however it should not be much work to make it
case insensitive,
> and I think that some "problem" tolerance in
decoder is always welcome.

A good idea. In fact, "v4+ Styles" is correct
according to specs, but every 
subtitle I've seen uses "V4+ Styles". I've made
all section headers case 
insensitive in svn.
_______________________________________________
MPlayer-users mailing list
MPlayer-usersmplayerhq.hu
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-use
rs