Skip to content

Conversation

davidbarsky
Copy link
Member

This PR fixes two types of warnings:

  1. Usage of deprecated methods on std::error::Error.
  2. An unused warning on extern crate proc_macro in tracing-attributes. This does not occur on the stable release of 1.41, but it does occur on 1.42.0-beta.1 and 1.42.0-nightly (cd1ef390e 2020-01-31).

Prior to merging this PR, I'd like to determine whether the second warning is a deliberate change or a regression.

@davidbarsky davidbarsky requested a review from a team February 2, 2020 18:26
@davidbarsky
Copy link
Member Author

It appears to be deliberate; I'm now able to build tracing-attributes without extern crate proc_macro on beta and nightly.

@davidbarsky
Copy link
Member Author

@mystor informed me that this is a deliberate change in Cargo (rust-lang/cargo#7700). Thanks!

Copy link
Member

@hawkw hawkw left a comment

Choose a reason for hiding this comment

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

approved modulo some nits

@davidbarsky davidbarsky merged commit 3c80984 into master Feb 2, 2020
@davidbarsky davidbarsky deleted the david/fix-error-deprecation-warnings branch February 2, 2020 22:42
hawkw added a commit that referenced this pull request Feb 3, 2020
Fixed

- **fmt**: Fixed empty `{}` printed after spans with no fields (f079f2d)
- **fmt**: Fixed inconsistent formatting when ANSI colors are disabled (506a482)
- **fmt**: Fixed mis-aligned levels when ANSI colors are disabled (eba1adb)
- Fixed warnings on nightly Rust compilers (#558)

Signed-off-by: Eliza Weisman <[email protected]>
hawkw added a commit that referenced this pull request Feb 3, 2020
Fixed

- **fmt**: Fixed empty `{}` printed after spans with no fields (f079f2d)
- **fmt**: Fixed inconsistent formatting when ANSI colors are disabled (506a482)
- **fmt**: Fixed mis-aligned levels when ANSI colors are disabled (eba1adb)
- Fixed warnings on nightly Rust compilers (#558)

Signed-off-by: Eliza Weisman <[email protected]>
hawkw added a commit that referenced this pull request Feb 3, 2020
Fixed

- **fmt**: Fixed empty `{}` printed after spans with no fields (f079f2d)
- **fmt**: Fixed inconsistent formatting when ANSI colors are disabled (506a482)
- **fmt**: Fixed mis-aligned levels when ANSI colors are disabled (eba1adb)
- Fixed warnings on nightly Rust compilers (#558)

Signed-off-by: Eliza Weisman <[email protected]>
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.

2 participants