<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 17, 2014 at 2:45 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=dblaikie@gmail.com&cc=&bcc=&su=&body=','_blank');return false;" class="cremed">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 class="">On Tue, Jun 17, 2014 at 2:14 PM, Kaelyn Takata <<a href="mailto:rikka@google.com" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=rikka@google.com&cc=&bcc=&su=&body=','_blank');return false;" class="cremed">rikka@google.com</a>> wrote:<br>

><br>
> Also be more proactive about checking a correction's visibility so that<br>
> a correction requiring a module import can be distinguished from the<br>
> original typo even if it looks identical. Otherwise the correction will<br>
> be excluded and the diagnostic about needing the module import won't be<br>
> emitted.<br>
><br>
> Note that the only change to checkCorrectionVisibility other than moving<br>
> where it is at in SemaLookup.cpp is to remove a parameter that was never<br>
> used within the function.<br>
<br>
</div>Is it possible to remove that parameter in a precursor commit? You can<br>
just do that right away without code review, presumably.<br></blockquote><div><br></div><div>D'oh. Yeah, you're right. I'll do that as part of respinning the patch set (to include the changes for passing unique_ptrs by value and more importantly to split apart two patches where a major patch somehow got squashed into a minor patch).</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
> ---<br>
>  include/clang/Sema/TypoCorrection.h |  26 +++++++--<br>
>  lib/Sema/SemaLookup.cpp             | 103 +++++++++++++++++++-----------------<br>
>  2 files changed, 76 insertions(+), 53 deletions(-)<br>
><br>
><br>
</div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=cfe-commits@cs.uiuc.edu&cc=&bcc=&su=&body=','_blank');return false;" class="cremed">cfe-commits@cs.uiuc.edu</a><br>

> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div></div>