whisperity added a comment. In D97297#2583391 <https://reviews.llvm.org/D97297#2583391>, @Eugene.Zelenko wrote:
> Please mention new option in Release Notes. The base check is not yet accepted or in. I plan to land the entire thing in one go once everything is accepted because a partial landing would result in a useless check (in one way or another). ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone-easily-swappable-parameters.rst:89 + warning about the two parameters are silenced. + Defaults to ``1``. + Might be any positive integer. ---------------- Eugene.Zelenko wrote: > Please use single back-ticks for option values. Same below. **All** values (including above the ``_Bool`` and such, or only the numeric constants? The ``LHS`` and ``RHS`` below are examples of source code, not a valid value for the option. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97297/new/ https://reviews.llvm.org/D97297 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
