]> git.lizzy.rs Git - rust.git/blob - src/librustc_typeck/check/demand.rs
First stab at fixing #54505
[rust.git] / src / librustc_typeck / check / demand.rs
1 // Copyright 2012 The Rust Project Developers. See the COPYRIGHT
2 // file at the top-level directory of this distribution and at
3 // http://rust-lang.org/COPYRIGHT.
4 //
5 // Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6 // http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8 // option. This file may not be copied, modified, or distributed
9 // except according to those terms.
10
11 use check::FnCtxt;
12 use rustc::infer::InferOk;
13 use rustc::traits::ObligationCause;
14
15 use syntax::ast;
16 use syntax::util::parser::PREC_POSTFIX;
17 use syntax_pos::Span;
18 use rustc::hir;
19 use rustc::hir::def::Def;
20 use rustc::hir::Node;
21 use rustc::hir::{Item, ItemKind, print};
22 use rustc::ty::{self, Ty, AssociatedItem};
23 use rustc::ty::adjustment::AllowTwoPhase;
24 use errors::{Applicability, DiagnosticBuilder, SourceMapper};
25
26 use super::method::probe;
27
28 impl<'a, 'gcx, 'tcx> FnCtxt<'a, 'gcx, 'tcx> {
29     // Requires that the two types unify, and prints an error message if
30     // they don't.
31     pub fn demand_suptype(&self, sp: Span, expected: Ty<'tcx>, actual: Ty<'tcx>) {
32         self.demand_suptype_diag(sp, expected, actual).map(|mut e| e.emit());
33     }
34
35     pub fn demand_suptype_diag(&self,
36                                sp: Span,
37                                expected: Ty<'tcx>,
38                                actual: Ty<'tcx>) -> Option<DiagnosticBuilder<'tcx>> {
39         let cause = &self.misc(sp);
40         match self.at(cause, self.param_env).sup(expected, actual) {
41             Ok(InferOk { obligations, value: () }) => {
42                 self.register_predicates(obligations);
43                 None
44             },
45             Err(e) => {
46                 Some(self.report_mismatched_types(&cause, expected, actual, e))
47             }
48         }
49     }
50
51     pub fn demand_eqtype(&self, sp: Span, expected: Ty<'tcx>, actual: Ty<'tcx>) {
52         if let Some(mut err) = self.demand_eqtype_diag(sp, expected, actual) {
53             err.emit();
54         }
55     }
56
57     pub fn demand_eqtype_diag(&self,
58                              sp: Span,
59                              expected: Ty<'tcx>,
60                              actual: Ty<'tcx>) -> Option<DiagnosticBuilder<'tcx>> {
61         self.demand_eqtype_with_origin(&self.misc(sp), expected, actual)
62     }
63
64     pub fn demand_eqtype_with_origin(&self,
65                                      cause: &ObligationCause<'tcx>,
66                                      expected: Ty<'tcx>,
67                                      actual: Ty<'tcx>) -> Option<DiagnosticBuilder<'tcx>> {
68         match self.at(cause, self.param_env).eq(expected, actual) {
69             Ok(InferOk { obligations, value: () }) => {
70                 self.register_predicates(obligations);
71                 None
72             }
73             Err(e) => {
74                 Some(self.report_mismatched_types(cause, expected, actual, e))
75             }
76         }
77     }
78
79     pub fn demand_coerce(&self,
80                          expr: &hir::Expr,
81                          checked_ty: Ty<'tcx>,
82                          expected: Ty<'tcx>,
83                          allow_two_phase: AllowTwoPhase)
84                          -> Ty<'tcx> {
85         let (ty, err) = self.demand_coerce_diag(expr, checked_ty, expected, allow_two_phase);
86         if let Some(mut err) = err {
87             err.emit();
88         }
89         ty
90     }
91
92     // Checks that the type of `expr` can be coerced to `expected`.
93     //
94     // NB: This code relies on `self.diverges` to be accurate. In
95     // particular, assignments to `!` will be permitted if the
96     // diverges flag is currently "always".
97     pub fn demand_coerce_diag(&self,
98                               expr: &hir::Expr,
99                               checked_ty: Ty<'tcx>,
100                               expected: Ty<'tcx>,
101                               allow_two_phase: AllowTwoPhase)
102                               -> (Ty<'tcx>, Option<DiagnosticBuilder<'tcx>>) {
103         let expected = self.resolve_type_vars_with_obligations(expected);
104
105         let e = match self.try_coerce(expr, checked_ty, expected, allow_two_phase) {
106             Ok(ty) => return (ty, None),
107             Err(e) => e
108         };
109
110         let cause = self.misc(expr.span);
111         let expr_ty = self.resolve_type_vars_with_obligations(checked_ty);
112         let mut err = self.report_mismatched_types(&cause, expected, expr_ty, e);
113
114         // If the expected type is an enum with any variants whose sole
115         // field is of the found type, suggest such variants. See Issue
116         // #42764.
117         if let ty::Adt(expected_adt, substs) = expected.sty {
118             let compatible_variants = expected_adt.variants
119                                                   .iter()
120                                                   .filter(|variant| variant.fields.len() == 1)
121                                                   .filter_map(|variant| {
122                 let sole_field = &variant.fields[0];
123                 let sole_field_ty = sole_field.ty(self.tcx, substs);
124                 if self.can_coerce(expr_ty, sole_field_ty) {
125                     let variant_path = self.tcx.item_path_str(variant.did);
126                     Some(variant_path.trim_left_matches("std::prelude::v1::").to_string())
127                 } else {
128                     None
129                 }
130             }).collect::<Vec<_>>();
131
132             if !compatible_variants.is_empty() {
133                 let expr_text = print::to_string(print::NO_ANN, |s| s.print_expr(expr));
134                 let suggestions = compatible_variants.iter()
135                     .map(|v| format!("{}({})", v, expr_text)).collect::<Vec<_>>();
136                 err.span_suggestions_with_applicability(
137                      expr.span,
138                      "try using a variant of the expected type",
139                      suggestions,
140                      Applicability::MaybeIncorrect,
141                 );
142             }
143         }
144
145         self.suggest_ref_or_into(&mut err, expr, expected, expr_ty);
146
147         (expected, Some(err))
148     }
149
150     pub fn get_conversion_methods(&self, span: Span, expected: Ty<'tcx>, checked_ty: Ty<'tcx>)
151                               -> Vec<AssociatedItem> {
152         let mut methods = self.probe_for_return_type(span,
153                                                      probe::Mode::MethodCall,
154                                                      expected,
155                                                      checked_ty,
156                                                      ast::DUMMY_NODE_ID);
157         methods.retain(|m| {
158             self.has_no_input_arg(m) &&
159                 self.tcx.get_attrs(m.def_id).iter()
160                 // This special internal attribute is used to whitelist
161                 // "identity-like" conversion methods to be suggested here.
162                 //
163                 // FIXME (#46459 and #46460): ideally
164                 // `std::convert::Into::into` and `std::borrow:ToOwned` would
165                 // also be `#[rustc_conversion_suggestion]`, if not for
166                 // method-probing false-positives and -negatives (respectively).
167                 //
168                 // FIXME? Other potential candidate methods: `as_ref` and
169                 // `as_mut`?
170                 .find(|a| a.check_name("rustc_conversion_suggestion")).is_some()
171         });
172
173         methods
174     }
175
176     // This function checks if the method isn't static and takes other arguments than `self`.
177     fn has_no_input_arg(&self, method: &AssociatedItem) -> bool {
178         match method.def() {
179             Def::Method(def_id) => {
180                 self.tcx.fn_sig(def_id).inputs().skip_binder().len() == 1
181             }
182             _ => false,
183         }
184     }
185
186     /// Identify some cases where `as_ref()` would be appropriate and suggest it.
187     ///
188     /// Given the following code:
189     /// ```
190     /// struct Foo;
191     /// fn takes_ref(_: &Foo) {}
192     /// let ref opt = Some(Foo);
193     ///
194     /// opt.map(|arg| takes_ref(arg));
195     /// ```
196     /// Suggest using `opt.as_ref().map(|arg| takes_ref(arg));` instead.
197     ///
198     /// It only checks for `Option` and `Result` and won't work with
199     /// ```
200     /// opt.map(|arg| { takes_ref(arg) });
201     /// ```
202     fn can_use_as_ref(&self, expr: &hir::Expr) -> Option<(Span, &'static str, String)> {
203         if let hir::ExprKind::Path(hir::QPath::Resolved(_, ref path)) = expr.node {
204             if let hir::def::Def::Local(id) = path.def {
205                 let parent = self.tcx.hir.get_parent_node(id);
206                 if let Some(Node::Expr(hir::Expr {
207                     id,
208                     node: hir::ExprKind::Closure(_, decl, ..),
209                     ..
210                 })) = self.tcx.hir.find(parent) {
211                     let parent = self.tcx.hir.get_parent_node(*id);
212                     if let (Some(Node::Expr(hir::Expr {
213                         node: hir::ExprKind::MethodCall(path, span, expr),
214                         ..
215                     })), 1) = (self.tcx.hir.find(parent), decl.inputs.len()) {
216                         let self_ty = self.tables.borrow().node_id_to_type(expr[0].hir_id);
217                         let self_ty = format!("{:?}", self_ty);
218                         let name = path.ident.as_str();
219                         let is_as_ref_able = (
220                             self_ty.starts_with("&std::option::Option") ||
221                             self_ty.starts_with("&std::result::Result") ||
222                             self_ty.starts_with("std::option::Option") ||
223                             self_ty.starts_with("std::result::Result")
224                         ) && (name == "map" || name == "and_then");
225                         if is_as_ref_able {
226                             return Some((span.shrink_to_lo(),
227                                          "consider using `as_ref` instead",
228                                          "as_ref().".into()));
229                         }
230                     }
231                 }
232             }
233         }
234         None
235     }
236
237     /// This function is used to determine potential "simple" improvements or users' errors and
238     /// provide them useful help. For example:
239     ///
240     /// ```
241     /// fn some_fn(s: &str) {}
242     ///
243     /// let x = "hey!".to_owned();
244     /// some_fn(x); // error
245     /// ```
246     ///
247     /// No need to find every potential function which could make a coercion to transform a
248     /// `String` into a `&str` since a `&` would do the trick!
249     ///
250     /// In addition of this check, it also checks between references mutability state. If the
251     /// expected is mutable but the provided isn't, maybe we could just say "Hey, try with
252     /// `&mut`!".
253     pub fn check_ref(&self,
254                  expr: &hir::Expr,
255                  checked_ty: Ty<'tcx>,
256                  expected: Ty<'tcx>)
257                  -> Option<(Span, &'static str, String)> {
258         let cm = self.sess().source_map();
259         // Use the callsite's span if this is a macro call. #41858
260         let sp = cm.call_span_if_macro(expr.span);
261         if !cm.span_to_filename(sp).is_real() {
262             return None;
263         }
264
265         match (&expected.sty, &checked_ty.sty) {
266             (&ty::Ref(_, exp, _), &ty::Ref(_, check, _)) => match (&exp.sty, &check.sty) {
267                 (&ty::Str, &ty::Array(arr, _)) |
268                 (&ty::Str, &ty::Slice(arr)) if arr == self.tcx.types.u8 => {
269                     if let hir::ExprKind::Lit(_) = expr.node {
270                         if let Ok(src) = cm.span_to_snippet(sp) {
271                             if src.starts_with("b\"") {
272                                 return Some((sp,
273                                              "consider removing the leading `b`",
274                                              src[1..].to_string()));
275                             }
276                         }
277                     }
278                 },
279                 (&ty::Array(arr, _), &ty::Str) |
280                 (&ty::Slice(arr), &ty::Str) if arr == self.tcx.types.u8 => {
281                     if let hir::ExprKind::Lit(_) = expr.node {
282                         if let Ok(src) = cm.span_to_snippet(sp) {
283                             if src.starts_with("\"") {
284                                 return Some((sp,
285                                              "consider adding a leading `b`",
286                                              format!("b{}", src)));
287                             }
288                         }
289                     }
290                 }
291                 _ => {}
292             },
293             (&ty::Ref(_, _, mutability), _) => {
294                 // Check if it can work when put into a ref. For example:
295                 //
296                 // ```
297                 // fn bar(x: &mut i32) {}
298                 //
299                 // let x = 0u32;
300                 // bar(&x); // error, expected &mut
301                 // ```
302                 let ref_ty = match mutability {
303                     hir::Mutability::MutMutable => self.tcx.mk_mut_ref(
304                                                        self.tcx.mk_region(ty::ReStatic),
305                                                        checked_ty),
306                     hir::Mutability::MutImmutable => self.tcx.mk_imm_ref(
307                                                        self.tcx.mk_region(ty::ReStatic),
308                                                        checked_ty),
309                 };
310                 if self.can_coerce(ref_ty, expected) {
311                     if let Ok(src) = cm.span_to_snippet(sp) {
312                         let sugg_expr = match expr.node { // parenthesize if needed (Issue #46756)
313                             hir::ExprKind::Cast(_, _)        |
314                             hir::ExprKind::Binary(_, _, _)   |
315                             _ if self.is_range_literal(expr) => format!("({})", src),
316                             _                                => src,
317                         };
318                         if let Some(sugg) = self.can_use_as_ref(expr) {
319                             return Some(sugg);
320                         }
321                         return Some(match mutability {
322                             hir::Mutability::MutMutable => {
323                                 (sp, "consider mutably borrowing here", format!("&mut {}",
324                                                                                 sugg_expr))
325                             }
326                             hir::Mutability::MutImmutable => {
327                                 (sp, "consider borrowing here", format!("&{}", sugg_expr))
328                             }
329                         });
330                     }
331                 }
332             }
333             (_, &ty::Ref(_, checked, _)) => {
334                 // We have `&T`, check if what was expected was `T`. If so,
335                 // we may want to suggest adding a `*`, or removing
336                 // a `&`.
337                 //
338                 // (But, also check check the `expn_info()` to see if this is
339                 // a macro; if so, it's hard to extract the text and make a good
340                 // suggestion, so don't bother.)
341                 if self.infcx.can_sub(self.param_env, checked, &expected).is_ok() &&
342                    sp.ctxt().outer().expn_info().is_none() {
343                     match expr.node {
344                         // Maybe remove `&`?
345                         hir::ExprKind::AddrOf(_, ref expr) => {
346                             if !cm.span_to_filename(expr.span).is_real() {
347                                 return None;
348                             }
349                             if let Ok(code) = cm.span_to_snippet(expr.span) {
350                                 return Some((sp, "consider removing the borrow", code));
351                             }
352                         }
353
354                         // Maybe add `*`? Only if `T: Copy`.
355                         _ => {
356                             if !self.infcx.type_moves_by_default(self.param_env,
357                                                                 checked,
358                                                                 sp) {
359                                 // do not suggest if the span comes from a macro (#52783)
360                                 if let (Ok(code),
361                                         true) = (cm.span_to_snippet(sp), sp == expr.span) {
362                                     return Some((
363                                         sp,
364                                         "consider dereferencing the borrow",
365                                         format!("*{}", code),
366                                     ));
367                                 }
368                             }
369                         }
370                     }
371                 }
372             }
373             _ => {}
374         }
375         None
376     }
377
378     // This function checks if the specified expression is a built-in range literal
379     // (See: librustc/hir/lowering.rs::LoweringContext::lower_expr() )
380     fn is_range_literal(&self, expr: &hir::Expr) -> bool {
381         use hir::{Path, QPath, ExprKind, TyKind};
382
383         // TODO how to work out std vs core here?
384         let ops_path = ["{{root}}", "std", "ops"];
385
386         let is_range_path = |path: &Path| {
387             let ident_names: Vec<_> = path.segments
388                 .iter()
389                 .map(|seg| seg.ident.as_str())
390                 .collect();
391
392             if let Some((last, preceding)) = ident_names.split_last() {
393                 last.starts_with("Range") &&
394                     preceding.len() == 3 &&
395                     preceding.iter()
396                         .zip(ops_path.iter())
397                         .all(|(a, b)| a == b)
398             } else {
399                 false
400             }
401         };
402
403         match expr.node {
404             ExprKind::Struct(QPath::Resolved(None, ref path), _, _) |
405             ExprKind::Path(QPath::Resolved(None, ref path)) => {
406                 return is_range_path(&path);
407             }
408
409             ExprKind::Call(ref func, _) => {
410                 if let ExprKind::Path(QPath::TypeRelative(ref ty, ref segment)) = func.node {
411                     if let TyKind::Path(QPath::Resolved(None, ref path)) = ty.node {
412                         let calls_new = segment.ident.as_str() == "new";
413
414                         return is_range_path(&path) && calls_new;
415                     }
416                 }
417             }
418
419             _ => {}
420         }
421
422         false
423     }
424
425     pub fn check_for_cast(&self,
426                       err: &mut DiagnosticBuilder<'tcx>,
427                       expr: &hir::Expr,
428                       checked_ty: Ty<'tcx>,
429                       expected_ty: Ty<'tcx>)
430                       -> bool {
431         let parent_id = self.tcx.hir.get_parent_node(expr.id);
432         if let Some(parent) = self.tcx.hir.find(parent_id) {
433             // Shouldn't suggest `.into()` on `const`s.
434             if let Node::Item(Item { node: ItemKind::Const(_, _), .. }) = parent {
435                 // FIXME(estebank): modify once we decide to suggest `as` casts
436                 return false;
437             }
438         };
439
440         let will_truncate = "will truncate the source value";
441         let depending_on_isize = "will truncate or zero-extend depending on the bit width of \
442                                   `isize`";
443         let depending_on_usize = "will truncate or zero-extend depending on the bit width of \
444                                   `usize`";
445         let will_sign_extend = "will sign-extend the source value";
446         let will_zero_extend = "will zero-extend the source value";
447
448         // If casting this expression to a given numeric type would be appropriate in case of a type
449         // mismatch.
450         //
451         // We want to minimize the amount of casting operations that are suggested, as it can be a
452         // lossy operation with potentially bad side effects, so we only suggest when encountering
453         // an expression that indicates that the original type couldn't be directly changed.
454         //
455         // For now, don't suggest casting with `as`.
456         let can_cast = false;
457
458         let needs_paren = expr.precedence().order() < (PREC_POSTFIX as i8);
459
460         if let Ok(src) = self.tcx.sess.source_map().span_to_snippet(expr.span) {
461             let msg = format!("you can cast an `{}` to `{}`", checked_ty, expected_ty);
462             let cast_suggestion = format!("{}{}{} as {}",
463                                           if needs_paren { "(" } else { "" },
464                                           src,
465                                           if needs_paren { ")" } else { "" },
466                                           expected_ty);
467             let into_suggestion = format!(
468                 "{}{}{}.into()",
469                 if needs_paren { "(" } else { "" },
470                 src,
471                 if needs_paren { ")" } else { "" },
472             );
473             let literal_is_ty_suffixed = |expr: &hir::Expr| {
474                 if let hir::ExprKind::Lit(lit) = &expr.node {
475                     lit.node.is_suffixed()
476                 } else {
477                     false
478                 }
479             };
480
481             let into_sugg = into_suggestion.clone();
482             let suggest_to_change_suffix_or_into = |err: &mut DiagnosticBuilder,
483                                                     note: Option<&str>| {
484                 let suggest_msg = if literal_is_ty_suffixed(expr) {
485                     format!(
486                         "change the type of the numeric literal from `{}` to `{}`",
487                         checked_ty,
488                         expected_ty,
489                     )
490                 } else {
491                     match note {
492                         Some(note) => format!("{}, which {}", msg, note),
493                         _ => format!("{} in a lossless way", msg),
494                     }
495                 };
496
497                 let suffix_suggestion = format!(
498                     "{}{}{}{}",
499                     if needs_paren { "(" } else { "" },
500                     src.trim_right_matches(&checked_ty.to_string()),
501                     expected_ty,
502                     if needs_paren { ")" } else { "" },
503                 );
504
505                 err.span_suggestion_with_applicability(
506                     expr.span,
507                     &suggest_msg,
508                     if literal_is_ty_suffixed(expr) {
509                         suffix_suggestion
510                     } else {
511                         into_sugg
512                     },
513                     Applicability::MachineApplicable,
514                 );
515             };
516
517             match (&expected_ty.sty, &checked_ty.sty) {
518                 (&ty::Int(ref exp), &ty::Int(ref found)) => {
519                     match (found.bit_width(), exp.bit_width()) {
520                         (Some(found), Some(exp)) if found > exp => {
521                             if can_cast {
522                                 err.span_suggestion_with_applicability(
523                                     expr.span,
524                                     &format!("{}, which {}", msg, will_truncate),
525                                     cast_suggestion,
526                                     Applicability::MaybeIncorrect // lossy conversion
527                                 );
528                             }
529                         }
530                         (None, _) | (_, None) => {
531                             if can_cast {
532                                 err.span_suggestion_with_applicability(
533                                     expr.span,
534                                     &format!("{}, which {}", msg, depending_on_isize),
535                                     cast_suggestion,
536                                     Applicability::MaybeIncorrect // lossy conversion
537                                 );
538                             }
539                         }
540                         _ => {
541                             suggest_to_change_suffix_or_into(
542                                 err,
543                                 Some(will_sign_extend),
544                             );
545                         }
546                     }
547                     true
548                 }
549                 (&ty::Uint(ref exp), &ty::Uint(ref found)) => {
550                     match (found.bit_width(), exp.bit_width()) {
551                         (Some(found), Some(exp)) if found > exp => {
552                             if can_cast {
553                                 err.span_suggestion_with_applicability(
554                                     expr.span,
555                                     &format!("{}, which {}", msg, will_truncate),
556                                     cast_suggestion,
557                                     Applicability::MaybeIncorrect  // lossy conversion
558                                 );
559                             }
560                         }
561                         (None, _) | (_, None) => {
562                             if can_cast {
563                                 err.span_suggestion_with_applicability(
564                                     expr.span,
565                                     &format!("{}, which {}", msg, depending_on_usize),
566                                     cast_suggestion,
567                                     Applicability::MaybeIncorrect  // lossy conversion
568                                 );
569                             }
570                         }
571                         _ => {
572                            suggest_to_change_suffix_or_into(
573                                err,
574                                Some(will_zero_extend),
575                            );
576                         }
577                     }
578                     true
579                 }
580                 (&ty::Int(ref exp), &ty::Uint(ref found)) => {
581                     if can_cast {
582                         match (found.bit_width(), exp.bit_width()) {
583                             (Some(found), Some(exp)) if found > exp - 1 => {
584                                 err.span_suggestion_with_applicability(
585                                     expr.span,
586                                     &format!("{}, which {}", msg, will_truncate),
587                                     cast_suggestion,
588                                     Applicability::MaybeIncorrect  // lossy conversion
589                                 );
590                             }
591                             (None, None) => {
592                                 err.span_suggestion_with_applicability(
593                                     expr.span,
594                                     &format!("{}, which {}", msg, will_truncate),
595                                     cast_suggestion,
596                                     Applicability::MaybeIncorrect  // lossy conversion
597                                 );
598                             }
599                             (None, _) => {
600                                 err.span_suggestion_with_applicability(
601                                     expr.span,
602                                     &format!("{}, which {}", msg, depending_on_isize),
603                                     cast_suggestion,
604                                     Applicability::MaybeIncorrect  // lossy conversion
605                                 );
606                             }
607                             (_, None) => {
608                                 err.span_suggestion_with_applicability(
609                                     expr.span,
610                                     &format!("{}, which {}", msg, depending_on_usize),
611                                     cast_suggestion,
612                                     Applicability::MaybeIncorrect  // lossy conversion
613                                 );
614                             }
615                             _ => {
616                                 err.span_suggestion_with_applicability(
617                                     expr.span,
618                                     &format!("{}, which {}", msg, will_zero_extend),
619                                     cast_suggestion,
620                                     Applicability::MachineApplicable
621                                 );
622                             }
623                         }
624                     }
625                     true
626                 }
627                 (&ty::Uint(ref exp), &ty::Int(ref found)) => {
628                     if can_cast {
629                         match (found.bit_width(), exp.bit_width()) {
630                             (Some(found), Some(exp)) if found - 1 > exp => {
631                                 err.span_suggestion_with_applicability(
632                                     expr.span,
633                                     &format!("{}, which {}", msg, will_truncate),
634                                     cast_suggestion,
635                                     Applicability::MaybeIncorrect  // lossy conversion
636                                 );
637                             }
638                             (None, None) => {
639                                 err.span_suggestion_with_applicability(
640                                     expr.span,
641                                     &format!("{}, which {}", msg, will_sign_extend),
642                                     cast_suggestion,
643                                     Applicability::MachineApplicable  // lossy conversion
644                                 );
645                             }
646                             (None, _) => {
647                                 err.span_suggestion_with_applicability(
648                                     expr.span,
649                                     &format!("{}, which {}", msg, depending_on_usize),
650                                     cast_suggestion,
651                                     Applicability::MaybeIncorrect  // lossy conversion
652                                 );
653                             }
654                             (_, None) => {
655                                 err.span_suggestion_with_applicability(
656                                     expr.span,
657                                     &format!("{}, which {}", msg, depending_on_isize),
658                                     cast_suggestion,
659                                     Applicability::MaybeIncorrect  // lossy conversion
660                                 );
661                             }
662                             _ => {
663                                 err.span_suggestion_with_applicability(
664                                     expr.span,
665                                     &format!("{}, which {}", msg, will_sign_extend),
666                                     cast_suggestion,
667                                     Applicability::MachineApplicable
668                                 );
669                             }
670                         }
671                     }
672                     true
673                 }
674                 (&ty::Float(ref exp), &ty::Float(ref found)) => {
675                     if found.bit_width() < exp.bit_width() {
676                        suggest_to_change_suffix_or_into(
677                            err,
678                            None,
679                        );
680                     } else if can_cast {
681                         err.span_suggestion_with_applicability(
682                             expr.span,
683                             &format!("{}, producing the closest possible value", msg),
684                             cast_suggestion,
685                             Applicability::MaybeIncorrect  // lossy conversion
686                         );
687                     }
688                     true
689                 }
690                 (&ty::Uint(_), &ty::Float(_)) | (&ty::Int(_), &ty::Float(_)) => {
691                     if can_cast {
692                         err.span_suggestion_with_applicability(
693                             expr.span,
694                             &format!("{}, rounding the float towards zero", msg),
695                             cast_suggestion,
696                             Applicability::MaybeIncorrect  // lossy conversion
697                         );
698                         err.warn("casting here will cause undefined behavior if the rounded value \
699                                   cannot be represented by the target integer type, including \
700                                   `Inf` and `NaN` (this is a bug and will be fixed)");
701                     }
702                     true
703                 }
704                 (&ty::Float(ref exp), &ty::Uint(ref found)) => {
705                     // if `found` is `None` (meaning found is `usize`), don't suggest `.into()`
706                     if exp.bit_width() > found.bit_width().unwrap_or(256) {
707                         err.span_suggestion_with_applicability(
708                             expr.span,
709                             &format!("{}, producing the floating point representation of the \
710                                       integer",
711                                      msg),
712                             into_suggestion,
713                             Applicability::MachineApplicable
714                         );
715                     } else if can_cast {
716                         err.span_suggestion_with_applicability(expr.span,
717                             &format!("{}, producing the floating point representation of the \
718                                       integer, rounded if necessary",
719                                      msg),
720                             cast_suggestion,
721                             Applicability::MaybeIncorrect  // lossy conversion
722                         );
723                     }
724                     true
725                 }
726                 (&ty::Float(ref exp), &ty::Int(ref found)) => {
727                     // if `found` is `None` (meaning found is `isize`), don't suggest `.into()`
728                     if exp.bit_width() > found.bit_width().unwrap_or(256) {
729                         err.span_suggestion_with_applicability(
730                             expr.span,
731                             &format!("{}, producing the floating point representation of the \
732                                       integer",
733                                      msg),
734                             into_suggestion,
735                             Applicability::MachineApplicable
736                         );
737                     } else if can_cast {
738                         err.span_suggestion_with_applicability(
739                             expr.span,
740                             &format!("{}, producing the floating point representation of the \
741                                       integer, rounded if necessary",
742                                      msg),
743                             cast_suggestion,
744                             Applicability::MaybeIncorrect  // lossy conversion
745                         );
746                     }
747                     true
748                 }
749                 _ => false,
750             }
751         } else {
752             false
753         }
754     }
755 }