<br><br><div class="gmail_quote">On Fri, Sep 9, 2011 at 7:46 AM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com">dgregor@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><div></div><div class="h5"><br><div><div>On Sep 9, 2011, at 12:19 AM, David Blaikie wrote:</div><br><blockquote type="cite"><div class="gmail_quote"><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><div>I like the direction of this patch, but I think that the note text used here</div>
</div><div><div><br></div><div><span style="white-space:pre-wrap">  </span>note: <span style="white-space:pre-wrap">add initialization to silence this warnin</span>g</div><div><br></div></div><div>isn't very helpful. If we're only going to print one note, I'd rather it say something like</div>

<div><br></div><div><span style="white-space:pre-wrap"> </span>note: initialize the variable 'x' to silence this warning</div></div></blockquote><div><br></div><div>Changed to use this suggested diagnostic message.</div>

</div></blockquote><br></div></div></div><div>Looks good! Please go ahead and commit.</div></div></blockquote><div><br></div><div>Thanks Doug, committed as 139463 (had myself a little confused about what I named that patch file when I sent it for review - been juggling things around a little too much recently)</div>
<div><br></div><div>- David </div></div>