Skip to content
This repository was archived by the owner on Aug 16, 2021. It is now read-only.

warning: lint unused_doc_comment has been renamed to unused_doc_comments #251

Closed
jamesray1 opened this issue Aug 15, 2018 · 8 comments
Closed

Comments

@jamesray1
Copy link

These warnings still occur after adding #[allow(renamed_and_removed_lints)]:

Compiling varint v0.1.0 (file:///home/james/Rust/jrl/varint-rs)
warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: #[warn(renamed_and_removed_lints)] on by default
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: #[warn(renamed_and_removed_lints)] on by default
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

warning: lint unused_doc_comment has been renamed to unused_doc_comments
  --> varint-rs/src/lib.rs:47:5
   |
47 | /     error_chain! {
48 | |         errors {
49 | |             /// Error for parsing a varint
50 | |             ParseError {
...  |
63 | |         }
64 | |     }
   | |_____^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)
@fogti
Copy link
Contributor

fogti commented Aug 18, 2018

This 'warning' is turned into an error in 'rustc' (with the current error-chain pkg in /src/vendor/error-chain). Thus, some versions of rustc even fail to build, because of this.

orig ref: https://twitter.com/ZschErik/status/1029062002109571073

@fogti
Copy link
Contributor

fogti commented Sep 2, 2018

pingback rust-lang/rust#51650

@lucab
Copy link

lucab commented Sep 2, 2018

Can you please double check which error-check versions are in your Cargo.lock?

@fogti
Copy link
Contributor

fogti commented Sep 5, 2018

I currently don't know anymore (I deleted the build 10 days ago), but I think it is in rustc-1.27.1 or 1.28.0 (build via gentoo portage ebuild), possible https://github.com/gentoo/gentoo/blob/master/dev-lang/rust/rust-1.28.0.ebuild.
rustc-1.25.0 builds fine.

@fogti
Copy link
Contributor

fogti commented Sep 5, 2018

e.g. error-chain 0.11.0 works with rustc 1.25.0, but error-chain 0.11.0 in rustc 1.27.1-r2 (https://github.com/gentoo/gentoo/blob/master/dev-lang/rust/rust-1.27.1-r2.ebuild) doesn't work anymore.

@lucab
Copy link

lucab commented Sep 5, 2018

0.12.0 should be fixing that, then: https://crates.io/crates/error-chain/0.12.0

@fogti
Copy link
Contributor

fogti commented Nov 7, 2019

@jamesray1 This issue can probably be closed.

@jamesray1
Copy link
Author

OK, I have no need for varint ATM so CBB to try and test/reproduce.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants