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

Philipp via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 3 05:36:54 PDT 2017


phst accepted this revision.
phst added inline comments.


================
Comment at: tools/clang-format/clang-format.el:126
+no active region. If no style is given uses `clang-format-style'. Use
+ASSUME-FILE-NAME to locate a style config file, if no  assume-file-name is given
+uses the function `buffer-file-name'."
----------------
nit: write arguments in docstrings always in caps: ASSUME-FILE-NAME


================
Comment at: tools/clang-format/clang-format.el:197
+If no style is given uses `clang-format-style'. Use ASSUME-FILE-NAME
+to locate a style config file. If no assume-file-name is given uses
+the function `buffer-file-name'."
----------------
Write the argument in the docstring as ASSUME-FILE-NAME


https://reviews.llvm.org/D37903





More information about the cfe-commits mailing list