Lgtm<br><div class="gmail_quote"><div dir="ltr">On Sat, Feb 10, 2018 at 12:16 PM Aaron Smith via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">asmith created this revision.<br>
asmith added reviewers: zturner, llvm-commits.<br>
Herald added subscribers: cfe-commits, klimek.<br>
<br>
This test would fail if the python path had spaces. Add a quote around the path to fix this problem and update some test values changed by the addition of quotes around the path.<br>
<br>
Tested on Windows and Linux with Python 3.x<br>
<br>
<br>
Repository:<br>
  rC Clang<br>
<br>
<a href="https://reviews.llvm.org/D43164" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43164</a><br>
<br>
Files:<br>
  test/Tooling/clang-diff-json.cpp<br>
<br>
<br>
Index: test/Tooling/clang-diff-json.cpp<br>
===================================================================<br>
--- test/Tooling/clang-diff-json.cpp<br>
+++ test/Tooling/clang-diff-json.cpp<br>
@@ -1,10 +1,10 @@<br>
 // RUN: clang-diff -ast-dump-json %s -- \<br>
-// RUN: | %python -c 'import json, sys; json.dump(json.loads(sys.stdin.read()), sys.stdout, sort_keys=True, indent=2)' \<br>
+// RUN: | '%python' -c 'import json, sys; json.dump(json.loads(sys.stdin.read()), sys.stdout, sort_keys=True, indent=2)' \<br>
 // RUN: | FileCheck %s<br>
<br>
-// CHECK: "begin": 299,<br>
+// CHECK: "begin": 301,<br>
 // CHECK: "type": "FieldDecl",<br>
-// CHECK: "end": 319,<br>
+// CHECK: "end": 321,<br>
 // CHECK: "type": "CXXRecordDecl",<br>
 class A {<br>
   int x;<br>
<br>
<br>
</blockquote></div>