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

add android dummy functions for mman related #7395

Merged
merged 1 commit into from
Jun 27, 2013
Merged

add android dummy functions for mman related #7395

merged 1 commit into from
Jun 27, 2013

Conversation

yichoi
Copy link
Contributor

@yichoi yichoi commented Jun 26, 2013

add android dummy functions which does not exist in boinic.

after #7257, some mman related functions are needed for android.

@brson
Copy link
Contributor

brson commented Jun 26, 2013

Sorry about the breakage.

bors added a commit that referenced this pull request Jun 27, 2013
add android dummy functions which does not exist in boinic.

after #7257, some mman related functions are needed for android.
@bors bors closed this Jun 27, 2013
@bors bors merged commit 6a77273 into rust-lang:master Jun 27, 2013
@yichoi yichoi deleted the android_dummy branch June 27, 2013 13:42
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jul 1, 2021
Fix invocation of `zst_offset` lint

The `zst_offset` lint was broken by a refactoring regression in 2108387. In the invocation of the `zst_offset` check [here](rust-lang/rust-clippy@2108387#diff-7f73f6fe28c04b654223c09c42fe02937d2351fc58a91d21ab812aaf6f9b185dR1927), we shadow the already-destructured receiver `recv`, and accidentally pass the first argument of the method as if it were the receiver.

This was not caught because the UI test expectation was never correct (a bad cast obscured the actual test result).

This PR:
- Fixes the existing test expectation
- Tests both `const` and `mut` raw pointers
- Passes the actual receiver to the lint's implementation

Fixes rust-lang#7395.

changelog: Fix [`zst_offset`] invocation and test
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