Skip to content

[stdlib] String.Index: Add _description & _debugDescription #61548

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

Merged
merged 1 commit into from
Oct 14, 2022

Conversation

lorentey
Copy link
Member

While we’re vacillating on how best to add CustomStringConvertible and CustomDebugStringConvertible conformances to String.Index, add the prospective implementations as underscored-but-public members, to make life a little more bearable for people who need to debug string index operations.

While we’re vacillating on how best to add CustomStringConvertible
and CustomDebugStringConvertible conformances to String.Index, add
the prospective implementations as underscored-but-public members,
to make life a little more bearable for people who need to debug
string index operations.
@lorentey lorentey requested a review from Azoy October 11, 2022 22:04
Copy link
Contributor

@Azoy Azoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes please 👏

@lorentey
Copy link
Member Author

@swift-ci test

@lorentey
Copy link
Member Author

@swift-ci test macOS platform

@lorentey
Copy link
Member Author

@swift-ci smoke test macOS platform

@Azoy
Copy link
Contributor

Azoy commented Oct 13, 2022

@swift-ci please smoke test macOS

@Azoy
Copy link
Contributor

Azoy commented Oct 13, 2022

@swift-ci smoke test macOS platform

@Azoy
Copy link
Contributor

Azoy commented Oct 13, 2022

@swift-ci please test macOS

@lorentey lorentey merged commit 8240f0b into swiftlang:main Oct 14, 2022
kastiglione added a commit to swiftlang/llvm-project that referenced this pull request Oct 28, 2022
Implement a type summary for Swift's `String.Index`.

The summary string follows the following:
1. Original proposal: https://forums.swift.org/t/improving-string-index-s-printed-descriptions/57027
2. Proposed implementation: swiftlang/swift#58479
3. Temporary(ish) near-`CustomStringConvertible` implementation: swiftlang/swift#61548

The associated test cases are taken from the test cases in swiftlang/swift#58479.

rdar://99211823
kastiglione added a commit to swiftlang/llvm-project that referenced this pull request Nov 2, 2022
Implement a type summary for Swift's `String.Index`.

The summary string follows the following:
1. Original proposal: https://forums.swift.org/t/improving-string-index-s-printed-descriptions/57027
2. Proposed implementation: swiftlang/swift#58479
3. Temporary(ish) near-`CustomStringConvertible` implementation: swiftlang/swift#61548

The associated test cases are taken from the test cases in swiftlang/swift#58479.

rdar://99211823
(cherry picked from commit c7146a3)
kastiglione added a commit to swiftlang/llvm-project that referenced this pull request Feb 9, 2023
Implement a type summary for Swift's `String.Index`.

The summary string follows the following:
1. Original proposal: https://forums.swift.org/t/improving-string-index-s-printed-descriptions/57027
2. Proposed implementation: swiftlang/swift#58479
3. Temporary(ish) near-`CustomStringConvertible` implementation: swiftlang/swift#61548

The associated test cases are taken from the test cases in swiftlang/swift#58479.

rdar://99211823
(cherry picked from commit c7146a3)
kastiglione added a commit to swiftlang/llvm-project that referenced this pull request Feb 9, 2023
Implement a type summary for Swift's `String.Index`.

The summary string follows the following:
1. Original proposal: https://forums.swift.org/t/improving-string-index-s-printed-descriptions/57027
2. Proposed implementation: swiftlang/swift#58479
3. Temporary(ish) near-`CustomStringConvertible` implementation: swiftlang/swift#61548

The associated test cases are taken from the test cases in swiftlang/swift#58479.

rdar://99211823
(cherry picked from commit c7146a3)
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.

2 participants