[PATCH] D140275: [clangd] Tweak to add doxygen comment to the function declaration

Oleg Skoromnik via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 19 01:07:14 PST 2022


tupos added a comment.

Also I tried to use it now, and I'm not sure where it is a good idea to allow this tweak from within a function body. Because it is quite annoying that in every function you get this code action.

On the other hand maybe it is ok to allow it from within the body, but the tweak then should be smarter and should check whether on the function declaration the doxygen comment is already present. Also if the tweak is smarter it is fine to get this code action, as it will motivate me to get rid of it and provide a proper documentation ;-).

What do you think?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140275



More information about the cfe-commits mailing list