Re: [FFmpeg-devel] [PATCHv2] lavf/srtdec: rewrite parsing logic

2016-01-01 Thread Clément Bœsch
On Sat, Dec 26, 2015 at 06:51:00PM +0100, Clément Bœsch wrote: > Fixes Ticket #5032 > > The samples in Ticket #5032 is using \r\r\n as line breaks. Since we > already are handling \r, or \n, or \r\n as line breaks, \r\n\n will be > considered as a double line breaks. This is an issue because > ff

[FFmpeg-devel] [PATCHv2] lavf/srtdec: rewrite parsing logic

2015-12-26 Thread Clément Bœsch
Fixes Ticket #5032 The samples in Ticket #5032 is using \r\r\n as line breaks. Since we already are handling \r, or \n, or \r\n as line breaks, \r\n\n will be considered as a double line breaks. This is an issue because ff_subtitles_read_text_chunk() will as a result stop extracting a chunk after