Re: [FFmpeg-devel] [PATCHv2] avdevice/lavfi: fix self assignment warning

2015-08-23 Thread Michael Niedermayer
On Sun, Aug 23, 2015 at 10:16:36AM +0200, Nicolas George wrote: > Le quintidi 5 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : > > FAIL(ret) expands to statements including a silly ret=ret. > > This triggers a -Wself-assign on confirmed clang 3.6, and so we fix it. > > > > Signed-off-by: Gane

Re: [FFmpeg-devel] [PATCHv2] avdevice/lavfi: fix self assignment warning

2015-08-23 Thread Ganesh Ajjanagadde
On Sun, Aug 23, 2015 at 4:16 AM, Nicolas George wrote: > Le quintidi 5 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : >> FAIL(ret) expands to statements including a silly ret=ret. >> This triggers a -Wself-assign on confirmed clang 3.6, and so we fix it. >> >> Signed-off-by: Ganesh Ajjanagadd

Re: [FFmpeg-devel] [PATCHv2] avdevice/lavfi: fix self assignment warning

2015-08-23 Thread Nicolas George
Le quintidi 5 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : > FAIL(ret) expands to statements including a silly ret=ret. > This triggers a -Wself-assign on confirmed clang 3.6, and so we fix it. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavdevice/lavfi.c | 2 +- > 1 file changed, 1 i

Re: [FFmpeg-devel] [PATCHv2] avdevice/lavfi: fix self assignment warning

2015-08-22 Thread Ganesh Ajjanagadde
On Sat, Aug 22, 2015 at 11:24 PM, Ganesh Ajjanagadde wrote: > FAIL(ret) expands to statements including a silly ret=ret. > This triggers a -Wself-assign on confirmed clang 3.6, and so we fix it. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavdevice/lavfi.c | 2 +- > 1 file changed, 1 inserti