[PATCH] D53638: [clangd] Downrank members from base class

[Sam McCall via Phabricator via cfe-commits]

sammccall added inline comments.


================
Comment at: clangd/Quality.cpp:380
+  if (InBaseClass)
+    Score *= 0.7;
+
----------------
ioeric wrote:
> sammccall wrote:
> > This seems like a pretty light penalty to me, I'd consider 0.5...
> 0.5 sounds reasonable. I think we should penalize the non-instance member case more if this is 0.5 though. Made that 0.2. WDYT?
Agree, LG!


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53638