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

Michael Niedermayer michael at niedermayer.cc
Sun Aug 23 20:07:13 CEST 2015


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: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> > ---
> >  libavdevice/lavfi.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> No objection, although I find that warning a rather bad idea if it does not
> have checks to see if the code is part of a macro.

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150823/ae7bd9a7/attachment.sig>


More information about the ffmpeg-devel mailing list