[PATCH] D108765: [docs] Fix documentation of clang-format BasedOnStyle type

Sun Aug 29 12:37:37 PDT 2021

HazardyKnusperkeks added inline comments.

================
Comment at: clang/docs/tools/dump_format_style.py:23
+  plurals = {
+    'IncludeCategory': 'IncludeCategories'
+  }
----------------
MyDeveloperDay wrote:
> HazardyKnusperkeks wrote:
> > FederAndInk wrote:
> > > HazardyKnusperkeks wrote:
> > > > Could you not just check if there is a y at the end and replace it with ies, otherweise add an s?
> > > Well, I thought about it, but then what about: whish -> whishes, leaf -> leaves, ... and irregulars? That's why I brought up the idea about using python inflect. Do you think it's enough for now to replace y -> ies and put an 's' to the others?
> > I'm okay with either way, in both cases there comes a time where someone must pay attention to add something here. We just have to look carefully in the review.
> it would be nice if in the event of a missing plural it complained.
But that would mean we have to add each plural here. So every new list, which is not of strings, would most likely also needed to be added here. I think that is too much, but then again there are not that much lists, so maybe it's worth the work.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108765/new/

https://reviews.llvm.org/D108765