<div dir="rtl"><div dir="ltr">Hi Aaron,</div><div dir="ltr"><br></div><div dir="ltr">There are going to be false positives with the local shadowing (compared with real bugs). </div><div dir="ltr">I don't know about the parameter and memeber ones. We could look into few of these warnings to get the "feel".<br>

</div><div dir="ltr"><br></div><div dir="ltr">Oliver - could you copy and paste the list of wd4457 and wd4458 from the VS 2014 build window ?</div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2014-08-01 17:14 GMT+03:00 Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span>:</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Fri, Aug 1, 2014 at 10:01 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>


> If I understand these warning correctly (they are not documented on MS web<br>
> site yet), they warn about local variable  shadowing some other definition.<br>
> If so, none are good to suppress, to the contrary, these new warnings are<br>
> likely to expose bugs in the current codebase.<br>
><br>
> Shadow situations cause very confusing bugs. We have long functions in<br>
> clang, for instance Sema::ClassifyName is about 300 lines. If I need to<br>
> modify this function, I glance at its header and see Scope *S. I would<br>
> expect S refer to the input paramater all along the function, rather being<br>
> shadowed by some local variable. I can check with VS goto declaration what<br>
> II refers to but why? I've seen it's an input parameter, right? name<br>
> shadowing is a trap.<br>
<br>
</div>Shadowing is generally a trap, but the local and parameter warnings<br>
are going to have a *lot* of false positives due to our naming<br>
conventions. This is especially true for things like loop counters,<br>
etc. I would rather not turn on that chatty of a diagnostic until<br>
we've sanitized the codebase further.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>