Re: C PATCH to add -Woverride-init-side-effects (PR c/64918)

2015-05-07 Thread Joseph Myers
On Fri, 24 Apr 2015, Marek Polacek wrote: As discussed in the PR, the initialized field with side-effects overwritten warning is sometimes not so useful, so it probably makes sense to provide an option so that users are able to specifically enable/disable it. Since the warning is enabled by

Re: C PATCH to add -Woverride-init-side-effects (PR c/64918)

2015-05-07 Thread Marek Polacek
Ping. (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64918#c6 is not addressed in this patch, it's something to be looked at before closing the PR.) On Fri, Apr 24, 2015 at 12:36:25AM +0200, Marek Polacek wrote: As discussed in the PR, the initialized field with side-effects overwritten warning

C PATCH to add -Woverride-init-side-effects (PR c/64918)

2015-04-23 Thread Marek Polacek
As discussed in the PR, the initialized field with side-effects overwritten warning is sometimes not so useful, so it probably makes sense to provide an option so that users are able to specifically enable/disable it. Since the warning is enabled by default at present, it is enabled by default