• RustyNova@lemmy.world
    link
    fedilink
    arrow-up
    3
    ·
    6 months ago

    It’s a lot less readable imo. As well than a cargo fmt later and it’s gone (unless there’s a nightly setting for it)

    • Doods@infosec.pub
      link
      fedilink
      arrow-up
      1
      arrow-down
      1
      ·
      6 months ago

      Formatters are off-topic for this, styles come first, formatters are developed later.

      My other reply:

      How about this one? it more closely mirrors the switch example:

      match suffix {
      'G' | 'g' => mem -= 30,
      'M' | 'm' => mem -= 20,
      'K' | 'k' => mem -= 10,
      _ => {},
      }
      

      How about this other one? it goes as far as cloning the switch example’s indentation:

      match suffix {
      'G' | 'g' => {
      	mem -= 30;
             }
      'M' | 'm' => {
      	mem -= 20;
             }
      'K' | 'k' => {
      	mem -= 10;
             }
      _ => {},
      }
      
      • folkrav@lemmy.ca
        link
        fedilink
        arrow-up
        4
        ·
        6 months ago

        I mean, I use formatters everywhere I can exactly so I don’t have to think about code style. I’ll take a full code base that’s consistent in a style I dislike, over having another subjective debate about which style is prettier or easier to read, any day. So whatever cargo fmt spits out is exactly what I’ll prefer, regardless of what it looks like, if only for mere consistency.