Skip to content

Issue #21 additional ipnet types#24

Open
sophacles wants to merge 2 commits intomainfrom
issue_9_additional_ipnet_types
Open

Issue #21 additional ipnet types#24
sophacles wants to merge 2 commits intomainfrom
issue_9_additional_ipnet_types

Conversation

@sophacles
Copy link
Copy Markdown
Owner

@sophacles sophacles commented May 19, 2025

This PR:

Add support for additional ipnet types

This involves a couple of things:

  • support for IpRange types and IpSubnets types. These are not simple
    fromstr implementations so the make_macro! approach didn't work.
    Instead a heck of a lot of proc-macro code was needed to parse and
    validate the args lists.

  • A separate set of tests for nightly vs stable builds in some cases.
    Reasons for this:
    1) compiler output is slightly different between nightly and stable.
    2) there is warning support in nightly and this emits warnings when
    the generated iterators might behave different than expected.

Fixes: #21

Checklist:

  • Tests provided for new functionality
  • Commit messages follow Conventional Commits format
  • Changelog reflects this change
  • Docs/README are updated appropraitely (if applicable)

@sophacles sophacles force-pushed the issue_9_additional_ipnet_types branch 3 times, most recently from 9481a46 to 32c9e74 Compare May 19, 2025 23:58
@sophacles sophacles changed the title Issue 9 additional ipnet types Issue #21 additional ipnet types May 20, 2025
sophacles added 2 commits May 19, 2025 19:10
This involves a couple of things:

* support for IpRange types and IpSubnets types. These are not simple
  fromstr implementations so the make_macro! approach didn't work.
  Instead a heck of a lot of proc-macro code was needed to parse and
  validate the args lists.

* A separate set of tests for nightly vs stable builds in some cases.
  Reasons for this:
  1) compiler output is slightly different between nightly and stable.
  2) there is warning support in nightly and this emits warnings when
     the generated iterators might behave different than expected.

Fixes: #21
@sophacles sophacles force-pushed the issue_9_additional_ipnet_types branch from 32c9e74 to 587a097 Compare May 20, 2025 00:10
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.

Support for ipnet::Ip*Subnet and ipnet::Ip*Range

1 participant