Skip to content

Remove the no_debug feature - #69667

Merged
bors merged 3 commits into
rust-lang:masterfrom
JohnTitor:no-debug
Mar 8, 2020
Merged

Remove the no_debug feature#69667
bors merged 3 commits into
rust-lang:masterfrom
JohnTitor:no-debug

Conversation

@JohnTitor

Copy link
Copy Markdown
Member

@JohnTitorJohnTitor added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 3, 2020
Comment threadsrc/librustc/middle/codegen_fn_attrs.rs Outdated

@nikomatsakisnikomatsakis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

r=me but I agree we should remove the constant that is no longer used, rather than leave it there with a comment that it is no longer used (unless there is some reason to keep it)

@JohnTitor

Copy link
Copy Markdown
MemberAuthor

Removed the const as well.
@bors r=nikomatsakis

@bors

bors commented Mar 6, 2020

Copy link
Copy Markdown
Collaborator

📌 Commit 18080e6 has been approved by nikomatsakis

@bors

bors commented Mar 6, 2020

Copy link
Copy Markdown
Collaborator

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 6, 2020
@CentrilCentril mentioned this pull request Mar 7, 2020
bors added a commit that referenced this pull request Mar 8, 2020
Rollup of 8 pull requests
Successful merges:
- #69667 (Remove the `no_debug` feature)
- #69687 (resolve, inconsistent binding mode: tweak wording)
- #69708 (On mismatched delimiters, only point at empty blocks that are in the same line)
- #69765 (reduce test size for Miri)
- #69773 (fix various typos)
- #69787 (mir::Local is Copy we can pass it by value in these cases)
- #69794 (Add `Layout::dangling()` to return a well-aligned `NonNull<u8>`)
- #69797 (Correct version that relaxed orphan rules)
Failed merges:
r? @ghost
@bors
bors merged commit 614cd8d into rust-lang:masterMar 8, 2020
@JohnTitor
JohnTitor deleted the no-debug branch March 8, 2020 09:20
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@JohnTitor@bors@nikomatsakis@bjorn3