[cfe-commits] [patch] Rename -Wself-assign-memvar to -Wself-assign-field

Chandler Carruth chandlerc at google.com
Mon Jul 2 17:00:40 PDT 2012


LGTM. If someone wants to re-paint the bikeshed, they should repaint all of
them. This seems a strict improvement as the number of colors goes down by
one.


On Mon, Jul 2, 2012 at 4:56 PM, Nico Weber <thakis at chromium.org> wrote:

> Hi,
>
> the attached patch renames -Wself-assign-memvar to
> -Wself-assign-field. matthewbg requested this, to match
> -Wunused-private-field.
>
> (If people don't like this, andersca points out that we should rename
> the flag to -Wself-assign-member-variable instead. Let me know which
> version you prefer.)
>
> Nico
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120702/f144e430/attachment.html>


More information about the cfe-commits mailing list