aaron.ballman added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/google-objc-require-category-method-prefixes.rst:41 + +.. option:: ExpectedPrefixes + ---------------- stephanemoore wrote: > This option seems to describe a list of class prefixes that are whitelisted. > If that is the case, perhaps `WhitelistedPrefixes` or > `WhitelistedClassPrefixes` would be a better name for the option? WDYT? No, please. We should try to avoid "white" and "black" lists in favor of more descriptive terms with less social connotations. I'd be fine with `ExpectedClassPrefixes`, `AllowedPrefixes`, `KnownPrefixes`, etc. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65917/new/ https://reviews.llvm.org/D65917 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
