Skip to content

Conversation

Catfish-Man
Copy link
Contributor

No description provided.

@Catfish-Man Catfish-Man self-assigned this May 17, 2019
@Catfish-Man Catfish-Man requested a review from milseman May 17, 2019 22:06
let kCFStringEncodingASCII: UInt32 = 0x0600

private func testForeignContiguous(slowString: NSSlowString, string: String) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is just the bit below pulled out so I can reuse it for the second set of data

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - fb2257064a88be1da61ffa2f2ebcf59c5ad00e79

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - fb2257064a88be1da61ffa2f2ebcf59c5ad00e79

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - fb2257064a88be1da61ffa2f2ebcf59c5ad00e79

@Catfish-Man
Copy link
Contributor Author

"LLDB :: SwiftREPL/BreakpointSimple.test" is absolutely not caused by adding unrelated tests

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test osx platform

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 7561f95

@Catfish-Man
Copy link
Contributor Author

This time Interpreter/switch_resilience.swift :/ still not caused by this

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 7561f95

@Catfish-Man
Copy link
Contributor Author

Test run hung >.<

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 7561f95

@Catfish-Man
Copy link
Contributor Author

@swift-ci please test linux platform

Copy link
Member

@milseman milseman left a comment

Choose a reason for hiding this comment

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

LGTM


// They become fast when mutated
slowString.makeNative()
expectTrue(slowString.isFastUTF8)
Copy link
Member

Choose a reason for hiding this comment

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

Oh right, this test predates contiguous strings proposal. You can go this route, or update the test to use isContiguousUTF8, makeContiguousUTF8(), and withUTF8().

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Noted. I'm probably going to leave it as unchanged as possible for now.

@Catfish-Man Catfish-Man merged commit 2dcd822 into swiftlang:master May 22, 2019
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.

3 participants