<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 15, 2013 at 9:42 AM, 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 dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Wed, May 15, 2013 at 2:01 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2013/5/15 David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>>:<br>
<div>> +++ llvm/trunk/lib/Support/Windows/Windows.h Wed May 15 02:36:59 2013<br>
> @@ -75,7 +75,7 @@ public:<br>
>    }<br>
><br>
>    // True if Handle is valid.<br>
> -  operator bool() const {<br>
> +  LLVM_EXPLICIT operator bool() const {<br>
>      return HandleTraits::IsValid(Handle) ? true : false;<br>
>    }<br>
<br>
</div>You've forgotten to IWYU.<br>
Hopefully r181873 is the right fix.<br></blockquote><div><br></div></div></div><div>Yep, looks fine - sorry for the breakage. (you could use LLVM_DELETED_FUNCTION for the ScopedHandle copy operations too, not that there' <br>

</div></div><br></div></div></blockquote><div style><br>Not that there's much point in a Windows-only header. (side note: might be nice to pull ScopedHandle (& anything else possible) out into generic LLVM headers - maybe with unit tests, then this would've ideally broken even for a non-Windows developer such as myself) </div>
</div><br></div></div>