Skip to content
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

GNRC extensions #122

Merged
merged 11 commits into from
Sep 16, 2024
Merged

GNRC extensions #122

merged 11 commits into from
Sep 16, 2024

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Sep 10, 2024

This contains a bunch of GNRC enhancements:

  • It simplifies some internal implementation, enhances the Debug form of IP addresses, and adds interconversion of GNRC IPv6 addresses with core::net::Ipv6Address.
  • It enhances printed output
  • It adds a means of registering to GNRC through callbacks
  • Enhanced config gating precision (not everything that has "gnrc" in its name is only available if MODULE_GNRC is active, eg. pktbuf can be used standalone too)
  • KernelPID gets a conversion Into<NonZero> to ease interoperability with zone identifiers
  • netif snip access is generalized

@chrysn chrysn changed the title gnrc/ipv6: Add conversion with and simplifications through core::net gnrc extensions Sep 11, 2024
All prerequisites are checked at build time, and this is a convenient
simplification for code that works with zone identifiers (which SHOULD
use zero as the special default value according to RFC4007)
@chrysn chrysn changed the title gnrc extensions GNRC extensions Sep 13, 2024
The previous attempt made wrong assumptions about how data is stored in
there, and the new interface is easier to use.
@chrysn chrysn enabled auto-merge September 16, 2024 12:38
@chrysn chrysn merged commit 917d545 into main Sep 16, 2024
79 checks passed
@chrysn chrysn deleted the ip-enhancements branch September 16, 2024 12:49
@Kijewski Kijewski mentioned this pull request Sep 16, 2024
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.

1 participant