<div dir="ltr"><div><div><div><div>bstrie: you're right it is a trade off, but I don't agree that its not worth it. We're talking about non-atomic incrementing of an integer - that is pretty much the cheapest thing you can do on a processor (not free of course, since caching, etc., but still very cheap). I've programmed a lot in C++ with ref counted pointers and never had a problem remembering that there is a cost, and it makes using them pleasant. I found all the clone()s in Rust unpleasant, it really put me off using ref counting. The transition from using references to using Rc was particularly awful. Given that this is something C++ programmers coming to Rust will be used to using, I believe ergonomics is especially important.<br>
<br></div>In this case I don't think we need to aim to be more 'bare metal' than C++. Transparent, ref counted pointers in C++ are popular and seem to work pretty well, although obviously not perfectly.<br><br>
</div>zwarich: I haven't thought this through to a great extent, and I don't think here is the right place to plan the API. But, you ought to still have control over whether an Rc pointer is copied or referenced. If you have an Rc<T> object and pass it to a function which takes an Rc<T>, it is copied, if it takes a &Rc<T> or a &T then it references (in the latter case with an autoderef-ref). If the function is parametric over U and takes a &U, then we instantiate U with either Rc<T> or T (in either case it would be passed by ref without an increment, deciding which is not changed by having a copy constructor). If the function takes a U literal, then U must be instantiated with Rc<T>. So, you still get to control whether you reference with an increment or not.<br>
<br></div>I think if Rc is copy, then it is always copied. I would not expect it to ever move. I don't think that is untenable, performance wise, after all it is what everyone is currently doing in C++. I agree the second option seems unpredictable and thus less pleasant.<br>
<br></div>Cheers, Nick<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 21, 2014 at 4:05 PM, Cameron Zwarich <span dir="ltr"><<a href="mailto:zwarich@mozilla.com" target="_blank">zwarich@mozilla.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>I sort of like being forced to use .clone() to clone a ref-counted value, since it makes the memory accesses and increment more explicit and forces you to think which functions actually need to take an Rc and which functions can simply take an &.</div>
<div><br></div><div>Also, if Rc becomes implicitly copyable, then would it be copied rather than moved on every use, or would you move it on the last use? The former seems untenable for performance reasons, since removing unnecessary ref-count operations is important for performance. The latter seems unpredictable, since adding a second use of a value in a function would mean that new code is implicitly executed wherever the first use is.</div>
<span class="HOEnZb"><font color="#888888"><div><br></div><div>Cameron</div></font></span><div><div class="h5"> <br><div><div>On Jun 20, 2014, at 8:49 PM, Nick Cameron <<a href="mailto:lists@ncameron.org" target="_blank">lists@ncameron.org</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr"><div>I think having copy constructors is the only way to get rid of `.clone()` all over the place when using` Rc`. That, to me, seems very important (in making smart pointers first class citizens of Rust, without this, I would rather go back to having @-pointers). The trouble is, I see incrementing a ref count as the upper bound on the work that should be done in a copy constructor and I see no way to enforce that.<br>

<br></div>So, I guess +1 to spirit of the OP, but no solid proposal for how to do it.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 21, 2014 at 8:00 AM, Benjamin Striegel <span dir="ltr"><<a href="mailto:ben.striegel@gmail.com" target="_blank">ben.striegel@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">I'm not a fan of the idea of blessing certain types with a compiler-defined whitelist. And if the choice is then between ugly code and copy constructors, I'll take ugly code over surprising code.<br>



</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 20, 2014 at 3:10 PM, Patrick Walton <span dir="ltr"><<a href="mailto:pcwalton@mozilla.com" target="_blank">pcwalton@mozilla.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 6/20/14 12:07 PM, Paulo Sérgio Almeida wrote:]<div><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Currently being Copy equates with being Pod. The more time passes and<br>
the more code examples I see, it is amazing the amount of ugliness that<br>
it causes. I wonder if there is a way out.<br>
</blockquote>
<br></div>
Part of the problem is that a lot of library code assumes that Copy types can be copied by just moving bytes around. Having copy constructors would mean that this simplifying assumption would have to change. It's doable, I suppose, but having copy constructors would have a significant downside.<br>




<br>
Patrick<br>
<br>
______________________________<u></u>_________________<br>
Rust-dev mailing list<br>
<a href="mailto:Rust-dev@mozilla.org" target="_blank">Rust-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/rust-dev" target="_blank">https://mail.mozilla.org/<u></u>listinfo/rust-dev</a><br>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
Rust-dev mailing list<br>
<a href="mailto:Rust-dev@mozilla.org" target="_blank">Rust-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/rust-dev" target="_blank">https://mail.mozilla.org/listinfo/rust-dev</a><br>
<br></blockquote></div><br></div>
_______________________________________________<br>Rust-dev mailing list<br><a href="mailto:Rust-dev@mozilla.org" target="_blank">Rust-dev@mozilla.org</a><br><a href="https://mail.mozilla.org/listinfo/rust-dev" target="_blank">https://mail.mozilla.org/listinfo/rust-dev</a><br>
</blockquote></div><br></div></div></div></blockquote></div><br></div>