[PATCH] D136925: [clangd] Index unscoped enums inside classes for code completion

Tom Praschan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 2 03:53:54 PDT 2022


tom-anders added a comment.

In D136925#3901509 <https://reviews.llvm.org/D136925#3901509>, @nridge wrote:

> Thanks!
>
> I don't think we need this much detail in the commit message; I think after the first line it would be sufficient to say:
>
>   Fixes https://github.com/clangd/clangd/issues/1082
>   See the issue for disk/memory usage measurements
>
> (Note also that "Fixes <issue>" triggers the github issue to be closed when the commit is merged.)
>
> I forget whether you have commit access -- let me know if you need me to merge the patch for you.

Yep, I do have access. I adjusted the message and pushed this, but the corresponding Github issue is not closed automatically because I don't have the necessary permisions (I think we already hat that problem in another patch)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136925



More information about the cfe-commits mailing list