[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