Skip to content

[5.1][SE-0240] Ordered Collection Diffing #23197

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 12 commits into from
Mar 12, 2019

Conversation

lorentey
Copy link
Member

@lorentey lorentey commented Mar 9, 2019

This is a cherry-pick of #21845, implementing SE-0240, Ordered Collection Diffing.

numist and others added 12 commits March 8, 2019 21:37
(cherry picked from commit c0c068d)
…k, causing some strange crashes

(cherry picked from commit 9b45fc4)
- Add an underscore to private/internal symbols
- Make access levels explicit, even when they’re implied from context
- Conform to max line length of 80 characters
- Conformances declared on separate extensions that implement them
- Arrange member declarations so that stored properties appear first
- Expand single-letter function and type parameter names where there is an obvious name that’s more descriptive
- RangeReplaceableCollection.fastApplicationEnumeration → CollectionDifference._fastEnumeratedApply

(cherry picked from commit 57f4afb)
@lorentey
Copy link
Member Author

lorentey commented Mar 9, 2019

@swift-ci please test

@lorentey lorentey requested review from numist and airspeedswift March 9, 2019 06:22
@swift-ci
Copy link
Contributor

swift-ci commented Mar 9, 2019

Build failed
Swift Test Linux Platform
Git Sha - 2a46fd0

@lorentey
Copy link
Member Author

lorentey commented Mar 9, 2019

https://ci.swift.org/job/swift-PR-Linux/11642/:

01:37:49 Test Case 'TestXMLDocument.test_xpath' started at 2019-03-09 07:37:49.068
01:37:49 
01:37:49 
01:37:49 0% tests passed, 1 tests failed out of 1
01:37:49 
01:37:49 Total Test time (real) =  75.33 sec
01:37:49 
01:37:49 The following tests FAILED:
01:37:49 	  1 - TestFoundation (SEGFAULT)
01:37:49 Errors while running CTest
01:37:49 FAILED: CMakeFiles/test.util 

This again! But how?

@lorentey
Copy link
Member Author

lorentey commented Mar 9, 2019

@swift-ci please test linux platform

@lorentey lorentey merged commit 8060872 into swiftlang:swift-5.1-branch Mar 12, 2019
@lorentey lorentey deleted the numist/diffing-5.1 branch March 12, 2019 02:50
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