Skip to content

Conversation

@alexcrichton
Copy link
Member

  • Stop using hardcoded numbers that have to all get updated when something changes (inevitable errors and rebase conflicts) as well as removes some unneeded -Z options (obsoleted over time).
  • Remove std::rt::borrowck

@alexcrichton
Copy link
Member Author

Updated with more cleanups

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it worth capitalising these statics while you're modifying them anyway?

@alexcrichton
Copy link
Member Author

Sounds like a plan to me, I'll update these soon.

@alexcrichton
Copy link
Member Author

Sadly the lang items weren't quite as unused as I thought they were, otherwise rebased to capitalize debugging opts and use a u64 instead of a uint.

Instead use a macro and generate them!
* borrowck_note_pure - unused
* borrowck_note_loan - unused
* no_debug_borrows - unused
* lint_llvm - equivalent to -Z no-prepopulate-passes + --llvm-passes lint
This hasn't been in use since `@mut` was removed
bors added a commit that referenced this pull request Jan 21, 2014
* Stop using hardcoded numbers that have to all get updated when something changes (inevitable errors and rebase conflicts) as well as removes some unneeded -Z options (obsoleted over time).
* Remove `std::rt::borrowck`
@bors bors closed this Jan 21, 2014
@bors bors merged commit d84c336 into rust-lang:master Jan 21, 2014
@alexcrichton alexcrichton deleted the zed-cleanup branch January 21, 2014 19:23
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 16, 2023
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 16, 2023
…-null, r=llogiq

fix(ptr_as_ptr): handle `std::ptr::null{_mut}`

close rust-lang#11066
close rust-lang#11665
close rust-lang#11911

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: [`ptr_as_ptr`]: handle `std::ptr::null` and `std::ptr::null_mut`
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