[FFmpeg-devel] [PATCH] lavf/flvdec: init AVPacket::pos to FLVTAG offset

[Suman Kancherla (సుమన్)]

Gentle ping!

On Wed, Oct 19, 2016 at 12:14 PM, Suman Kancherla (సుమన్) <
sumankancherla at google.com> wrote:

> Thanks for taking time to review my patch! I appreciate it very much!!
>
> "Suman-" is my ID on github; Unless it is objectionable, I intend to use
> it for code submissions.
>
> Thanks!
> Suman
>
>
> On Wed, Oct 19, 2016 at 12:03 PM, Michael Niedermayer <
> michael at niedermayer.cc> wrote:
>
>> On Tue, Oct 18, 2016 at 03:13:23PM -0700, Suman- wrote:
>> > Current code doesn't initialize AVPacket::pos. Made it point to FLVTAG
>> so flv_read_packet can decode from pos
>> > ---
>> >  libavformat/flvdec.c                    |  1 +
>> >  tests/ref/seek/acodec-adpcm-swf         | 46
>> ++++++++++++++++-----------------
>> >  tests/ref/seek/acodec-adpcm-swf-trellis | 46
>> ++++++++++++++++-----------------
>> >  tests/ref/seek/lavf-flv_fmt             | 36 +++++++++++++-------------
>> >  tests/ref/seek/vsynth_lena-flashsv      | 40
>> ++++++++++++++--------------
>> >  tests/ref/seek/vsynth_lena-flv          | 40
>> ++++++++++++++--------------
>> >  6 files changed, 105 insertions(+), 104 deletions(-)
>>
>> Is it intended that the author of this patch is set to "Suman-" ?
>>
>> If its incomplete then please resubmit the patch with correctly set
>> author
>>
>>
>> [...]
>> --
>> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> Many things microsoft did are stupid, but not doing something just because
>> microsoft did it is even more stupid. If everything ms did were stupid
>> they
>> would be bankrupt already.
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>>
>
>
> --
> Many thanks!
> Suman
>
>


-- 
Many thanks!
Suman