Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-02-02 Thread James Almer
On 2/2/2018 5:20 PM, Derek Buitenhuis wrote: > On 2/2/2018 7:46 PM, James Almer wrote: >> Pushed as is. Removing the two lines after printing bogus values for >> four releases is imo not nice. For starters, it can't be backported. >> >> They will be removed alongside AVStream->codec in the future.

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-02-02 Thread Derek Buitenhuis
On 2/2/2018 7:46 PM, James Almer wrote: > Pushed as is. Removing the two lines after printing bogus values for > four releases is imo not nice. For starters, it can't be backported. > > They will be removed alongside AVStream->codec in the future. fftools/ffprobe.c:2918:1: error: unknown type nam

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-02-02 Thread James Almer
On 1/25/2018 3:26 AM, Li, Zhong wrote: >>> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On >> Behalf >>> Of Hendrik Leppkes >>> Sent: Friday, January 19, 2018 6:10 PM >>> To: FFmpeg development discussions and patches >>> >>

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-01-24 Thread Li, Zhong
> > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On > Behalf > > Of Hendrik Leppkes > > Sent: Friday, January 19, 2018 6:10 PM > > To: FFmpeg development discussions and patches > > > > Subject: Re: [FFmpeg-devel] [PATCH] ffprobe: Initializ

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-01-21 Thread Li, Zhong
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Hendrik Leppkes > Sent: Friday, January 19, 2018 6:10 PM > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height > > On F

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-01-19 Thread Hendrik Leppkes
On Fri, Jan 19, 2018 at 6:05 AM, Zhong Li wrote: > coded_width/height are unnitialized and will be overwritten by > dec_ctx->width/height in avcodec_open2() > > This fixes tiket #6958. > > Signed-off-by: Zhong Li > --- > fftools/ffprobe.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --

[FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

2018-01-18 Thread Zhong Li
coded_width/height are unnitialized and will be overwritten by dec_ctx->width/height in avcodec_open2() This fixes tiket #6958. Signed-off-by: Zhong Li --- fftools/ffprobe.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index 0e7a771..233760d 10