MyDeveloperDay added inline comments.
================ Comment at: clang/test/ClangScanDeps/Inputs/header-search-pruning/cdb.json:1 -[ - { - "directory": "DIR", - "command": "clang -E DIR/header-search-pruning.cpp -Ibegin -I1 -Ia -I3 -I4 -I5 -I6 -Ib -I8 -Iend DEFINES -fmodules -fcxx-modules -fmodules-cache-path=DIR/module-cache -fimplicit-modules -fmodule-map-file=DIR/module.modulemap", - "file": "DIR/header-search-pruning.cpp" - } -] +[{ + "directory" : "DIR", ---------------- jansvoboda11 wrote: > ChuanqiXu wrote: > > jansvoboda11 wrote: > > > Please undo the whitespace changes in `ClangScanDeps` tests. > > It looks like that it's formatted by clang-format surprisingly. I would > > undo this manually. > Thanks. Just as a drive by assuming you are using a fairly recent clang-format, then clang-format should ONLY clang-format .json files if you have added the following code into your .clang-format ``` BasedOnStyle: LLVM ColumnLimit: 0 AlwaysBreakTemplateDeclarations: No Language: Cpp --- Language: Json BasedOnStyle: LLVM ``` Doing so would render like this ``` $ clang-format cdb.json [ { "directory": "DIR", "command": "clang -E DIR/header-search-pruning.cpp -Ibegin -I1 -Ia -I3 -I4 - I5 -I6 -Ib -I8 -Iend DEFINES -fmodules -fcxx-modules -fmodules-cache-path=DIR/mo dule-cache -fimplicit-modules -fmodule-map-file=DIR/module.modulemap", "file": "DIR/header-search-pruning.cpp" } ] ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113391/new/ https://reviews.llvm.org/D113391 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits