Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Tue, Oct 06, 2015 at 22:52:28 -0400, Ganesh Ajjanagadde wrote: > ping: the reason I persist in this is because (long-term) the only 2 > compilers where we can reasonably reach a near "warning is regression" > state and benefit from it are clang and gcc. BTW, are the "remaining" (meaning: more

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Wed, Oct 07, 2015 at 17:47:44 -0400, Ganesh Ajjanagadde wrote: > Since I am currently in the warning suppression business, I can help > out at least in some basic checking. Unfortunately, I believe the icc > is not freely licensed, which means I can't test this stuff. I'll get in touch "when I

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Wed, Oct 07, 2015 at 16:07:53 -0400, Ronald S. Bultje wrote: > > BTW, are the "remaining" (meaning: more than from gcc) warnings emitted > > by the Intel C(++) Compiler redundant or false? The icc configuration > > should then probably silence them. > > I doubt anyone has ever investigated

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Wed, Oct 7, 2015 at 5:35 PM, Moritz Barsnick wrote: > On Wed, Oct 07, 2015 at 16:07:53 -0400, Ronald S. Bultje wrote: >> > BTW, are the "remaining" (meaning: more than from gcc) warnings emitted >> > by the Intel C(++) Compiler redundant or false? The icc configuration >> >

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

Hi, On Wed, Oct 7, 2015 at 3:46 PM, Moritz Barsnick wrote: > On Tue, Oct 06, 2015 at 22:52:28 -0400, Ganesh Ajjanagadde wrote: > > ping: the reason I persist in this is because (long-term) the only 2 > > compilers where we can reasonably reach a near "warning is regression" >

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Sat, Sep 19, 2015 at 01:00:17AM -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] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Wed, Oct 7, 2015 at 8:56 AM, Michael Niedermayer wrote: > On Sat, Sep 19, 2015 at 01:00:17AM -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] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Sat, Oct 3, 2015 at 8:38 AM, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 1:32 PM, Ganesh Ajjanagadde > wrote: >> On Sat, Sep 19, 2015 at 1:00 AM, Ganesh Ajjanagadde >> wrote: >>> Commit 6dac8c8327 disabled

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Sat, Sep 26, 2015 at 1:32 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 1:00 AM, Ganesh Ajjanagadde > wrote: >> Commit 6dac8c8327 disabled av_uninit for clang, due to some useless warnings. >> The concept of av_uninit is inherently

Re: [FFmpeg-devel] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

On Sat, Sep 19, 2015 at 1:00 AM, 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] [PATCH 1/2] avutil/attributes: extend av_uninit to clang

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.