Skip to content

[stdlib] De-gyb Arrays.swift.gyb #16230

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 8 commits into from
May 30, 2018
Merged

[stdlib] De-gyb Arrays.swift.gyb #16230

merged 8 commits into from
May 30, 2018

Conversation

lancep
Copy link
Contributor

@lancep lancep commented Apr 28, 2018

Split up Array, ArraySlice and ContiguousArray into one file each, with an ArrayShared file for common stuff.

@lancep lancep changed the title De-gyb Array [stdlib] De-gyb Arrays.swift.gyb Apr 28, 2018
@lancep
Copy link
Contributor Author

lancep commented Apr 28, 2018

@swift-ci please test

Copy link
Contributor

@gottesmm gottesmm left a comment

Choose a reason for hiding this comment

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

+1 on less gyb! Found some nits.

@@ -0,0 +1,375 @@
//===--- Arrays.swift.gyb -------------------------------------*- swift -*-===//
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong file name.

@@ -0,0 +1,407 @@
// ###sourceLocation(file: "/Users/lanceparker/Development/swift-source/swift/stdlib/public/core/Arrays.swift.gyb", line: 1)
Copy link
Contributor

Choose a reason for hiding this comment

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

Bad title and ### source location. You should probably grep for these.

@gottesmm
Copy link
Contributor

(sorry I just meant to leave comments, not approve/disapprove)

@lancep
Copy link
Contributor Author

lancep commented Apr 28, 2018

@swift-ci please test

@lancep
Copy link
Contributor Author

lancep commented May 30, 2018

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 637c0ca

@lancep
Copy link
Contributor Author

lancep commented May 30, 2018

@swift-ci please test

@lancep lancep merged commit fd808f3 into swiftlang:master May 30, 2018
@lancep lancep deleted the de-gyb_array branch May 30, 2018 21:11
kitasuke pushed a commit to kitasuke/swift that referenced this pull request Jun 9, 2018
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