[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