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

2015-10-08 Thread Ganesh Ajjanagadde
On Thu, Oct 8, 2015 at 7:28 AM, Ronald S. Bultje wrote: > Hi, > > On Wed, Oct 7, 2015 at 8:50 PM, Ganesh Ajjanagadde wrote: > >> On Wed, Oct 7, 2015 at 8:21 PM, Michael Niedermayer >> wrote: >> > On Wed, Oct 07, 2015 at 06:22:08PM -0400, Ganesh Ajjanagadde wrote: >> >> Commit 6dac8c8327 disabled

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

2015-10-08 Thread Ronald S. Bultje
Hi, On Wed, Oct 7, 2015 at 8:50 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 7, 2015 at 8:21 PM, Michael Niedermayer > wrote: > > On Wed, Oct 07, 2015 at 06:22:08PM -0400, Ganesh Ajjanagadde wrote: > >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless > warnings. > >> The conc

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

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 8:44 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Oct 7, 2015 at 6:22 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 >>

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

2015-10-07 Thread Ganesh Ajjanagadde
On Wed, Oct 7, 2015 at 8:21 PM, Michael Niedermayer wrote: > On Wed, Oct 07, 2015 at 06:22:08PM -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

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

2015-10-07 Thread Ronald S. Bultje
Hi, On Wed, Oct 7, 2015 at 6:22 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 > > http://fate.ffmpeg.org/log.cgi?time=2

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

2015-10-07 Thread Michael Niedermayer
On Wed, Oct 07, 2015 at 06:22:08PM -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 > http://fate.ffmpeg.org/log.cgi?time=20

[FFmpeg-devel] [PATCHv3] 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&log=compile&slot=x86_64-darwin-clang-polly-vectorize-stripmine-