• @RustyNova@lemmy.world
    link
    fedilink
    32 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
      cake
      link
      02 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
        42 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.