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

2022-07-27 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath added a comment. I do not have commit access. Could you please commit on my behalf? Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129311/new/ https://reviews.llvm.org/D129311

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

2022-07-27 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath updated this revision to Diff 448099. sridhar_gopinath marked an inline comment as done. sridhar_gopinath added a comment. Addressed nitpick Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129311/new/ https://reviews.llvm.org/D129311

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

2022-07-26 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath marked 3 inline comments as done. sridhar_gopinath added inline comments. Comment at: clang/tools/clang-format/git-clang-format:201 if opts.diff: -print_diff(old_tree, new_tree) - elif opts.diffstat: -print_diffstat(old_tree, new_tree) - else: -c

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

2022-07-21 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath marked 2 inline comments as done. sridhar_gopinath added inline comments. Comment at: clang/tools/clang-format/git-clang-format:579 with temporary_index_file(old_tree): - subprocess.check_call(['git', 'checkout', '--patch', new_tree]) + subprocess.

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

2022-07-21 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath updated this revision to Diff 446581. sridhar_gopinath added a comment. Removed '--' in the git-diff commands as it only applies to files and not commits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129311/new/ https://reviews.ll

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

2022-07-19 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath marked 4 inline comments as done. sridhar_gopinath added inline comments. Comment at: clang/tools/clang-format/git-clang-format:539-540 # filter. - subprocess.check_call(['git', 'diff', '--diff-filter=M', old_tree, new_tree, - '--']) +

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

2022-07-19 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath updated this revision to Diff 445954. sridhar_gopinath marked 4 inline comments as done. sridhar_gopinath added a comment. Changes to the args order and changing check_call -> run Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129311

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

2022-07-18 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath updated this revision to Diff 445652. sridhar_gopinath added a comment. Updated subprocess.call -> subprocess.run Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129311/new/ https://reviews.llvm.org/D129311 Files: clang/tools/clan

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

2022-07-11 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath updated this revision to Diff 443726. sridhar_gopinath added a comment. Replaced subprocess.check_call with subprocess.call since the former crashes when the return code is not zero. + formatting changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:/

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

2022-07-07 Thread Sridhar Gopinath via Phabricator via cfe-commits
sridhar_gopinath created this revision. sridhar_gopinath added reviewers: curdeius, owenpan. Herald added a project: All. sridhar_gopinath requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. In diff and diffstat modes, the return code is != 0 e