Rename lint to invalid_null_ptr_usage.

Author: boxdot

CHANGELOG.md

@@ -1872,7 +1872,7 @@ Released 2018-09-13
 [`into_iter_on_array`]: https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_array
 [`into_iter_on_ref`]: https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
 [`invalid_atomic_ordering`]: https://rust-lang.github.io/rust-clippy/master/index.html#invalid_atomic_ordering
-[`invalid_null_usage`]: https://rust-lang.github.io/rust-clippy/master/index.html#invalid_null_usage
+[`invalid_null_ptr_usage`]: https://rust-lang.github.io/rust-clippy/master/index.html#invalid_null_ptr_usage
 [`invalid_ref`]: https://rust-lang.github.io/rust-clippy/master/index.html#invalid_ref
 [`invalid_regex`]: https://rust-lang.github.io/rust-clippy/master/index.html#invalid_regex
 [`invalid_upcast_comparisons`]: https://rust-lang.github.io/rust-clippy/master/index.html#invalid_upcast_comparisons

clippy_lints/src/lib.rs

@@ -804,7 +804,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
         &pattern_type_mismatch::PATTERN_TYPE_MISMATCH,
         &precedence::PRECEDENCE,
         &ptr::CMP_NULL,
-        &ptr::INVALID_NULL_USAGE,
+        &ptr::INVALID_NULL_PTR_USAGE,
         &ptr::MUT_FROM_REF,
         &ptr::PTR_ARG,
         &ptr_eq::PTR_EQ,
@@ -1512,7 +1512,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
         LintId::of(&partialeq_ne_impl::PARTIALEQ_NE_IMPL),
         LintId::of(&precedence::PRECEDENCE),
         LintId::of(&ptr::CMP_NULL),
-        LintId::of(&ptr::INVALID_NULL_USAGE),
+        LintId::of(&ptr::INVALID_NULL_PTR_USAGE),
         LintId::of(&ptr::MUT_FROM_REF),
         LintId::of(&ptr::PTR_ARG),
         LintId::of(&ptr_eq::PTR_EQ),
@@ -1836,7 +1836,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
         LintId::of(&mut_key::MUTABLE_KEY_TYPE),
         LintId::of(&open_options::NONSENSICAL_OPEN_OPTIONS),
         LintId::of(&option_env_unwrap::OPTION_ENV_UNWRAP),
-        LintId::of(&ptr::INVALID_NULL_USAGE),
+        LintId::of(&ptr::INVALID_NULL_PTR_USAGE),
         LintId::of(&ptr::MUT_FROM_REF),
         LintId::of(&ranges::REVERSED_EMPTY_RANGES),
         LintId::of(&regex::INVALID_REGEX),

clippy_lints/src/ptr.rs

@@ -135,12 +135,12 @@ declare_clippy_lint! {
     /// // Good
     /// unsafe { std::slice::from_raw_parts(NonNull::dangling().as_ptr(), 0); }
     /// ```
-    pub INVALID_NULL_USAGE,
+    pub INVALID_NULL_PTR_USAGE,
     correctness,
     "invalid usage of a null pointer, suggesting `NonNull::dangling()` instead."
 }
 
-declare_lint_pass!(Ptr => [PTR_ARG, CMP_NULL, MUT_FROM_REF, INVALID_NULL_USAGE]);
+declare_lint_pass!(Ptr => [PTR_ARG, CMP_NULL, MUT_FROM_REF, INVALID_NULL_PTR_USAGE]);
 
 impl<'tcx> LateLintPass<'tcx> for Ptr {
     fn check_item(&mut self, cx: &LateContext<'tcx>, item: &'tcx Item<'_>) {
@@ -187,7 +187,7 @@ impl<'tcx> LateLintPass<'tcx> for Ptr {
                 if is_null_path(arg) {
                     span_lint_and_sugg(
                         cx,
-                        INVALID_NULL_USAGE,
+                        INVALID_NULL_PTR_USAGE,
                         arg.span,
                         "pointer must be non-null",
                         "change this to",

src/lintlist/mod.rs

@@ -978,7 +978,7 @@ vec![
         module: "atomic_ordering",
     },
     Lint {
-        name: "invalid_null_usage",
+        name: "invalid_null_ptr_usage",
         group: "correctness",
         desc: "invalid usage of a null pointer, suggesting `NonNull::dangling()` instead.",
         deprecation: None,

tests/ui/invalid_null_ptr_usage.fixed

@@ -0,0 +1,6 @@
+// run-rustfix
+
+fn main() {
+    let _slice: &[usize] = unsafe { std::slice::from_raw_parts(core::ptr::NonNull::dangling().as_ptr(), 0) };
+    let _slice: &[usize] = unsafe { std::slice::from_raw_parts(core::ptr::NonNull::dangling().as_ptr(), 0) };
+}

tests/ui/invalid_null_usage.rs renamed to tests/ui/invalid_null_ptr_usage.rs

@@ -1,13 +1,13 @@
 error: pointer must be non-null
-  --> $DIR/invalid_null_usage.rs:4:64
+  --> $DIR/invalid_null_ptr_usage.rs:4:64
    |
 LL |     let _slice: &[usize] = unsafe { std::slice::from_raw_parts(std::ptr::null(), 0) };
    |                                                                ^^^^^^^^^^^^^^^^ help: change this to: `core::ptr::NonNull::dangling().as_ptr()`
    |
-   = note: `#[deny(clippy::invalid_null_usage)]` on by default
+   = note: `#[deny(clippy::invalid_null_ptr_usage)]` on by default
 
 error: pointer must be non-null
-  --> $DIR/invalid_null_usage.rs:5:64
+  --> $DIR/invalid_null_ptr_usage.rs:5:64
    |
 LL |     let _slice: &[usize] = unsafe { std::slice::from_raw_parts(core::ptr::null(), 0) };
    |                                                                ^^^^^^^^^^^^^^^^^ help: change this to: `core::ptr::NonNull::dangling().as_ptr()`

tests/ui/invalid_null_usage.stderr renamed to tests/ui/invalid_null_ptr_usage.stderr

@@ -1,6 +1,6 @@
 // run-rustfix
 
 fn main() {
-    let _slice: &[usize] = unsafe { std::slice::from_raw_parts(core::ptr::NonNull::dangling().as_ptr(), 0) };
+    let _slice: &[usize] = unsafe { std::slice::from_raw_parts(std::ptr::null(), 0) };
     let _slice: &[usize] = unsafe { std::slice::from_raw_parts(core::ptr::NonNull::dangling().as_ptr(), 0) };
 }