Skip to content

Use enum named fields in to_string macro #294

Merged
Peternator7 merged 5 commits intoPeternator7:masterfrom
Arian8j2:master
Jan 21, 2024
Merged

Use enum named fields in to_string macro #294
Peternator7 merged 5 commits intoPeternator7:masterfrom
Arian8j2:master

Conversation

@Arian8j2
Copy link
Contributor

@Arian8j2 Arian8j2 commented Sep 6, 2023 •
edited
Loading

Not much, Just see the example:

#[derive(strum_macros::Display)]
enum Color {
    #[strum(to_string = "saturation is {sat}")]
    Purple { sat: usize },
}

assert_eq!(
    String::from("saturation is 10"),
    (Color::Purple { sat: 10 }).to_string().as_ref()
);

@Arian8j2
Copy link
Contributor Author

Arian8j2 commented Sep 6, 2023

Seems like environments in CI that failed uses old version of rustc and doesn't support format like these:

let name = "alex";
format!("my name is {name}")

and formats like these are used internally to implement this pr, but i think explicitly assigning each variable like these:

let name = "alex";
format!("my name is {name}", name = name)

will make this possible in older rustc versions, I will try to implement this tomorrow.

@Arian8j2 Arian8j2 changed the title Use named variables in to_string macro Use enum named fields in to_string macro Sep 7, 2023
@Arian8j2 Arian8j2 force-pushed the master branch 2 times, most recently from 4e4ed04 to ead1654 Compare September 8, 2023 07:17
@supleed2 supleed2 mentioned this pull request Oct 12, 2023
Closed
1 task
@Peternator7 Peternator7 merged commit 73f5db2 into Peternator7:master Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Arian8j2 @Peternator7