Skip to content

Tracking issue for {IpAddr,SockAddr}::ip_v{4,6} methods #36949

Description

@achanda

These methods are proposed:
IpAddr::is_ipv4()
SockAddr::is_ipv4()
These return true if the underlying IP address is an IPv4 address.

IpAddr::is_ipv6()
SockAddr::is_ipv6()
These return true if the underlying IP address is an IPv6 address.

#36707
#36762

Activity

  1. added
    T-libs-api[DEPRECATED; DO NOT USE]
    B-unstableBlocker: Implemented in the nightly compiler and unstable.
    on Oct 4, 2016
  2. achanda commented on Dec 13, 2016

    @achanda
    ContributorAuthor

    Can I propose stabilising these methods?

  3. alexcrichton commented on Dec 13, 2016

    @alexcrichton
    Member

    Sounds reasonable to me!

    @rfcbot fcp merge

  4. rfcbot commented on Dec 13, 2016

    @rfcbot

    Team member @alexcrichton has proposed to merge this. The next step is review by the rest of the tagged teams:

    No concerns currently listed.

    Once these reviewers reach consensus, this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

    See this document for info about what commands tagged team members can give me.

  5. achanda commented on Dec 21, 2016

    @achanda
    ContributorAuthor

    Shouldn't @rfcbot mark this for FCP now?

  6. aturon commented on Dec 21, 2016

    @aturon
    Contributor

    Yes -- the bot has been buggy lately. Thanks for catching it -- I'll mark it manually.

    🔔 This issue is entering its final comment period for stabilization 🔔

  7. added
    final-comment-periodIn the final comment period and will be merged soon unless new substantive objections are raised.
    on Dec 21, 2016
  8. rfcbot commented on Jan 23, 2017

    @rfcbot

    🔔 This is now entering its final comment period, as per the review above. 🔔

    psst @alexcrichton, I wasn't able to add the final-comment-period label, please do so.

  9. achanda commented on Jan 23, 2017

    @achanda
    ContributorAuthor

    Sorry for bugging on this again, but shouldn't this be accepted now since it has been in FCP for more than a week?

  10. alexcrichton commented on Jan 23, 2017

    @alexcrichton
    Member

    @achanda this is for stabilizing, so we'll have a batch PR that rounds up everything and stabilizes close to the end of the cycle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    B-unstableBlocker: Implemented in the nightly compiler and unstable.T-libs-api[DEPRECATED; DO NOT USE]final-comment-periodIn the final comment period and will be merged soon unless new substantive objections are raised.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions