-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
feat: implement PartialEq
for SimpleExpr
and its related types
#620
Conversation
Thank you for the effort. pub type DynIden = SeaRc<Iden>;
#[drive(Clone, ... many)]
#[repr(transparent)]
pub struct SeaRc<I>(i: RcOrArc<dyn I>);
// we only implement it for Iden
impl<I: Iden> SeaRc<I> {
pub fn new(i: I) -> DynIden {
Self(RcOrArc::new(i))
}
} That way we don't have to maintain two types! Although it is a breaking change, I hope that the typical user would not need to change a lot of things, if they are already using |
It works! But with a bit more extra efforts :P Let me think about it and revisit this tmr morning |
5aeece0
to
590387d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@billy1624 thank you! I have one question.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Phenomenal!
🎉 Released In 0.29.0 🎉Your participation is what makes us unique; your adoption is what drives us forward. |
🎉 Released In 0.29.1 🎉Thank you everyone for the contribution! |
PR Info
New Features
PartialEq
forSimpleExpr
and on its related typesBreaking Changes
SeaRc
intoRcOrArc
SeaRc
which resemble the API ofSeaRC
as it used to be