[tor-commits] [tor/master] rust/docs: fix critical typo for missing_docs lint

nickm at torproject.org nickm at torproject.org
Thu Aug 16 12:43:13 UTC 2018


commit fef2ba2267af831ff8579f8f63a898135954f44d
Author: cypherpunks <cypherpunks at torproject.org>
Date:   Fri Aug 3 20:06:40 2018 +0000

    rust/docs: fix critical typo for missing_docs lint
    
    Fix typo from fe66d06a45a4714141eba992fe87ec3dd5fa1c22.
    The exclamation point is what lets an attribute apply to
    an entire crate, without the ! it's practically a placebo.
    
    Fix on commits af182d4ab51d6a1a70559bbdcd4ab842aa855684 and
    b6059297d7cb76f0e00e2098e38d6677d3033340, and note there are
    still missing docs in both crypto and protover, for now.
    
    https://doc.rust-lang.org/reference/attributes.html
---
 doc/HACKING/CodingStandardsRust.md | 2 +-
 src/rust/crypto/lib.rs             | 3 ++-
 src/rust/protover/lib.rs           | 3 ++-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/doc/HACKING/CodingStandardsRust.md b/doc/HACKING/CodingStandardsRust.md
index d9496c08f..fc562816d 100644
--- a/doc/HACKING/CodingStandardsRust.md
+++ b/doc/HACKING/CodingStandardsRust.md
@@ -104,7 +104,7 @@ repo.
  Documentation
 ---------------
 
-You MUST include `#[deny(missing_docs)]` in your crate.
+You MUST include `#![deny(missing_docs)]` in your crate.
 
 For function/method comments, you SHOULD include a one-sentence, "first person"
 description of function behaviour (see requirements for documentation as
diff --git a/src/rust/crypto/lib.rs b/src/rust/crypto/lib.rs
index f72a859dd..3b58935fd 100644
--- a/src/rust/crypto/lib.rs
+++ b/src/rust/crypto/lib.rs
@@ -24,7 +24,8 @@
 //! assert!(result == [b'X'; DIGEST256_LEN]);
 //! ```
 
-#[deny(missing_docs)]
+// XXX: add missing docs
+//#![deny(missing_docs)]
 
 // External crates from cargo or TOR_RUST_DEPENDENCIES.
 extern crate digest;
diff --git a/src/rust/protover/lib.rs b/src/rust/protover/lib.rs
index 5da562c1e..4ea20665c 100644
--- a/src/rust/protover/lib.rs
+++ b/src/rust/protover/lib.rs
@@ -22,7 +22,8 @@
 //! protocols to develop independently, without having to claim compatibility
 //! with specific versions of Tor.
 
-#[deny(missing_docs)]
+// XXX: add missing docs
+//#![deny(missing_docs)]
 
 extern crate libc;
 extern crate smartlist;





More information about the tor-commits mailing list