[FFmpeg-devel] [PATCH 2/2] Using size_t* instead of int** for dynarrays

[Michael Niedermayer]

On Mon, Apr 27, 2015 at 07:33:59AM +0530, Niklesh Lalwani wrote:
> On Mon, Apr 27, 2015 at 7:10 AM, Michael Niedermayer <michaelni at gmx.at>
> >> wrote:
> >>
> >> > From: Niklesh <niklesh.lalwani at iitb.ac.in>
> >> >
> >> > Signed-off-by: Niklesh <niklesh.lalwani at iitb.ac.in>
> >> > ---
> >> >  libavcodec/movtextdec.c | 45
> >> ++++++++++++++++++++-------------------------
> >> >  1 file changed, 20 insertions(+), 25 deletions(-)
> >> this doesnt apply cleanly (can be applied with patch -F9 though it
> >> seems)
> >>
> >> Did you apply this patch over [PATCH 1/2] posted? Could this be the
> problem?

i tried to apply patch 1 and then patch 2 IIRC


> 
> > btw, can you add a fate test ?
> >>
> >> I haven't used FATE yet, so it might take a while for me to understand it
> and add a fate test (I do intend to get familiar with it in a few days).

sure, no hurry


> 
> > or provide some testcase for (i hope you didnt already and i forgot)
> >>
> Can I provide the 3gpp file I used my code for testing?

yes

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150427/bbc37d82/attachment.asc>