Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upImplement Unsized Rvalues #51131
Conversation
rust-highfive
assigned
nikomatsakis
May 28, 2018
This comment has been minimized.
This comment has been minimized.
|
(rust_highfive has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
rust-highfive
added
the
S-waiting-on-review
label
May 28, 2018
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
qnighy
force-pushed the
qnighy:unsized-locals
branch
2 times, most recently
from
f6f4804
to
4dc11f5
May 29, 2018
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
qnighy
force-pushed the
qnighy:unsized-locals
branch
from
4dc11f5
to
1b782d5
May 29, 2018
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
This comment has been minimized.
This comment has been minimized.
|
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
This comment has been minimized.
This comment has been minimized.
|
|
qnighy
force-pushed the
qnighy:unsized-locals
branch
2 times, most recently
from
2c27df5
to
55a1e92
Jun 1, 2018
qnighy
changed the title
[WIP] Implement Unsized Rvalues
Implement Unsized Rvalues
Jun 2, 2018
This comment has been minimized.
This comment has been minimized.
|
I found several missing sizedness checking, namely
As adding As these plans are going to be large (for me, at least), I want to have this PR merged for now and prepare different PRs. |
This comment has been minimized.
This comment has been minimized.
|
|
qnighy
force-pushed the
qnighy:unsized-locals
branch
from
55a1e92
to
be84642
Jun 4, 2018
This comment has been minimized.
This comment has been minimized.
|
|
qnighy
force-pushed the
qnighy:unsized-locals
branch
from
be84642
to
e0d31c2
Jun 5, 2018
This comment has been minimized.
This comment has been minimized.
rust-highfive
assigned
aturon
and unassigned
nikomatsakis
Jun 9, 2018
This comment has been minimized.
This comment has been minimized.
|
This PR seems to be missed, perhaps because it was initially |
This comment has been minimized.
This comment has been minimized.
|
Ping from triage @aturon! This PR needs your review. |
This comment has been minimized.
This comment has been minimized.
|
cc @rust-lang/compiler I'm not a suitable reviewer for this, can someone take it? |
This comment has been minimized.
This comment has been minimized.
|
Ah, r? @nikomatsakis based on a previous comment |
rust-highfive
assigned
nikomatsakis
and unassigned
aturon
Jun 22, 2018
bors
added
the
S-waiting-on-author
label
Aug 18, 2018
qnighy
added some commits
May 28, 2018
qnighy
force-pushed the
qnighy:unsized-locals
branch
from
f808c1f
to
c488d59
Aug 18, 2018
This comment has been minimized.
This comment has been minimized.
|
Rebased. r? @eddyb |
rust-highfive
assigned
eddyb
and unassigned
nikomatsakis
Aug 19, 2018
This comment has been minimized.
This comment has been minimized.
|
@bors r+ |
This comment has been minimized.
This comment has been minimized.
|
|
bors
added
S-waiting-on-bors
and removed
S-waiting-on-author
labels
Aug 19, 2018
This comment has been minimized.
This comment has been minimized.
bors
added a commit
that referenced
this pull request
Aug 19, 2018
This comment has been minimized.
This comment has been minimized.
|
|
qnighy commentedMay 28, 2018
•
edited
This PR is the first step to implement RFC1909: unsized rvalues (#48055).
Implemented
Sizedis removed for arguments and local bindings. (under#![feature(unsized_locals)])Not implemented in this PR
Sizedchecks:fn method(self)(ICEs now)[e; n]wherenisn't constCurrent status
__rust_probestack(rust-lang-nursery/compiler-builtins#244)#![feature(unsized_locals)]Forbidwill be another PRA(unsized-expr)Loosen object-safety rules (will be another PR)ImplementBox<FnOnce>(will be another PR)Reduce temporaries (will be another PR)