[clang-tools-extra] [clang-tidy] add new check readability-enum-initial-value (PR #86129)

Congcong Cai via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 27 18:38:51 PDT 2024


HerrCai0907 wrote:

> > Try getting rid of findNextTokenSkippingComments, and maybe just jump to begin() + token width
> 
> Do you think it is a good idea to use `ECD->getLocation().getLocWithOffset(ECD->getName().size())`?

It doesn't work for macro. I think the most precise way is to fix it is using `findNextTokenSkippingComments`. Why should we avoid to use `findNextTokenSkippingComments`? Is it for performance reasons?

https://github.com/llvm/llvm-project/pull/86129


More information about the cfe-commits mailing list