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

ref: Updates for rust 1.48 #288

Merged
merged 3 commits into from
Nov 20, 2020
Merged

ref: Updates for rust 1.48 #288

merged 3 commits into from
Nov 20, 2020

Conversation

flub
Copy link
Contributor

@flub flub commented Nov 20, 2020

#skip-changelog

@flub flub requested a review from a team November 20, 2020 11:46
Copy link
Member

@Swatinem Swatinem left a comment

Choose a reason for hiding this comment

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

these strip_prefix calls are much nicer

It seems this was only on my local machine and CI needs the existing
format?  Strange
let addr = if addr.starts_with("0x") {
u64::from_str_radix(&addr[2..], 16)
let addr = if let Some(addr) = addr.strip_prefix("0x") {
u64::from_str_radix(addr, 16)
Copy link
Member

Choose a reason for hiding this comment

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

Please convert these to a match statement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

huh, why did this not occur to me. much nicer

@flub flub merged commit 013fd4c into master Nov 20, 2020
@flub flub deleted the ref/rust-stable branch November 20, 2020 12:23
jan-auer added a commit that referenced this pull request Dec 22, 2020
* master: (21 commits)
  fix: New nightly clippy lint (#295)
  fix(release): Do not publish examples to crates.io
  release: 8.0.0
  ref(debuginfo): Remove re-exports of error types (#300)
  meta: Update changelog
  ref(debuginfo): Switch to error with kind (#299)
  ref(minidump): Switch to error with kind (#298)
  ref(unreal): Switch to error with kind (#297)
  ref(symcache): Switch to error with kind (#296)
  feat: Rename architecture to wasm32 (#294)
  fix(debuginfo): Prefer DWARF names for Dart functions (#293)
  fix: clippy (#292)
  hack(dwarf): Skip line program sequences at 0 (#291)
  feat(wasm): support for wasm (#166)
  ci: Run rust nighlty on a schedule (#289)
  ref: Updates for rust 1.48 (#288)
  fix(debuginfo): Update dmsort to 1.0.1 to avoid panic due to UB (#287)
  ci: Use GHA instead of zeus (#286)
  ref: Introduce explicit NameMangling and better DemangleOptions (#275)
  meta: Bump all semver-major dependencies (#283)
  ...
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