<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 26, 2014 at 11:48 PM, 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 class="">On Wed, Jun 25, 2014 at 9:53 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>

> On Wed, Jun 25, 2014 at 5:52 PM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:<br>
>><br>
>> On 26/06/2014 03:16, Chandler Carruth wrote:<br>
>><br>
>>><br>
>>> On Thu, Jun 26, 2014 at 2:09 AM, Alp Toker <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a><br>
>>> <mailto:<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>>> wrote:<br>
>>><br>
>>>     I've made that change, plus a couple more updated uses, and landed<br>
>>>     this in r211749.<br>
><br>
> FWIW, perhaps I wasn't clear - but my "I'd like to loop back around<br>
> with Chandler to nail down the design choice" was "I'm not signing off<br>
> on this patch without Chandler", so the commit may've been a bit<br>
> premature.<br>
<br>
</div>Given this confusion, I think appropriate to back this patch out until<br>
it's actually been signed off on.<br>
<br>
Sorry for my vague language/lack of clarity over incremental review<br>
compared to sign-off.</blockquote></div><br>Frankly, I think your language was fine. I even misread this, but I can't think of anything you could have written more clearly -- I just needed to read it more carefully.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Ultimately, I completely agree with your tradeoff assessment here. Personally, I'm on the side of a single type because I think the model of optional-external-storage with identical behavior is a reasonable semantic model for the type.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I also think that it would be good to keep consistency in naming and API with raw_..._ostream classes (in part to better distinguish them from standard _ostream variants).</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Personally, given your analysis David, I'm happy with you making the call between one type or two. <br></div></div>