search

ctaggart / autorust

MIT License
12 stars 4 forks source link

specify parameters to be boxed for recursive types #84

Closed ctaggart closed 3 years ago

ctaggart commented 3 years ago

fix #73 By allowing specific parameters to be boxed for recursive types.

rylev commented 3 years ago

@ctaggart So I guess there's no way to automatically detect which types need to be boxed?

ctaggart commented 3 years ago

@rylev, correct.

rylev commented 3 years ago

@ctaggart I see a few where we box a type because it has a field which optionally contains itself. We should be able to detect this case no? We just need to look to see if the field's type is the same as the struct.

ctaggart commented 3 years ago

@rylev, that case, yes. We can add an GitHub issue for detecting and boxing that automatically. Let's not do it as part of this PR.