Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-08 Thread Nicolas George
Le sextidi 16 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > Anyway, it seems to me that for whatever reason short circuit > evaluation is not respected by the GCC preprocessor (I lack the energy > or interest to investigate why, it seems ridiculous to me): >

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-08 Thread Ganesh Ajjanagadde
On Thu, Oct 8, 2015 at 3:11 AM, Nicolas George wrote: > Le sextidi 16 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> Anyway, it seems to me that for whatever reason short circuit >> evaluation is not respected by the GCC preprocessor (I lack the energy >> or interest to

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 3:34 PM, Michael Niedermayer wrote: > On Wed, Oct 07, 2015 at 12:29:35PM -0400, Ganesh Ajjanagadde wrote: >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. >> The concept of av_uninit is inherently useful though. This

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 12:29:35PM -0400, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

Re: [FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 12:29 PM, Ganesh Ajjanagadde wrote: > Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. > The concept of av_uninit is inherently useful though. This patch silences a > bunch of warnings on clang e.g >

[FFmpeg-devel] [PATCHv2] avutil/attributes: extend av_uninit to clang

2015-10-07 Thread Ganesh Ajjanagadde
Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. The concept of av_uninit is inherently useful though. This patch silences a bunch of warnings on clang e.g http://fate.ffmpeg.org/log.cgi?time=20150918181527=compile=x86_64-darwin-clang-polly-vectorize-stripmine-3.7.