Skip to content

Move some net2 functionality into libstd#1461

Merged
alexcrichton merged 1 commit intorust-lang:masterfrom
sfackler:net2
Feb 18, 2016
Merged

Move some net2 functionality into libstd#1461
alexcrichton merged 1 commit intorust-lang:masterfrom
sfackler:net2

Conversation

@sfackler
Copy link
Copy Markdown
Member

@sfackler sfackler added the T-libs-api Relevant to the library API team, which will review and decide on the RFC. label Jan 13, 2016
@alexcrichton
Copy link
Copy Markdown
Member

🔔 This RFC is now entering its week-long final comment period 🔔

Please don't everyone comment all at once!

More seriously, though, the libs team is interested specifically in diving into the landing strategy for this in libstd, either:

  • This new functionality lands insta-stable
  • The new functionality lands as an unstable extension trait

My own personal opinion is that we should land these as insta-stable. The APIs have been vetted in net2-rs for quite some time now, and I don't think we'd really gain all that much more use through another round of FCP for the APIs.

Curious what others think though!

@alexcrichton alexcrichton added final-comment-period Will be merged/postponed/closed in ~10 calendar days unless new substational objections are raised. and removed I-nominated labels Feb 11, 2016
@briansmith
Copy link
Copy Markdown

I don't think these should be added via traits. Over time, the standard library would become a ridiculous mess of such traits as new functionality is added, if it were to follow that pattern.

@sfackler
Copy link
Copy Markdown
Member Author

The traits would only exist until the methods were stabilized.

@alexcrichton
Copy link
Copy Markdown
Member

The libs team discussed this during triage yesterday and the conclusion was to merge this RFC adding these methods as insta-stable. Thanks for the RFC @sfackler!

@sfackler
Copy link
Copy Markdown
Member Author

We're going to hold off on the implementation until just after the next release to allow for the longest possible time before these hit a stable release and are locked in forever.

@tshepang
Copy link
Copy Markdown
Member

How about simply not making them insta-stable? That to me sounds better than delaying the landing.

@Centril Centril added A-impls-libstd Standard library implementations related proposals. A-net Proposals relating to networking. labels Nov 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-impls-libstd Standard library implementations related proposals. A-net Proposals relating to networking. final-comment-period Will be merged/postponed/closed in ~10 calendar days unless new substational objections are raised. T-libs-api Relevant to the library API team, which will review and decide on the RFC.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants