[PATCH] D68589: [lit] Leverage argparse features to remove some code

[serge via Phabricator via llvm-commits]

serge-sans-paille added inline comments.


================
Comment at: llvm/utils/lit/lit/cl_arguments.py:204
+        n = int(arg)
+    except:
+        raise _arg_error('positive integer', arg)
----------------
It's generally better to catch the conversion error explicitly (here ``ValueError``)


================
Comment at: llvm/utils/lit/lit/cl_arguments.py:211
+def _arg_error(desc, arg):
+    msg = "require %s, but found '%s'" % (desc, arg)
+    return argparse.ArgumentTypeError(msg)
----------------
require*s*


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68589/new/

https://reviews.llvm.org/D68589