[FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

Moritz Barsnick barsnick at gmx.net
Mon Mar 14 22:14:37 CET 2016


On Mon, Mar 14, 2016 at 19:37:33 +0100, Thilo Borgmann wrote:
> > So
> >   if (x)
> >       SafeCFRelease(a);
> >   else
> >       SafeCFRelease(b);
> > 
> > can work.
> 
> At least the clang preprocessor keeps the outer {} so that this should work.
> I'm not arguing - if experience says compilers act differently here, so
> be it. I already changed it locally...

I think it's not about the compilers, it's about providing a macro
which safely behaves like a function in every usecase, without any need
for care or syntax around its usage. This explanation is quite okay:
http://stackoverflow.com/a/154264/3974309

Moritz


More information about the ffmpeg-devel mailing list