[PATCH] D37903: Fix assume-filename handling in clang-format.el

Philipp via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 19 06:51:05 PDT 2017


phst accepted this revision.
phst added a comment.
This revision is now accepted and ready to land.

Your use case sounds good to me. Please be sure to document the new parameter, though.



================
Comment at: tools/clang-format/clang-format.el:154
+                             `("-output-replacements-xml"
+                               ,@(and assume-file (list "-assume-filename" assume-file))
+                               "-style" ,style
----------------
Please add a comment referencing the LLVM bug you've just filed so that others aren't surprised about this.


https://reviews.llvm.org/D37903





More information about the cfe-commits mailing list