Re: [FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

2015-06-22 Thread Philip Langdale
On Mon, 22 Jun 2015 19:56:06 +0530 Niklesh Lalwani wrote: > Updated patch. > > Thanks, > Niklesh Pushed. Thanks. --phil ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

2015-06-22 Thread Niklesh Lalwani
Updated patch. Thanks, Niklesh 0001-movtextdec.c-Improve-upon-dynarrays-and-text_to_ass.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

2015-06-20 Thread Philip Langdale
On Thu, 18 Jun 2015 22:26:52 +0530 Niklesh Lalwani wrote: > Updated patch attached. > > On 18-Jun-2015 2:52 PM, "Clément Bœsch" wrote: > av_dynarray_add(&style_flags, &index, flag); > > > +s_temp->style_flag = AV_RB8(tsmb); > > > +count = i; > > > > > +

Re: [FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

2015-06-18 Thread Niklesh Lalwani
Updated patch attached. On 18-Jun-2015 2:52 PM, "Clément Bœsch" wrote: av_dynarray_add(&style_flags, &index, flag); > > +s_temp->style_flag = AV_RB8(tsmb); > > +count = i; > > > +av_dynarray_add(&s, &count, s_temp); > > missing check > >

Re: [FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

2015-06-18 Thread Clément Bœsch
On Wed, Jun 17, 2015 at 08:37:04PM +0530, Niklesh Lalwani wrote: > From: Niklesh > > This Patch uses a single dynarray instead of 3 dynarrays used before. Changes > are also done in text_to_ass() which previously failed to produce correct > results for a few specific cases. > Signed-off-by: Nik

[FFmpeg-devel] [PATCH 1/2] movtextdec.c: Improve upon dynarrays and text_to_ass

2015-06-17 Thread Niklesh Lalwani
From: Niklesh This Patch uses a single dynarray instead of 3 dynarrays used before. Changes are also done in text_to_ass() which previously failed to produce correct results for a few specific cases. Signed-off-by: Niklesh --- libavcodec/movtextdec.c | 83 -