<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Sep 9, 2011, at 12:19 AM, David Blaikie wrote:</div><br class="Apple-interchange-newline"><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 class="h5">I like the direction of this patch, but I think that the note text used here</div>
</div><div class="im"><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>Looks good! Please go ahead and commit.</div><div><br></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div><br></body></html>