• @RustyNova
    link
    37 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)

    • @[email protected]
      link
      fedilink
      07 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
        link
        fedilink
        47 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.