<div dir="ltr"><div>Hi Nick and Michael,<br></div><div><br></div><div>Is there any reason not to add the explicit keyword to one parameter constructor? Does this patch look OK?</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Tue, May 7, 2013 at 5:10 PM, Shankar Kalpathi Easwaran <span dir="ltr"><<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
  LGTM, but explicit was not added for some reason which I dont know. You should ask Michael/Nick on that.<br>
<br>
  Please add them as reviewers as well.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D765" target="_blank">http://llvm-reviews.chandlerc.com/D765</a><br>
</blockquote></div><br></div>