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.

Sign up for GitHub

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

Jump to bottom

[5.1] [stdlib] Make unsafe array initializer public #23519

Merged
merged 1 commit into from
Mar 23, 2019
Merged

[5.1] [stdlib] Make unsafe array initializer public #23519

merged 1 commit into from
Mar 23, 2019

Conversation

natecook1000
Copy link
Member

This is a cherry pick of #23134.

  • Explanation: This implements SE-0245. The public versions of this initializer call
    into the existing, underscored version, which avoids the need for availability constraints.
  • Scope: An assessment of the impact/importance of the change. For example, is the change a source-breaking language change, etc.
  • SR Issue: SR-3087
  • Risk: This is an additive change, with no modification to the existing ABI.
  • Testing: This includes validation tests to test the different postconditions and throwing semantics.
  • Reviewers: @jrose-apple, @Catfish-Man

@natecook1000
[stdlib] Make unsafe array initializer public (swiftlang#23134)
704a60f 
[stdlib] Make unsafe array initializer public

This implements SE-0245. The public versions of this initializer call
into the existing, underscored version, which avoids the need for
availability constraints.
@natecook1000 natecook1000 changed the title [stdlib] Make unsafe array initializer public (#23134) [5.1] [stdlib] Make unsafe array initializer public Mar 23, 2019
@natecook1000
Copy link
Member Author

@swift-ci Please test

@airspeedswift airspeedswift merged commit b6f7e86 into swiftlang:swift-5.1-branch Mar 23, 2019
@natecook1000 natecook1000 deleted the 5.1-array-unsafe-init branch August 29, 2019 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@airspeedswift airspeedswift

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@natecook1000 @airspeedswift