<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jan 8, 2014 at 11:57 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@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="im">On 8 January 2014 14:45, <a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a> <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>


> FWIW there are other non-pointers that use pointer-like semantics, such as<br>
> boost::optional. So there's precedence in some pretty mainstream APIs for<br>
> this design.<br>
<br>
</div>Thanks for the pointer. I still find using * and -> confusing in<br>
something that behaves like a value. If we do decide to keep them,<br>
would you guys be ok with<br>
<br>
* Still adding the explicit get and getError methods (boost::optional<br>
has a get).<br>
* Removing the error_code conversion operator. IMHO foo.getError() is<br>
always nicer than error_code(foo).</blockquote><div><br></div><div>I agree, especially on removing error_code operator.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">
> But I don't work with code using this type very much, so it's best left up<br>
> to you/them to decide what kind of API they're most comfortable with.<br>
><br>
> Perhaps a thread link for the "as was observed before" reference would also<br>
> be useful.<br>
<br>
</div>Good point. It was<br>
<a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-November/068094.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-November/068094.html</a>.<br>
<div class="HOEnZb"><div class="h5"><br>
Cheers,<br>
Rafael<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>