Skip to content

Fix Issue #8: Remove redundant trait definitions for Trait #9

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

Closed
wants to merge 1 commit into from

Conversation

velechva
Copy link

@velechva velechva commented Oct 18, 2023

Fix redundant Trait impl's that are causing issue #8

see: #8

@velechva velechva changed the title Remove redundant trait definitions for Trait Fix Issue #8: Remove redundant trait definitions for Trait Oct 18, 2023
@rlovell3
Copy link

Velechva, if this doesn't get merged into the master, I believe we all will need to start cloning your branch. I can't believe nobody is working on this other than you. Gives me a chill.....

@rlovell3
Copy link

I've looked at all the warnings (76? Really?), and even tried to implement the compiler's recommendations about using either dyn()... or unsafe impl<T ...> Trait for T { }, and in all cases, any changes quickly convert into errors. I think this must be the hidden can of Rust worms that nobody wants to address. Anybody looking at this should clone the repository and just try compiling it yourself. This is like the weird uncle you dread coming over for holiday dinners.

@velechva velechva closed this by deleting the head repository Mar 5, 2025
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.

3 participants