I'd be very happy for the diagnostic text to use more specialized terminology when it can -- I think that makes the user experience quite a bit better.<div><br></div><div>Not sure its worth having two flags though.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 2, 2012 at 5:38 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I will again say that Objective-C has "instance variables", not "fields" or "member variables". But even then I think "field" is better than "memvar" or "member variable", even though "ivar" is fairly well-known in Objective-C (and is present in some of our warnings).<br>

<div class="HOEnZb"><div class="h5"><br>
<br>
On Jul 2, 2012, at 5:00 PM, Chandler Carruth wrote:<br>
<br>
> 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.<br>
><br>
><br>
> On Mon, Jul 2, 2012 at 4:56 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
> Hi,<br>
><br>
> the attached patch renames -Wself-assign-memvar to<br>
> -Wself-assign-field. matthewbg requested this, to match<br>
> -Wunused-private-field.<br>
><br>
> (If people don't like this, andersca points out that we should rename<br>
> the flag to -Wself-assign-member-variable instead. Let me know which<br>
> version you prefer.)<br>
><br>
> Nico<br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
</div></div></blockquote></div><br></div>