I'm trying to update some older code I wrote that basically looks like:
trait Foo<T>{}
struct Bar<A, B: Foo<A>>{
b: B
}
This used to work totally fine, but now I am getting a compile error:
src/test.rs:19:12: 19:13 error: parameter `A` is never used
src/test.rs:19 struct Bar<A, B: Foo<A>> {
^
src/test.rs:19:12: 19:13 help: consider removing `A` or using a marker such as `core::marker::PhantomData`
So I can try to remove the type parameter and get something like this:
struct Bar<A>{
b: Foo<A>
}
however this is not really what I want. In my original code B
resolves to a sized type, but now Foo<A>
is unsized.
The other suggested solution is to try using this PhantomData the error mentions, resulting in:
struct Bar<A, B: Foo<A>> {
b: B,
marker: PhantomData<A>
}
but this seems really messy to me. Reading the docs for PhantomData
seem to indicate this is meant to be used with unsafe code, but I am not working with unsafe code anywhere here. All I want is for Bar
to contain an instance some type that implements Foo
.
Is this really the only way to handle this situation now, or am I missing something?
It seems that what you want is the following: for any type
A
,B
must implementFoo<A>
. That suggests that you rely on functionality ofFoo
that does not depend on the value ofA
, which means that you can changeFoo
to have an associated type rather than a type parameter.will become
Then you can remove the
A
type parameter everywhere and require justB: Foo
.To elaborate on
PhantomData
, it has nothing to do with unsafe code, it is used to determine variance of a type parameter when the compiler cannot infer it. See RFC 738 for more information on variance andPhantomData
.Depending on how your real
Foo
is, you might be able to work with associated types instead, like this:otherwise (if you do need to have the type parameter on
Foo
),PhantomData
is indeed the way to go.You were not the only person finding
PhantomData
's docs confusing (seePhantomData
is incomprehensible). As a result, the documentation forPhantomData
has been recently improved by Steve Klabnik and now it does mention this scenario explicitly (and not just unsafe code).