[PATCH] D129311: [clang-format] Update return code

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 16 12:57:55 PDT 2022


owenpan added inline comments.


================
Comment at: clang/tools/clang-format/git-clang-format:539
   # filter.
-  subprocess.check_call(['git', 'diff', '--diff-filter=M', old_tree, new_tree,
-                         '--'])
+  return subprocess.call(['git', 'diff', '--diff-filter=M',
+                          old_tree, new_tree, '--exit-code', '--'])
----------------
Can we use `run()` instead of `call()` here and below?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129311



More information about the cfe-commits mailing list