rust/clippy: comments on why we have specific allows

diff --git a/rust/src/lib.rs b/rust/src/lib.rs
index 12e4a796eda889cec5c78a57d75fe7e62b64b956..a6828ff8036228e891a1dcd30c098ac569930fb4 100644 (file)
--- a/rust/src/lib.rs
+++ b/rust/src/lib.rs
@@ -21,14 +21,20 @@
 #![allow(clippy::needless_return)]
 #![allow(clippy::let_and_return)]
 
-// Clippy lints we want to suppress due to style, or simply too noisy
-// and not a priority right now.
-#![allow(clippy::missing_safety_doc)]
-#![allow(clippy::too_many_arguments)]
+// We find this makes sense at time.
+#![allow(clippy::module_inception)]
 
-// To be fixed, but remove the noise for now.
+// The match macro is not always more clear. But its use is
+// recommended where it makes sense.
 #![allow(clippy::match_like_matches_macro)]
-#![allow(clippy::module_inception)]
+
+// Something we should be conscious of, but due to interfacing with C
+// is unavoidable at this time.
+#![allow(clippy::too_many_arguments)]
+
+// TODO: All unsafe functions should have a safety doc, even if its
+// just due to FFI.
+#![allow(clippy::missing_safety_doc)]
 
 #[macro_use]
 extern crate bitflags;