medium-ip does not compile complaining of needing EthernetAddress#703
Closed
gopakumarce wants to merge 4 commits intosmoltcp-rs:gat-devicefrom
gopakumarce:gopa-gat-device
Closed
medium-ip does not compile complaining of needing EthernetAddress#703gopakumarce wants to merge 4 commits intosmoltcp-rs:gat-devicefrom gopakumarce:gopa-gat-device
gopakumarce wants to merge 4 commits intosmoltcp-rs:gat-devicefrom
gopakumarce:gopa-gat-device
Conversation
The current `'a` lifetime in the `Device` trait is essentially a workaround for lack of GATs. I'm just experimenting how this would look like, it'll have to wait until GATs are stable to go in. The main benefit is structs implementing `Device` can now borrow stuff. This wasn't possible before because the `for<'d> T: Device<'d>` bounds would essentially imply `T: 'static`.
Member
|
can you add the feature combination that previously failed here so that it doesn't break again in the future? Thanks! |
Author
|
@Dirbaio thx, I have added the combination to the workflow |
Author
|
Also @Dirbaio now that rust 1.65 is released, would you be merging this to main and releasing an official version with this change ? |
Dirbaio
force-pushed
the
gat-device
branch
3 times, most recently
from
b92fc8f to
13cc7f8
Compare
Member
|
It seems you've accidentally rebased on top of an old version of #572, could you remove the unrelated commits? |
Member
|
oh, the issue was you opened this against the GAT branch. Please open all PRs against master. Rebased it for you at #706 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The structure in question seems to be used only for ipv4 fragmentation, hence marking it the same