• sbv@sh.itjust.works
    link
    fedilink
    English
    arrow-up
    78
    ·
    9 months ago

    I feel like modern compilers would turn their nose up at that shit. “Dead code? Ewww! No way I’m letting that into my syntax tree!”

      • bus_factor@lemmy.world
        link
        fedilink
        arrow-up
        25
        ·
        9 months ago

        Golang won’t even compile with dead code. Unfortunately that’s too strict, you just end up commenting out the whole block instead. At least the commented out code is obvious in review, and some automated checks catch it if you have them.

        • ramjambamalam
          link
          fedilink
          arrow-up
          3
          ·
          9 months ago

          commenting out the whole block

          var foo is declared but not used is such a pain in my asshole when doing this.

        • voxel@sopuli.xyz
          link
          fedilink
          arrow-up
          14
          ·
          edit-2
          9 months ago

          most of my (rust) projects have zero (or maybe 1-2) warnings, unless I’m in the middle of working on a feature

          • micka190@lemmy.world
            link
            fedilink
            arrow-up
            24
            ·
            9 months ago

            First thing I do on my projects is enable warnings as errors and increase my warning levels when reasonable.

            Unfortunately, the same can’t be said on the projects I work on at work. Drives me crazy that we get likes 300+ warnings whenever we run the app and that we can’t change it because "they’re just warnings*.

            • Potatos_are_not_friends@lemmy.world
              link
              fedilink
              arrow-up
              5
              ·
              9 months ago

              It’s not that we want to ignore warnings. It’s just that most warnings take time to fix.

              There’s only so many hours in a day. And we have to accept technical debt in order to deliver, and then pay it off later.

              My job does it well by doing a “spring” and “summer” cleaning where we can turn out 100+ warnings into the single digits. Then busy season happens again and we’re back to 200+.

              • micka190@lemmy.world
                link
                fedilink
                arrow-up
                3
                ·
                9 months ago

                It’s not that we want to ignore warnings.

                Speak for yourself, I promise you the team I work on actively ignores warnings and doesn’t even want to solve them as they pop-up. Being told you can’t compare doubles (because of precision loss) and ignoring it is on the developer and isn’t even that hard to fix. Most of our warnings come from shit like that.

                Like, I get it. It’s probably not worth it to hunt down every “unused variable” warning (especially in an API where we used to have a variable for it and we don’t use it anymore and we don’t want to break the existing API so we just leave it there), but there’s things that are just trivial to fix when you’re working on code that’s right next to it.

            • MonkderZweite@feddit.ch
              link
              fedilink
              arrow-up
              3
              ·
              9 months ago

              Drives me crazy that we get likes 300+ warnings whenever we run the app and that we can’t change it because "they’re just warnings*.

              Laughs in Xamarin. Only 300?

              Well, it was a year ago, is Xamarin now finally changed to the new thing (what was it’s name)?

    • 1stTime4MeInMCU@mander.xyz
      link
      fedilink
      English
      arrow-up
      12
      ·
      9 months ago

      Guess what? Flight Software usually uses ancient proprietary compilers for specialized hardware running an RTOS, rip 😢

      • sbv@sh.itjust.works
        link
        fedilink
        English
        arrow-up
        6
        ·
        9 months ago

        Woah woah WOAH WOAH.

        So you’re saying software for the Artemis landers aren’t being built with the latest TypeScript compiler and running on a canary version of v8?!