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

Convert all uses of vec::slice to vec::view Issue #3869 #4969

Closed
wants to merge 2 commits into from
Closed

Convert all uses of vec::slice to vec::view Issue #3869 #4969

wants to merge 2 commits into from

Conversation

nickdesaulniers
Copy link

Issue #3869
review? @nikomatsakis

Convert all uses of vec::slice to vec::view Issue #3869
Rename const_view to const_slice
Renamed mut_view to mut_slice
Fix windows build error. buf is borrowed by the call to
as_mut_buf() and so we must invoke slice() outside of that
call.

Nick Desaulniers and others added 2 commits February 14, 2013 16:14
Rename const_view to const_slice
Renamed mut_view to mut_slice
`as_mut_buf()` and so we must invoke `slice()` outside of that
call.
bors added a commit that referenced this pull request Feb 15, 2013
Issue #3869
review? @nikomatsakis 

Convert all uses of vec::slice to vec::view Issue #3869
Rename const_view to const_slice
Renamed mut_view to mut_slice
Fix windows build error.  `buf` is borrowed by the call to
`as_mut_buf()` and so we must invoke `slice()` outside of that
call.
@bors bors closed this Feb 16, 2013
bors added a commit to rust-lang-ci/rust that referenced this pull request May 2, 2020
…consts-fp, r=phansch

Prevent `replace_consts` lint within match patterns

Currently `replace_consts` lint applies within match patterns but the suggestion is incorrect as function calls are disallowed in them. To fix this we prevent the lint from firing within patterns.

Fixes rust-lang#4969

changelog: Fix false positive in `replace_consts` lint
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.

3 participants