Repository navigation
How box should work #413
Description
Activity
It has to clean up allocated resources if
result_expressioninbox (place_expression) result_expressionthrows an exception. It also needs to support in-place constructor in collections via various methods likeemplace_frontandemplace_backonstd::dequein C++.emplace_backcould be expressed straightforwardly using&outwithout even involvingbox:fn emplace_back<'s, T>(self: &'s mut Vec<T>) -> &'s out T;see also alternative design (needs RFC) with prototype at Rust PR 18233: rust-lang/rust/pull/18233
(in particular, any design needs to ensure that it handles failure of the
<value-expr>inbox (<place-expr>) <value-expr>properly; I think the description outlined in this issue is relying on&outdoing the cleanup on failure automatically, but I'm not sure that design works out once we get rid of drop flags...)(ah, apologies to @thestinger who said much the same thing a few comments up.)
&outpointers are linear, and therefore don't work well with unwinding, as there is no correct destructor for the&out Twhen you don't have aT, so locals can't be linear.Something like:
trait AllocHole<'a, out T> { // associated field. const associated fields can't be assigned to, // but writing to an &out associated field destroys its owner (this // doesn't work with trait objects). const internals: &'a out T } // fundep version. replace out parameters with associated types for trait Allocator<'a, T, PLACE, out HOLE> where HOLE : AllocHole<'a, T>, PLACE: Copy { fn alloc(&'a out self, p: PLACE) -> HOLE; } // We may need some hacks to convince typeck // that Box<#[generic j]> impls Allocator<#[generic j], (), ...> impl<'a, T> Allocator<T, (), BoxHole<'a, T>> for Box<T> fn alloc(&'a out self, p: ()) -> BoxHole<'a, T> { unsafe { let ptr = malloc::<T>(); *self = Box(ptr); BoxHole(transmute(ptr)) } } } struct BoxHole<'a, T>(&'a out T); impl<'a, T> Drop for BoxHole<'a, T> { fn drop(&mut self) { unsafe { free::<T>(transmute(self.0)); } } } // box(PLACE) EXPR should translate to // { // let mut result; // let mut hole = (&out result).alloc(PLACE); // *hole = EXPR; // result // }
We would need some kind of linear types and constant linear associated types, but it could work.
I don't think my proposal requires typeck hacks/HKT/whatever, because the following code,
which is pretty similar, works today:trait MyTrait<T, P> { fn work(&mut self, p: P, t: T); } impl<T> MyTrait<T, ()> for Option<T> { fn work(&mut self, _: (), _: T) {} } fn main() { let mut o = None; o.work((), 0u); println!("{}", o); }
Of course, to be generic over allocators, one still needs HKT, Π₂TB, or Π₁TB + Associated Types.
- addedT-langRelevant to the language team, which will review and decide on the RFC.Relevant to the language team, which will review and decide on the RFC.
on Jan 19, 2018 - addedA-expressionsTerm language related proposals & ideasTerm language related proposals & ideasA-operatorOperators related proposals.Operators related proposals.A-placement-newProposals relating to placement new / box expressions.Proposals relating to placement new / box expressions.A-allocationProposals relating to allocation.Proposals relating to allocation.A-uninit&uninit related proposals & ideas&uninit related proposals & ideasA-traits-libstdStandard library trait related proposals & ideasStandard library trait related proposals & ideas
on Nov 27, 2018 i have an idea for how it should work: not
Reacted by kennytm
We need to come up with a final design for how the
boxoperator should work.Here is what I personally think would be ideal; unfortunately it involves features which are not yet part of the language.
The
boxoperator would desugar to a call tomake_box(see below). If a value-level argument is required to specify where the box should be allocated, such as for arenas (i.e.box(here) valsyntax), its type is specified by theWhereassociated type. Most of the time this defaults to().MakeBoxitself is a higher-kinded trait, implemented for the generic box type itself. (So if you haveimpl MakeBox for SomeBox,val: T, then e.g.box val: SomeBox<T>.)make_boxrelies on&outreferences such as described by RFC PR 98 (see also the comments), which is a reference type which points to an uninitialized value and represents the obligation to initialize it.make_boxtakes as argument the obligation to initialize a box, e.g. aSomeBox<T>, allocates the box itself, and returns to the caller the obligation to initialize the contained value.We would have
impls such as:Given an expression:
from_string()).SomeBox<Foo>, wherefoo: Foo,SomeBox: MakeBox, andhere: <SomeBox as MakeBox>::Where.hereis omitted, it defaults to(), sobox foobecomesbox(()) foo.Box.let rc_foo: Rc<_> = box foo;. When we gain general type ascription syntax, it could instead bebox foo: Rc<_>. (In each case the result type isRc<Foo>.)This latter point is why it is important for the trait to be higher-kinded. If the trait is higher-kinded, it is sufficient to annotate the outer type (the box type), and the type argument can be inferred, because it is the same as the type of the value being
boxed. IfMakeBoxwere formulated using an associated type instead, as e.g.Deref, then the type ascription would be required to specify the full type, including the type of the contained value.Finally, given the above expression:
This would desugar to:
The second line initializes
the_boxusingmake_box, and then initializes the contained value withfoo. (foohere is an arbitrary expression, not necessarily a variable.)