-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add linux sll support #99
Add linux sll support #99
Conversation
…inst it so clippy does not complain about overlaps
…ors, tests and utilities
Wow thanks for that PR. I will have a deep look into it tomorrow, but so far it looks incredible. |
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.
I don't know what to say, other than this is excellent work. Thanks a lot.
I would merge your PR and move the missing topics you mentioned in the PR description into a ticket. If you find time to implement them a new PR would be very welcome.
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.
Minor fixes for non-std support
I will fix the errors that my no-std fix introduced in the evening. |
Ok perfect, sounds good! When I finish other work I have I'll to look into to finish implementing the rest. |
That's an amazing addition. Thanks for the work @RabadanDotDev and @JulianSchmid for reviewing ❤️ |
Work mentioned in #97. I implemented the part of the public interface I was most interested in (SlicedPacket::from_linux_sll) and all the additions/extensions to the rest of the code it needed. There are some parts missing, but if you are interested in merging, I can work on them when I get the time.
Specifically, some points missing are: