Skip to content

Conversation

@Catfish-Man
Copy link
Contributor

@Catfish-Man Catfish-Man commented Nov 4, 2025

Fixes rdar://163960939

…g compared to a native one has the same utf16 count but a different utf8 count
@Catfish-Man Catfish-Man self-assigned this Nov 4, 2025
@Catfish-Man Catfish-Man requested a review from a team as a code owner November 4, 2025 19:44
@Catfish-Man
Copy link
Contributor Author

@swift-ci please smoke test

@parkera parkera requested a review from jmschonfeld November 4, 2025 19:48
@Catfish-Man
Copy link
Contributor Author

@swift-ci please smoke test

@Catfish-Man Catfish-Man enabled auto-merge (squash) November 4, 2025 20:58
@Catfish-Man
Copy link
Contributor Author

@swift-ci please smoke test

@Catfish-Man
Copy link
Contributor Author

@swift-ci please smoke test

@Catfish-Man
Copy link
Contributor Author

None of this code even builds on Windows, so that must be a spurious failure

@Catfish-Man
Copy link
Contributor Author

@swift-ci please smoke test Windows platform

@glessard
Copy link
Contributor

glessard commented Nov 5, 2025

The Windows failure seems unrelated, but also other runs aren't failing in the same place.
(For reference: https://ci-external.swift.org/job/swift-PR-windows/47501/)

@glessard
Copy link
Contributor

glessard commented Nov 5, 2025

@swift-ci please test Windows platform

@shahmishal shahmishal disabled auto-merge November 6, 2025 00:35
@shahmishal shahmishal merged commit 60d09ff into main Nov 6, 2025
2 of 3 checks passed
@shahmishal shahmishal deleted the count-mixup branch November 6, 2025 00:35
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.

6 participants