<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 30, 2013 at 1:55 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

* should we consider adding an ArrayRef implicit ctor from "None"?<br>
(see llvm/ADT/None.h and its use in llvm/ADT/Optional.h) Then you<br>
could replace all the "ArrayRef<T>()" calls with "None" (you might<br>
even want to do a sed replace on existing instances of this as a<br>
separate patch (a purely mechanical patch is easy to review/apply)).</blockquote><div><br></div><div style>What's wrong with `ArrayRef<T>()`? It's explicit and clear. What is the benefit of writing `None` instead? IMHO if I saw "None" that would just confuse me and send me on a wild goose chase that would eventually terminate on finding the implicit ArrayRef ctor and then saying to myself "why the heck didn't they just write `ArrayRef<T>`"?</div>
<div style><br></div><div style>-- Sean Silva </div></div></div></div>