dtolnay / linkme

Safe cross-platform linker shenanigans
Apache License 2.0
646 stars 41 forks source link

!Copy test shouldn't use a ZST #20

Closed arcnmx closed 5 years ago

arcnmx commented 5 years ago

This is a bit pedantic but the crate currently doesn't really support ZSTs the way one might expect it to so it seems a bit silly to have a test make use of them. Though on the flip side maybe there should be a test that explicitly tests that ZSTs "work" as currently implemented (check that they're are always empty). IMO it may be worth just making them fail to typecheck/compile for now if the supported rustc versions are fine with const size_of? or just fixing ZST support but eh.