[MPlayer-dev-eng] [PATCH]Fix subcp crash(hopefully)

[Salvador Eduardo Tropea]

陆然 wrote:

>Hi,
>On Wednesday 15 January 2003 00:08, Salvador Eduardo Tropea wrote:
>
>  
>
>>Then we could do the iconv convertion to a temporal buffer and do the
>>remove.
>>This is need by other formats also.
>>    
>>
>
>But I don't think do iconv first is better, since it convert all the codepage 
>to utf-8. Maybe the orignal codepage could be better for eliminate those {}s.
>
No if you convert the UTF-8 into an Unicode 16 bits buffer.
This is easy to do.

>>Here are some examples, but you can hit similar things randomly.
>>Dialogue: Marked=0,0:52:43.21,0:52:45.84,1 MENOS DE
>>SOMBRA,NTP,0000,0000,0000,,{\fs36}El nombre de ese muchacho es...
>>Dialogue:
>>Marked=0,0:52:46.84,0:52:48.84,FANSUB,NTP,0000,0000,0000,,SUBTITULADO
>>POR {\fnOrient\c&HFFFFFF&}ISAGARA FANSUB
>>    
>>
>
>So '{' is always followed by '\', we should find some document about these 
>things.
>
This seems to be right, I downloaded the spec and it says:

"All Override codes appear within braces { } except the newline \n and 
\N codes.
All override codes are always preceded by a backslash \
Several overrides can be used within one set of braces."

So, yes, {\ seems to be the correct start.

SET

-- 
Salvador Eduardo Tropea (SET). (Electronics Engineer)
Visit my home page: http://welcome.to/SetSoft or
http://www.geocities.com/SiliconValley/Vista/6552/
Alternative e-mail: set at computer.org set at ieee.org 
Address: Curapaligue 2124, Caseros, 3 de Febrero
Buenos Aires, (1678), ARGENTINA Phone: +(5411) 4759 0013