Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-10 Thread wm4
On Fri, 10 Feb 2017 15:48:55 + Saverio Blasi wrote: > > You probably shouldn't update options_buffer_size before the reallocation > > actually succeeded. (Probably doesn't matter with the current code, but for > > robustness...) > This is now addressed. > > > Still not sure why there's a

Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-10 Thread Saverio Blasi
eg-devel-boun...@ffmpeg.org] On Behalf Of wm4 Sent: 08 February 2017 09:28 To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg On Wed, 8 Feb 2017 08:41:56 + Saverio Blasi wrote: > - This patch contains the changes to interface the

Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-10 Thread Saverio Blasi
fmpeg-devel-boun...@ffmpeg.org] On Behalf Of Mark Thompson Sent: 08 February 2017 13:06 To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg On 08/02/17 08:41, Saverio Blasi wrote: > - This patch contains the changes to int

Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-08 Thread wm4
On Wed, 8 Feb 2017 13:05:40 + Mark Thompson wrote: > On 08/02/17 08:41, Saverio Blasi wrote: > > +avctx->extradata = av_malloc(avctx->extradata_size + > > AV_INPUT_BUFFER_PADDING_SIZE); > > +if (!avctx->extradata) { > > +av_log(avctx, AV_LOG_ERROR, "Failed to all

Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-08 Thread Mark Thompson
On 08/02/17 08:41, Saverio Blasi wrote: > - This patch contains the changes to interface the Turing codec > (http://turingcodec.org/) with ffmpeg. The patch was modified to address the > comments in the review as follows: > - Added a pkg-config file to list all dependencies required by libturin

Re: [FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-08 Thread wm4
On Wed, 8 Feb 2017 08:41:56 + Saverio Blasi wrote: > - This patch contains the changes to interface the Turing codec > (http://turingcodec.org/) with ffmpeg. The patch was modified to address the > comments in the review as follows: > - Added a pkg-config file to list all dependencies re

[FFmpeg-devel] [PATCH v5] Added Turing codec interface for ffmpeg

2017-02-08 Thread Saverio Blasi
- This patch contains the changes to interface the Turing codec (http://turingcodec.org/) with ffmpeg. The patch was modified to address the comments in the review as follows: - Added a pkg-config file to list all dependencies required by libturing. This should address the issue pointed out by