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.

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

[lldb] Add type summary for String.Index #5515

Merged

Conversation

kastiglione
Copy link

@kastiglione kastiglione commented Oct 27, 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: [DRAFT][stdlib] String.Index: Add custom printing swift#58479
  3. Temporary(ish) near-CustomStringConvertible implementation: [stdlib] String.Index: Add _description & _debugDescription swift#61548

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

rdar:https://99211823

@kastiglione
Copy link
Author

@swift-ci test

@kastiglione
Copy link
Author

cc @lorentey in case you have any thoughts

@kastiglione
Copy link
Author

@swift-ci test

Copy link

@augusto2112 augusto2112 left a comment

Choose a reason for hiding this comment

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

Great change! Two small nits.

@kastiglione
Copy link
Author

@swift-ci test

@kastiglione
Copy link
Author

@swift-ci test

@kastiglione kastiglione merged commit c7146a3 into stable/20220421 Oct 28, 2022
@kastiglione kastiglione deleted the dl/lldb-add-type-summary-for-string.index branch October 28, 2022 17:46
kastiglione added a commit 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:https://99211823
(cherry picked from commit c7146a3)
kastiglione added a commit 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:https://99211823
(cherry picked from commit c7146a3)
kastiglione added a commit 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:https://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.

None yet

4 participants