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

Slicing syntax should work for &&[T] #17503

Closed
nrc opened this issue Sep 24, 2014 · 3 comments · Fixed by #19780
Closed

Slicing syntax should work for &&[T] #17503

nrc opened this issue Sep 24, 2014 · 3 comments · Fixed by #19780
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.

Comments

@nrc
Copy link
Member

nrc commented Sep 24, 2014

Probably. At least (*foo)[..42] is pretty ugly

Either we implement Slice for &[T] (currently only implemented for [T]) or we do the same autoref/deref for slices as we do for method call.

@sfackler
Copy link
Member

sfackler commented Oct 5, 2014

It should also work through Deref.

@japaric
Copy link
Member

japaric commented Nov 7, 2014

This seems to be working:

#![feature(slicing_syntax)]

fn main() {
    let s: &[int] = &[0, 1, 2, 3, 4];
    let ss: &&[int] = &s;
    let sss: &&&[int] = &ss;

    println!("{}", s[..3]);
    println!("{}", ss[3..]);
    println!("{}", sss[2..4]);
}

As of:

rustc 0.13.0-dev (e84e7a00d 2014-11-06 08:06:50 +0000)

@alexcrichton
Copy link
Member

Flagging as needtest.

@alexcrichton alexcrichton added the E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added. label Nov 7, 2014
bors added a commit that referenced this issue Dec 18, 2014
Closes #5988.
Closes #10176.
Closes #10456.
Closes #12744.
Closes #13264.
Closes #13324.
Closes #14182.
Closes #15381.
Closes #15444.
Closes #15480.
Closes #15756.
Closes #16822.
Closes #16966.
Closes #17351.
Closes #17503.
Closes #17545.
Closes #17771.
Closes #17816.
Closes #17897.
Closes #17905.
Closes #18188.
Closes #18232.
Closes #18345.
Closes #18389.
Closes #18400.
Closes #18502.
Closes #18611.
Closes #18783.
Closes #19009.
Closes #19081.
Closes #19098.
Closes #19127.
Closes #19135.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants