HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.
This revision is now accepted and ready to land.

Looks good and I agree with the choices.



================
Comment at: clang/docs/tools/dump_format_style.py:23
+  plurals = {
+    'IncludeCategory': 'IncludeCategories'
+  }
----------------
Could you not just check if there is a y at the end and replace it with ies, 
otherweise add an s?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108765

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to