The ‘as’ operator should not be used with numeric operands

Guideline: The 'as' operator should not be used with numeric operands gui_ADHABsmK9FXz
status: draft
tags: subset, reduce-human-error
category: advisory
decidability: decidable
scope: module
release: <TODO>

The binary operator as should not be used with:

  • a numeric type, including all supported integer, floating, and machine-dependent arithmetic types; or

  • bool; or

  • char

as either the right operand or the type of the left operand.

Exception: as may be used with usize as the right operand and an expression of raw pointer type as the left operand.

An as cast from a numeric value to a raw pointer type is outside the scope of this guideline; it is prohibited by the guideline A numeric value shall not be converted to a pointer.

Rationale: rat_v56bjjcveLxQ
status: draft
parent needs: gui_ADHABsmK9FXz

Although the conversions performed by as between numeric types are all well-defined, as coerces the value to fit in the destination type, which may result in unexpected data loss if the value needs to be truncated, rounded, or produce a nearest possible non-equal value.

Although some conversions are lossless, others are not symmetrical. Instead of relying on either a defined lossy behaviour or risking loss of precision, the code can communicate intent by using Into or From and TryInto or TryFrom to signal which conversions are intended to perfectly preserve the original value, and which are intended to be fallible. The latter cannot be used from const functions, indicating that these should avoid using fallible conversions.

A pointer-to-address cast produces an integer that represents the pointer’s machine address. The address is truncated when the destination integer type is too small. The usize type is guaranteed to be wide enough to hold the address value.

The exception permits extracting an address only. Converting a numeric value back to a raw pointer is governed by the guideline A numeric value shall not be converted to a pointer.

Non-Compliant Example: non_compl_ex_hzGUYoMnK59w
status: draft
parent needs: gui_ADHABsmK9FXz

as used here can change the value range or lose precision. Even when it doesn’t, nothing enforces the correct behaviour or communicates whether we intend to allow lossy conversions, or only expect valid conversions.

#[allow(dead_code)]
fn f1(x: u16, y: i32, _z: u64, w: u8) {
  let _a = w as char;           // non-compliant
  let _b = y as u32;            // non-compliant - changes value range, converting negative values
  let _c = x as i64;            // non-compliant - could use .into()

  let d = y as f32;            // non-compliant - lossy
  let e = d as f64;            // non-compliant - could use .into()
  let _f = e as f32;            // non-compliant - lossy

  let _g = e as i64;            // non-compliant - lossy despite object size

  let b: u32 = 0;
  let p1: * const u32 = &b;
  let _a1 = p1 as usize;        // compliant by exception
  let _a2 = p1 as u16;          // non-compliant - may lose address range
  let _a3 = p1 as u64;          // non-compliant - use usize to indicate intent
}
Compliant Example: compl_ex_uilHTIOgxD37
status: draft
parent needs: gui_ADHABsmK9FXz

Valid conversions that are guaranteed to preserve exact values can be communicated better with into() or from(). Valid conversions that risk losing value, where doing so would be an error, can communicate this and include an error check, with try_into or try_from. Bit reinterpretation through transmute is a different operation from numeric value conversion. Every transmute must independently satisfy its safety requirements.

miri
use std::convert::TryInto;

#[allow(dead_code)]
fn f2(x: u16, y: i32, _z: u64, w: u8) {
  let _a: char            = w.into();
  let _b: Result <u32, _> = y.try_into(); // produce an error on range clip
  let _c: i64             = x.into();

  let d = f32::from(x);  // u16 is within range, u32 is not
  let _e = f64::from(d);
  // let f = f32::from(e); // no From exists

  // let g = ...            // no From exists

  let h: u32 = 0;
  let p1: * const u32 = &h;
  let _address = p1 as usize; // compliant by exception

  unsafe {
    // Reinterpret the bits of z as an IEEE double-precision value.
    // This is not a numeric value conversion.
    #[allow(unnecessary_transmutes)]
    let _f1: f64 = std::mem::transmute(_z);
  }
}