-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Closed
Labels
A-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcE-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Description
Currently within
macro_rules! uint_module { ($T:ty) => (
we have, after things that do use $T
, the pair of tests:
#[test]
fn test_uint_to_str_overflow() { ... }
#[test]
fn test_uint_from_str_overflow() { ... }
that do not reference $T
.
As far as I can tell, these tests do not need to be inside the macro definition. They should live outside it, so that we will not generate code for them and redundantly run them once for every integer type variant.
Metadata
Metadata
Assignees
Labels
A-testsuiteArea: The testsuite used to check the correctness of rustcArea: The testsuite used to check the correctness of rustcE-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.