Skip to content

Don't allow using const fn arguments as "args_required_const" - #61536

Merged
bors merged 2 commits into
rust-lang:masterfrom
oli-obk:args_required_const_in_const_fn
Jun 5, 2019
Merged

Don't allow using const fn arguments as "args_required_const"#61536
bors merged 2 commits into
rust-lang:masterfrom
oli-obk:args_required_const_in_const_fn

Conversation

@oli-obk

Copy link
Copy Markdown
Contributor

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 5, 2019
@oli-obk
oli-obkforce-pushed the args_required_const_in_const_fn branch from a336fdd to dcd46d6CompareJune 5, 2019 07:49
Comment threadsrc/test/ui/consts/const_arg_promotable2.rs
@eddyb

eddyb commented Jun 5, 2019

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Jun 5, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 192c1d0 has been approved by eddyb

@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 Jun 5, 2019
Centril added a commit to Centril/rust that referenced this pull request Jun 5, 2019
…t_fn, r=eddyb
Don't allow using const fn arguments as "args_required_const"
r? @eddyb
@CentrilCentril mentioned this pull request Jun 5, 2019
bors added a commit that referenced this pull request Jun 5, 2019
Rollup of 5 pull requests
Successful merges:
- #61503 (Fix cfg(test) build for x86_64-fortanix-unknown-sgx)
- #61534 (Edit docs of ExitStatus)
- #61536 (Don't allow using const fn arguments as "args_required_const")
- #61538 (Don't use GNU noexec stack note)
- #61546 (azure: Fix some minor issues which have broken our configuration )
Failed merges:
r? @ghost
@bors
bors merged commit 192c1d0 into rust-lang:masterJun 5, 2019
@oli-obk
oli-obk deleted the args_required_const_in_const_fn branch March 16, 2021 12:14
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.

5 participants

@oli-obk@eddyb@bors@Centril@rust-highfive