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

mfherbst updated this revision to Diff 92986. mfherbst added a comment. Fixed typos (MainFile => MainSourceFile) https://reviews.llvm.org/D31326 Files: run-clang-tidy.py Index: run-clang-tidy.py === --- run-clang-tidy.py +++ ru

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

mfherbst updated this revision to Diff 105592. mfherbst added a comment. Herald added a subscriber: JDevlieghere. - Adapted patch to changes suggested by alexfh - Added comment why setting MainSourceFile to an empty string if fixes are exported https://reviews.llvm.org/D31326 Files: run-clan

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

mfherbst added inline comments. Comment at: run-clang-tidy.py:105 + elif merged['MainSourceFile'] != content['MainSourceFile']: +# The values given for MainSourceFile are inconsistent. +# Just empty MainSourceFile blank: alexfh wrote: > This

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

mfherbst updated this revision to Diff 105818. mfherbst added a comment. Correct nits and typos. https://reviews.llvm.org/D31326 Files: clang-tidy/tool/run-clang-tidy.py Index: clang-tidy/tool/run-clang-tidy.py === --- clang-ti

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

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 wrot

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

mfherbst updated this revision to Diff 105835. mfherbst added a comment. Correct the nits suggested by alexfh https://reviews.llvm.org/D31326 Files: clang-tidy/tool/run-clang-tidy.py Index: clang-tidy/tool/run-clang-tidy.py ===

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

mfherbst added a comment. I do not have commit rights to the svn if that's what you're asking, but I could send the patch to llvm-commits if that makes it easier for you. https://reviews.llvm.org/D31326 ___ cfe-commits mailing list cfe-commits@list

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

mfherbst updated this revision to Diff 107254. mfherbst added a comment. Adapt patch to the changes since the initial submission. Most notably the introduction of clang-tidy-4.0 changed the format in which clang-tidy dumps the required changes. This needs to be taken into account when merging t

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

mfherbst updated this revision to Diff 107633. mfherbst added a comment. Remove compatiblity with clang-tidy < 4.0.0 https://reviews.llvm.org/D31326 Files: run-clang-tidy.py Index: run-clang-tidy.py === --- run-clang-tidy.py +++

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

mfherbst added a comment. Many thanks! https://reviews.llvm.org/D31326 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits