[PATCH] D18271: Avoid -Wshadow warnings about constructor parameters named after fields
Reid Kleckner via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 7 17:16:40 PDT 2016
On Thu, Apr 7, 2016 at 5:12 PM, David Blaikie <dblaikie at gmail.com> wrote:
> On Thu, Apr 7, 2016 at 5:05 PM, Reid Kleckner via cfe-commits <
> cfe-commits at lists.llvm.org> wrote:
>> rnk updated this revision to Diff 52982.
>> rnk marked 3 inline comments as done.
>> rnk added a comment.
>> - Add -Wshadow-all and -Wshadow-field-in-constructor, also address review
> Strikes me as strange that we're adding a new name for a flag that matches
> the behavior of GCC's -Wshadow, rather than introducing the better one
> under a new name. (that way people already using it would continue to get
> the same behavior - people who came from GCC would get the same behavior
> they were already getting, etc)
Richard expressed a desire to be able to tell people to "just use
-Wshadow". I think it's important to have short, high-level options that do
the right thing, and given how many users have asked us for this change to
-Wshadow, I think this new behavior is the right thing.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits