[clang] edbb8a8 - [clang-format] Add return code to git-clang-format

via cfe-commits cfe-commits at lists.llvm.org
Sun Jan 16 02:43:34 PST 2022


Author: owenca
Date: 2022-01-16T02:41:10-08:00
New Revision: edbb8a843c130e60d71cb73e56a33d5ba2cc0ec9

URL: https://github.com/llvm/llvm-project/commit/edbb8a843c130e60d71cb73e56a33d5ba2cc0ec9
DIFF: https://github.com/llvm/llvm-project/commit/edbb8a843c130e60d71cb73e56a33d5ba2cc0ec9.diff

LOG: [clang-format] Add return code to git-clang-format

https://github.com/llvm/llvm-project/issues/53220

Differential Revision: https://reviews.llvm.org/D117414

Added: 
    

Modified: 
    clang/tools/clang-format/git-clang-format

Removed: 
    


################################################################################
diff  --git a/clang/tools/clang-format/git-clang-format b/clang/tools/clang-format/git-clang-format
index 7838fd82c1dd6..7968c43875744 100755
--- a/clang/tools/clang-format/git-clang-format
+++ b/clang/tools/clang-format/git-clang-format
@@ -161,10 +161,12 @@ def main():
       print('Running clang-format on the following files:')
       for filename in changed_lines:
         print('    %s' % filename)
+
   if not changed_lines:
     if opts.verbose >= 0:
       print('no modified files to format')
-    return
+    return 0
+
   if len(commits) > 1:
     old_tree = commits[1]
     new_tree = run_clang_format_and_save_to_tree(changed_lines,
@@ -179,10 +181,13 @@ def main():
   if opts.verbose >= 1:
     print('old tree: %s' % old_tree)
     print('new tree: %s' % new_tree)
+
   if old_tree == new_tree:
     if opts.verbose >= 0:
       print('clang-format did not modify any files')
-  elif opts.
diff :
+    return 0
+
+  if opts.
diff :
     print_
diff (old_tree, new_tree)
   elif opts.
diff stat:
     print_
diff stat(old_tree, new_tree)
@@ -194,6 +199,8 @@ def main():
       for filename in changed_files:
         print('    %s' % filename)
 
+  return 1
+
 
 def load_git_config(non_string_options=None):
   """Return the git configuration as a dictionary.


        


More information about the cfe-commits mailing list