Re: [FFmpeg-devel] [PATCH 3/3 v2] avformat/matroskadec: reference the existing data buffer when creating packets

2018-04-06 Thread James Almer
On 4/6/2018 8:30 PM, Michael Niedermayer wrote:
> On Thu, Apr 05, 2018 at 12:32:47PM -0300, James Almer wrote:
>> Newly allocated data buffers (wavpack, prores, compressed buffers)
>> are padded to meet the requirements of AVPacket.
>>
> 
>> About 10x speed up in matroska_parse_frame().
> 
> thats a nice speedup
> patch seems to work fine
> 
> thx

Pushed. thanks.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH 3/3 v2] avformat/matroskadec: reference the existing data buffer when creating packets

2018-04-06 Thread Michael Niedermayer
On Thu, Apr 05, 2018 at 12:32:47PM -0300, James Almer wrote:
> Newly allocated data buffers (wavpack, prores, compressed buffers)
> are padded to meet the requirements of AVPacket.
> 

> About 10x speed up in matroska_parse_frame().

thats a nice speedup
patch seems to work fine

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel