Thanks! All done.<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 28, 2012 at 11:27 AM, 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"><div style="word-wrap:break-word"><div class="im"><br><div><div>On Jun 27, 2012, at 15:12 , Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:</div>
<br><blockquote type="cite">New patch!<div><br></div><div>In this version:</div><div>* no longer uses <span style="font-family:arial,sans-serif;font-size:13px">FoldingSetNodeID, which hopefully addressed performance concerns (this means "s->a_ = s->a_" is no longer caught, but that has never happened to me anyway)</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px">* ObjC support as requested by Jordan</span></div><div><span style="font-family:arial,sans-serif;font-size:13px">* Fix to an existing ObjC test that triggered this warning (accidentally I think)</span></div>

</blockquote><br></div></div><div>Yeah, this is probably a better effort/reward tradeoff. Some small notes:</div><div><br></div><div>- In the ObjC one you're already comparing the two decls equal to each other; you might as well not bother to check for "self". It's very rare in idiomatic Objective-C to assign to other objects' ivars anyway, but in the rare case that you did, you'd get the "s->a_ = s->a_" warning.</div>
<div>- In ObjC they're called "instance variables", not "member variables". Can you make the diagnostic more flexible?</div><div>- A random LFunction change snuck into this patch. :-)</div><div><br>
</div><div>Thanks!</div><span class="HOEnZb"><font color="#888888"><div>Jordan</div></font></span></div>
</blockquote></div><br></div>