<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Sep 4, 2011, at 3:59 PM, David Blaikie wrote:</div><blockquote type="cite">Hmm - apologies. Realized I'd not actually sent the relevant patch, but just a copy of the previous one that Doug had already applied.<div><br></div><div>Here's the right one.<br><br><div class="gmail_quote">On Tue, Aug 30, 2011 at 12:18 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><br><br><div class="gmail_quote"><div><div></div><div class="h5">On Fri, Aug 26, 2011 at 6:39 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><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><blockquote type="cite"><div>Side note: Do we really need that first note if we're providing the second note? I realize in some cases the first note is all we'll get, because there's no initialization we can guess to provide. But in the cases where there is an init suggestion, could we just provide that instead of the "variable '%s' is declared here"?</div>
</blockquote><div><br></div></div><div>Good point. It seems that we don't need to provide the first note if we know we're providing the second. Looping in Ted to get his opinion.</div></div></div></blockquote></div>
<br></div><div>Well - here's a patch you can use if you decide it's the right way to go. (I'm only seeing one clang test failure in something unrelated: Modules/lookup.cpp)</div></blockquote><div><br></div></div>
</div><div>
Ted - any thoughts on this?</div><div><br></div><div>- David</div><div><br></div><div>[should I reinclude a patch when I bump a CR? generally I just assume people will see the original email with the patch attached & I don't want to fill up people's mailboxes more than necessary] </div>
</div></blockquote></div></div></blockquote></div><br><div>I like the direction of this patch, but I think that the note text used here</div><div><br></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>note: <span class="Apple-tab-span" style="white-space:pre">add initialization to silence this warnin</span>g</div><div><br></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 class="Apple-tab-span" style="white-space:pre"> </span>note: initialize the variable 'x' to silence this warning</div><div><br></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div></body></html>