stephanemoore added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/google-objc-require-category-method-prefixes.rst:41 + +.. option:: ExpectedPrefixes + ---------------- aaron.ballman wrote: > 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. Makes sense. Would `ExemptClassPrefixes` work? 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