ioeric added inline comments.

================
Comment at: clangd/Quality.h:74
     Keyword,
+    Operator,
   } Category = Unknown;
----------------
hokein wrote:
> Maybe name it `OverloadedOperator` to avoid confusion with other 
> non-overloaded operators like `?:`
Ternary operator is not a symbol AFAIK?  It's an expression. Even if it's an 
operator, I think we would still want to categorize it as operator. 


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55061



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

Reply via email to