[MPlayer-dev-eng] Re: [PATCH] Better handing of ssa subtitle

[matthieu]

 >On Fri, 30 Jan 2004 00:55:41 +0100
 >matthieu <mat100 at ifrance.com> wrote:
 >
 >> somme ssa file like this don't have space between the beginning of the
 >> text and the last comma, so the actual parser don't work.
 >>
 >> In my patch the number of comma are based on the number of comma of the
 >> line after "[Events]". So it will work with all script.
 >> I hope I didn't broke anything.
 >
 >Well it breaks.
 >There are a lot of non-standard SSA files around that do not
 >even have a section but EVENTS and do nut lable it at all.
 >Also the Format line is many times omited.
 >
 >Thus: patch rejected.
 >
 >			Attila Kinali
 >

Ok, i have make a new patch that work even if there are no section.
But for better handing in this case I have to decrease the default 
maxcomma to 9.
But if you realy what set it to 32, for no section one, it will work 
like before minus the bug with double comma (",,") that I corrected.
I there are a section, and we are in complex case like
Dialogue: 
Marked=0,0:00:01.00,0:00:05.00,*Default,~,0000,0000,0000,,TOTO,TATA
it will also work.


Matthieu
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: subreader2.patch
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040207/8fb87115/attachment.asc>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 252 bytes
Desc: OpenPGP digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040207/8fb87115/attachment.pgp>