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

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 22:27:38 PDT 2022


owenpan accepted this revision.
owenpan added a comment.
This revision is now accepted and ready to land.

Thanks for the explanations! LGTM.



================
Comment at: clang/tools/clang-format/git-clang-format:539-540
   # filter.
-  subprocess.check_call(['git', 'diff', '--diff-filter=M', old_tree, new_tree,
-                         '--'])
+  return subprocess.run(['git', 'diff', '--diff-filter=M',
+                        '--exit-code', old_tree, new_tree]).returncode
 
----------------
Nit: align `--exit-code` with `git` or binpack the arguments.


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