[PATCH] D31326: Add option to export fixes to run-clang-tidy.py

Michael F. Herbst via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 10 01:25:43 PDT 2017


mfherbst added inline comments.


================
Comment at: run-clang-tidy.py:93
+def merge_replacement_files(tmpdir, fixfile):
+  """Merge all replacement files in a directory into a single fixfile"""
+  # MainSourceFile: The key is required by the definition inside
----------------
alexfh wrote:
> I'm not sure "fixfile" is a word. Just "file" maybe?
I thought I would refer to the name of the input argument, but I guess you are right, file makes it less clunky to read.


https://reviews.llvm.org/D31326





More information about the cfe-commits mailing list