[FFmpeg-devel] [PATCH] avcodec/htmlsubtitles: Be a bit more picky on syntax

[Michael Niedermayer]

On Sun, Jul 09, 2017 at 02:28:28PM +0200, Clément Bœsch wrote:
> On Sat, Jul 08, 2017 at 11:07:48PM +0200, Michael Niedermayer wrote:
> [...]
> > 5 days passed with no comments and no suggestions on how to implement
> > this better.
> > does anyone object to the fix as in the patch to be applied ?
> 
> misc patterns I found you may want to give a try (note that these are
> completely broken garbage you may want to ignore):
> 
> 82
> 00:04:50,430 --> 00:05:01,030
> <<flash gordon
> <E9>pisode 4
> saison 1>
> www.SeriesSub.com>
> 
> 
> 57
> 00:03:38,318 --> 00:03:42,778
> <<THE HIGH ROLLERS>>
> <<Grandes Jogadores>>
> 
> 
> 2
> 00:01:10,000 --> 00:01:14,500
> >>> RebelSubTeam <<<
> 
> 
> 306
> 00:20:31,849 --> 00:20:56,839
> <font color="##FFFF00">
> <<<<www.egfire.com>>>></font>
> 
> 
> 37
> 00:02:37,750 --> 00:02:43,700
> <b>~ASUKO MARCH!~
> >>:<<
> translation by: cangii
> Retiming by: furransu</b>
> 
> 
> 1214
> 02:11:00,738 --> 02:11:05,538
> <b><<< ÓÍÀÊÑ ÒÈÉÌ < 2015 > UNACS TEAM >>></b>
> 
> 
> 1
> 00:00:01,000 --> 00:00:02,308
> <<<<<< Antes <<<<<<
> Ela é hum...
> 
> 
> 544
> 00:53:43,780 --> 00:53:45,941
> <<That's OK, >> - he calmed himself.
> 
> 
> 545
> 00:53:46,216 --> 00:53:49,083
> <<lt's not a long way to the hotel,
> the hotel is within easy reach.
> 
> 
> 894
> 00:37:59,693 --> 00:38:01,591
> mint asztalt foglaltatni
> a <<>Le Cirque-ben.
> 
> 
> I really have a LOT of matches, so these are very common patterns.
> 
> Feel free to apply your patch if it generally doesn't make these cases
> worse.

patch posted that adds this to fate
ill upload the srt file (copy and paste of above)

the patch seems not to change the output for this
i will apply the fate test before the patch here

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170711/05ef93c4/attachment.sig>