Suggest ptr::null_mut::<T>() instead of ptr::null::<T>() as *mut T

[nyurik]

What it does

Use the null_mut() directly instead of first creating a read-only null() and casting it

Advantage

• shorter easier to read code
• avoid accidental later mis-casting

Drawbacks

none

Example

let a = ptr::null::<T>() as *mut T;
let b = ptr::null_mut::<T>() as *const T;

Could be written as:

let a = ptr::null_mut::<T>();
let b = ptr::null::<T>();

[samueltardieu]

@rustbot claim