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
add a missing subscript(bounds:)
implementation
#1893
Conversation
- an older version of the standard library supplied a bogus implementation by error. Calling *that* would have been a crashing bug.
Can one of the admins verify this patch? |
11 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the improvement!
@swift-nio-bot add to allowlist |
An improvement that is deprecated-on-arrival. 🤷🏽 |
This also triggered a failure in the compatibility suite (sorry for failing to notice). |
I believe that the Swift team are in charge of updating that. |
Motivation: This is a backport of apple#1893 for SwiftNIO1. Modifications: Add an implementation of subscript(_ bounds:) to StaticString. Result: * StaticString can be sliced without crashing, though in O(n) time. * The package should build with the July 8 2021 snapshot of Swift.
An older version of the standard library supplied a bogus implementation by error.
Calling that would have been a crashing bug.
Motivation:
Fix #1892,
which was triggered by apple/swift#38161,
which is a fix for https://bugs.swift.org/browse/SR-14848.
Modifications:
Add the missing
subscript(bounds:)
implementation. It should always have been there.Result:
Package should build with the July 8 2021 snapshot of Swift.