WIP: Add lint on cast Fn to all numerical except usize.

clippy_lints/src/lib.rs

@@ -673,6 +673,7 @@ pub fn register_plugins(reg: &mut rustc_plugin::Registry) {
         types::UNIT_ARG,
         types::UNIT_CMP,
         types::UNNECESSARY_CAST,
+        types::FN_TO_NUMERIC_CAST,
         unicode::ZERO_WIDTH_SPACE,
         unsafe_removed_from_name::UNSAFE_REMOVED_FROM_NAME,
         unused_io_amount::UNUSED_IO_AMOUNT,

clippy_lints/src/types.rs

@@ -679,6 +679,23 @@ declare_clippy_lint! {
     "cast to the same type, e.g. `x as i32` where `x: i32`"
 }
 
+/// **What it does:** Checks for casts function pointer to the numeric type.
+///
+/// **Why is this bad?** Cast pointer not to usize truncate value.
+///
+/// **Known problems:** None.
+///
+/// **Example:**
+/// ```rust
+/// fn test_fn() -> i16;
+/// let _ = test_fn as i32
+/// ```
+declare_clippy_lint! {
+    pub FN_TO_NUMERIC_CAST,
+    correctness,
+    "cast function pointer to the numeric type"
+}
+
 /// **What it does:** Checks for casts from a less-strictly-aligned pointer to a
 /// more-strictly-aligned pointer
 ///
@@ -891,7 +908,8 @@ impl LintPass for CastPass {
             CAST_POSSIBLE_WRAP,
             CAST_LOSSLESS,
             UNNECESSARY_CAST,
-            CAST_PTR_ALIGNMENT
+            CAST_PTR_ALIGNMENT,
+            FN_TO_NUMERIC_CAST
         )
     }
 }
@@ -975,6 +993,24 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for CastPass {
                     },
                 }
             }
+
+            match &cast_from.sty {
+                ty::TyFnDef(..) |
+                ty::TyFnPtr(..) => {
+                    if cast_to.is_numeric() && cast_to.sty != ty::TyUint(UintTy::Usize){
+                        span_lint_and_sugg(
+                            cx,
+                            FN_TO_NUMERIC_CAST,
+                            expr.span,
+                            &format!("casting a Fn to {} may truncate the function address value.", cast_to),
+                            "if you need address of function, use cast to `usize` instead:",
+                            format!("{} as usize", &snippet(cx, ex.span, "x"))
+                        );
+                    }
+                }
+                _ => ()
+            }
+
             if_chain!{
                 if let ty::TyRawPtr(from_ptr_ty) = &cast_from.sty;
                 if let ty::TyRawPtr(to_ptr_ty) = &cast_to.sty;

tests/ui/types_fn_to_int.rs

@@ -0,0 +1,9 @@
+enum Foo {
+    A(usize),
+    B
+}
+
+fn main() {
+    let x = Foo::A;
+    let y = x as i32;
+}

tests/ui/types_fn_to_int.stderr

@@ -0,0 +1,14 @@
+error: casting a Fn to i32 may truncate the function address value.
+ --> $DIR/types_fn_to_int.rs:8:13
+  |
+8 |     let y = x as i32;
+  |             ^^^^^^^^
+  |
+  = note: #[deny(fn_to_numeric_cast)] on by default
+help: if you need address of function, use cast to `usize` instead:
+  |
+8 |     let y = x as usize;
+  |             ^^^^^^^^^^
+
+error: aborting due to previous error
+