Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-10-06 Thread Ganesh Ajjanagadde
On Sat, Oct 3, 2015 at 9:14 AM, Ronald S. Bultje wrote: > Hi, > > On Thu, Sep 17, 2015 at 7:03 AM, Ganesh Ajjanagadde > wrote: > >> On Thu, Sep 17, 2015 at 6:15 AM, Hendrik Leppkes >> wrote: >> > On Thu, Sep 17, 2015 at 12:19 AM, Ganesh Ajjanagadde >> > wrote: >> >> This patch moves the pointer

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-10-03 Thread Ronald S. Bultje
Hi, On Thu, Sep 17, 2015 at 7:03 AM, Ganesh Ajjanagadde wrote: > On Thu, Sep 17, 2015 at 6:15 AM, Hendrik Leppkes > wrote: > > On Thu, Sep 17, 2015 at 12:19 AM, Ganesh Ajjanagadde > > wrote: > >> This patch moves the pointer validity check outside the macro, > >> and silences the -Waddress obs

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-10-03 Thread Ganesh Ajjanagadde
On Thu, Sep 17, 2015 at 6:03 AM, Ganesh Ajjanagadde wrote: > On Thu, Sep 17, 2015 at 6:15 AM, Hendrik Leppkes wrote: >> On Thu, Sep 17, 2015 at 12:19 AM, Ganesh Ajjanagadde >> wrote: >>> This patch moves the pointer validity check outside the macro, >>> and silences the -Waddress observed with G

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-09-17 Thread Ganesh Ajjanagadde
On Thu, Sep 17, 2015 at 6:15 AM, Hendrik Leppkes wrote: > On Thu, Sep 17, 2015 at 12:19 AM, Ganesh Ajjanagadde > wrote: >> This patch moves the pointer validity check outside the macro, >> and silences the -Waddress observed with GCC 5.2. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavc

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-09-17 Thread Hendrik Leppkes
On Thu, Sep 17, 2015 at 12:19 AM, Ganesh Ajjanagadde wrote: > This patch moves the pointer validity check outside the macro, > and silences the -Waddress observed with GCC 5.2. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/libx264.c | 8 +--- > 1 file changed, 5 insertions(+), 3 d

Re: [FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-09-17 Thread Hendrik Leppkes
On Thu, Sep 17, 2015 at 12:19 AM, Ganesh Ajjanagadde wrote: > This patch moves the pointer validity check outside the macro, > and silences the -Waddress observed with GCC 5.2. > What kind of bogus warnings is GCC going to invent next? This is a simple macro, and all use of this variable is prope

[FFmpeg-devel] [PATCH] avcodec/libx264: silence -Waddress

2015-09-16 Thread Ganesh Ajjanagadde
This patch moves the pointer validity check outside the macro, and silences the -Waddress observed with GCC 5.2. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/libx264.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c inde