[PATCH] D82620: [clang-format] Preserve whitespace in selected macros

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 3 09:06:47 PDT 2020


MyDeveloperDay added inline comments.


================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2706
+
+  For example: STRINGIZE
+
----------------
JakeMerdichAMD wrote:
> curdeius wrote:
> > Shouldn't there be a configuration example like what's in `ForEachMacros` doc?
> > ```
> >   In the .clang-format configuration file, this can be configured like:
> > 
> >   .. code-block:: yaml
> > 
> >     WhitespaceSensitiveMacros: ['STRINGIZE', 'PP_STRINGIZE']
> > 
> >   For example: BOOST_PP_STRINGIZE.
> > ```
> > 
> Done. I also added PP_STRINGIZE and BOOST_PP_STRINGIZE as defaults; seems reasonable.
The change I think you made here gets lost when the ClangFormatOptions.rst file is regenerated

Could you update the `Format.h` and regenerate rather than making the code changes in the rst itself?

I ran into this issue when I tried to regenerate.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82620/new/

https://reviews.llvm.org/D82620





More information about the cfe-commits mailing list