• sunbeam60@lemmy.one
    link
    fedilink
    arrow-up
    3
    arrow-down
    12
    ·
    7 个月前

    Yeah but even a single automated test would catch it and reject the PR. You just need a single test.

    • frezik@midwest.social
      link
      fedilink
      arrow-up
      17
      ·
      7 个月前

      No, you can’t assume that. The probability of hitting the condition each time is low. If there aren’t very many calls that hit this, it could easily slip through. Especially on 64-bit int platforms.

      • sunbeam60@lemmy.one
        link
        fedilink
        arrow-up
        2
        arrow-down
        1
        ·
        7 个月前

        Yes agree if you’re talking about unit tests. I’m thinking smoke tests, which is are the most common automated tests in games, where I’ve spent most of professional career. The amount of booleans checks that happen in a single frame I doubt the game wouldn’t crash within the first couple seconds.