-
Notifications
You must be signed in to change notification settings - Fork 4
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
Possible future improvements #25
Comments
This comment was marked as outdated.
This comment was marked as outdated.
|
We could use this lint: |
rust-lang/rust#106418 is in it's FCP with a disposition to merge! |
Oh, that would be big. We could have a feature that enables the current unsafe work around and therefor keeps the msrv down? |
I would probably do it the other way around, add a crate feature that enables the new behavior but raises the MSRV. In any case, we could remove the |
These are code improvements or features that could be added in the future if demand is there:
nightly
crate feature and remove parts of thesafe
crate feature implementation.PartialOrd
andOrd
codegen, done in Improve code generation ofOrd
andPartialOrd
#38.#[derive_where(Clone; T = u8)]
, which will generateimpl Clone for Test<u8>
.derive
) instead or bad code quality (empty bracers). Currently Rust doesn't have acompile_warn
.quote
and build any Rust code purely with syn structures instead.match
and access fields directly when not dealing with an enum. This is mainly modeled after std'sderive
, which was probably done to make it easy to implement as it makes code generation between enums and structs very similar, but after many refactors, there is no real advantage anymore.Ident
s, some implementation don't need them, but we don't want to generate them ad-hoc as they do some allocations. The main concern here is adding another dependency.The text was updated successfully, but these errors were encountered: