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

Syntax breaking batch #36857

Merged
merged 12 commits into from
Oct 1, 2016
Merged

Syntax breaking batch #36857

merged 12 commits into from
Oct 1, 2016

Conversation

Manishearth
Copy link
Member

No description provided.

CensoredUsername and others added 9 commits September 17, 2016 01:17
I am using `ThinAttributes` rather than a vector for attributes
attached to generics, since I expect almost all lifetime and types
parameters to not carry any attributes.
These weren't flagged by the lint because they were still technically
created somewhere... if you created them before, somehow.
This applies the HIR changes from the previous commits to the AST, and
is thus a syntax-[breaking-change]

Renames `PatKind::Vec` to `PatKind::Slice`, since these are called slice
patterns, not vec patterns. Renames `TyKind::Vec`, which represents the
type `[T]`, to `TyKind::Slice`. Renames `TyKind::FixedLengthVec` to
`TyKind::Array`.
@Manishearth
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Sep 30, 2016

⌛ Trying commit 173ebc3 with merge 7be4351...

@bors
Copy link
Contributor

bors commented Sep 30, 2016

💥 Test timed out

@pnkfelix
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Sep 30, 2016

⌛ Trying commit 173ebc3 with merge 14b30b0...

bors added a commit that referenced this pull request Sep 30, 2016
@bors
Copy link
Contributor

bors commented Oct 1, 2016

💥 Test timed out

@Manishearth
Copy link
Member Author

@bors try- r+ retry

@bors
Copy link
Contributor

bors commented Oct 1, 2016

📌 Commit 173ebc3 has been approved by Manishearth

@Manishearth
Copy link
Member Author

@bors p=10

@bors
Copy link
Contributor

bors commented Oct 1, 2016

⌛ Testing commit 173ebc3 with merge 5d3ae8a...

…, r=Manishearth

 This commit makes the return type of AstBuilder.stmt_let_typed match the return type of other AstBuilder.stmt* functions. This avoids unnecessary boxing/unboxing whenever Stmt's are stored in a Vec, which is the default use case.nnThis is a potentially plugin breaking change.
…orite-data-structure, r=pnkfelix

 Contains a syntax-[breaking-change] as a separate commit (cc rust-lang#31645).nnAlso renames slice patterns from `PatKind::Vec` to `PatKind::Slice`.
@Manishearth
Copy link
Member Author

@bors r+ force

commit messages need r=foo

@bors
Copy link
Contributor

bors commented Oct 1, 2016

📌 Commit 406fe7e has been approved by Manishearth

@Manishearth
Copy link
Member Author

@bors force

@bors
Copy link
Contributor

bors commented Oct 1, 2016

⌛ Testing commit 406fe7e with merge df9fa1a...

bors added a commit that referenced this pull request Oct 1, 2016
@bors bors merged commit 406fe7e into rust-lang:master Oct 1, 2016
@Manishearth Manishearth deleted the syntax-rollup branch October 2, 2016 17:15
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.

5 participants