Skip to content

FALSE is not 1u8 but 0u8, of cource. #8137

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

Closed
wants to merge 1 commit into from
Closed

FALSE is not 1u8 but 0u8, of cource. #8137

wants to merge 1 commit into from

Conversation

omasanori
Copy link
Contributor

Obvious mistake, but there is no failure because no code uses FALSE.

@bblum
Copy link
Contributor

bblum commented Jul 30, 2013

my goodness :P

@thestinger
Copy link
Contributor

Why do we have these TRUE and FALSE constants at all? It could just be treating those as bool in the API if they're really a u8 with the value 0 or 1.

bors added a commit that referenced this pull request Jul 30, 2013
Obvious mistake, but there is no failure because no code uses FALSE.
@bors bors closed this Jul 31, 2013
@bstrie
Copy link
Contributor

bstrie commented Jul 31, 2013

@thestinger looks like it's for comparing against the output of C functions?

flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 30, 2021
Fix commits and formatting of CHANGELOG.md

r? `@Manishearth`

Follow up to rust-lang#8136

I think the beta commit update didn't take the backport we've done into account. I fixed the commit ranges. And while I was at it, I also applied my usual formatting to the changelog entries.

changelog: none
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.

5 participants