Skip to content

Fixed the fmt conflict caused by using use std::fmt::Debug#322

Merged
Peternator7 merged 1 commit intoPeternator7:masterfrom
disoul:fix/multiple-fmt
Jan 7, 2024
Merged

Fixed the fmt conflict caused by using use std::fmt::Debug#322
Peternator7 merged 1 commit intoPeternator7:masterfrom
disoul:fix/multiple-fmt

Conversation

@disoul
Copy link
Contributor

@disoul disoul commented Jan 2, 2024 •
edited
Loading 

use std::fmt::Debug; // <--- add this line, for example some people might wish to custom implement the Debug trait

#[derive(Eq, PartialEq, EnumString, Display, EnumCount, EnumDiscriminants, EnumIs)]
pub enum Color {
    /// Docs on red
    #[strum(to_string = "RedRed")]
    Red,
    #[strum(serialize = "b", to_string = "blue")]
    Blue { hue: usize },
    #[strum(serialize = "y", serialize = "yellow")]
    Yellow,
    #[strum(disabled)]
    Green(String),
}

impl Debug for Color {
    ...
}

This will cause the following error.

error[E0034]: multiple applicable items in scope
 --> strum_tests/src/lib.rs:5:44
  |
5 | #[derive(Debug, Eq, PartialEq, EnumString, Display, EnumCount, EnumDiscriminants, EnumIs)]
  |                                            ^^^^^^^ multiple `fmt` found
  |
  = note: candidate #1 is defined in an impl of the trait `Debug` for the type `str`
  = note: candidate #2 is defined in an impl of the trait `std::fmt::Display` for the type `str`
  = note: this error originates in the derive macro `Display` (in Nightly builds, run with -Z macro-backtrace for more info)

@Peternator7
Copy link
Owner

Peternator7 commented Jan 7, 2024

Good catch; thanks for the fix

@Peternator7 Peternator7 merged commit e34e66c into Peternator7:master Jan 7, 2024
mich181189 pushed a commit to mich181189/strum that referenced this pull request Jan 17, 2024
@disoul @mich181189
Fixed the fmt conflict caused by using use std::fmt::Debug (Peter…
71786c2 
…nator7#322)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Peternator7 Peternator7 Peternator7 approved these changes

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

@disoul @Peternator7