• sexual_tomato@lemmy.dbzer0.com
      link
      fedilink
      arrow-up
      5
      ·
      edit-2
      8 hours ago

      Infinite loop and hard coded magic constant; this should have a configurable timeout and a resource file the string is read from so we can internationalize the application. Additionally, the use of a goto with a hard coded line number is a runtime bug waiting to happen after unrelated refactors; it’s best to use a looping construct that has more deterministic bounds.