Add parameter to format-check any commit

Adds a parameter to check-last-commit, so we can check either
HEAD~<n> or an actual git hash id.
Still default to the last commit HEAD~0.

Change-Id: Iadcbadddb31c2887c3e0395ceef34b7c006967e4

1 files changed, 17 insertions, 1 deletions

diff --git a/solenv/clang-format/check-last-commit b/solenv/clang-format/check-last-commit
index 2db666ab2aec..57939815bb0a 100755
--- a/solenv/clang-format/check-last-commit
+++ b/solenv/clang-format/check-last-commit
@@ -12,6 +12,9 @@ use warnings;
 use lib "solenv/clang-format";
 use ClangFormat;
 
+my $commit = 0;
+my $commit_id = 'HEAD';
+
 sub check_style()
 {
     if ( ! -e ".git" )
@@ -25,12 +28,13 @@ sub check_style()
     my @bad_names = ();
     my $blacklist_names = ClangFormat::get_blacklist();
     my $clang_format = ClangFormat::find();
+    my $parent = $commit + 1;
 
     # Get a list of non-deleted changed files.
     # Explicitly use the low-level 'git diff-tree' (rather than plain 'git
     # diff') so we get the new, but not the old files for renames and/or
     # copies.
-    open (FILES, "git diff-tree -r --diff-filter=AM --name-only HEAD^ HEAD |") ||  die "Cannot run git diff.";
+    open (FILES, "git diff-tree -r --diff-filter=AM --name-only ${commit_id}~${parent} ${commit_id}~${commit} |") ||  die "Cannot run git diff.";
     while (my $filename = <FILES>)
     {
         chomp $filename;
@@ -75,6 +79,18 @@ sub check_style()
     }
 }
 
+if (scalar(@ARGV) == 1)
+{
+    if (($ARGV[0] !~ /^[0-9]+$/) || (scalar($ARGV[0]) >= 8))
+    {
+        $commit_id = $ARGV[0];
+    }
+    else
+    {
+        $commit = $ARGV[0];
+    }
+}
+
 check_style();
 
 exit(0);