Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enum Values now derives PartialEq. #37

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

martinfrances107
Copy link

@martinfrances107 martinfrances107 commented Mar 15, 2024

I love this module... It is really useful.

but I am at the stage where I want Field to provide PartialEq,

[ To get this to compile both Rational and SRational and Value need to derive PartialEq. ]

/// A type and values of a TIFF/Exif field.
#[derive(Clone, PartialEq)]
pub enum Value {
    /// Vector of 8-bit unsigned integers.
    Byte(Vec<u8>),
    /// Vector of slices of 8-bit bytes containing 7-bit ASCII characters.
    /// The trailing null characters are not included.  Note that
    /// the 8th bits may present if a non-conforming data is given.
    Ascii(Vec<Vec<u8>>),
    /// Vector of 16-bit unsigned integers.
    Short(Vec<u16>),
    /// Vector of 32-bit unsigned integers.
    Long(Vec<u32>),

...


}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant