https://github.com/AaronBallman commented:
Thanks for the PR! I think we need to think about this a little bit; my concern is that folks often use clang-query to write queries they'll eventually be using in clang-tidy checks. However, the tidy checks are using the C++ DSL which won't support trailing commas (and I don't think could be made to support them). So while it might make things easier in some cases, it may be a surprising difference between the tools. WDYT? If we go forward with this, we should add a release note to `clang-tools-extra/docs/ReleaseNotes.rst`. We don't currently have any docs for clang-query, but if we had those docs, we should probably note the difference between the C++ DSL and clang-query in them. (That's not something you have to worry about for your patch though.) https://github.com/llvm/llvm-project/pull/148018 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits