[clang-tools-extra] [clang-doc] add nested namespace test case (PR #97681)

Paul Kirth via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 12 07:28:34 PDT 2024


https://github.com/ilovepi commented:

I think this mostly looks good, but I am concerned with the line number matching. It appears that whenever you’ve up’s the file you’ve had to update all the line numbers. 

Is it feasible to rewrite some of the checks to use the @LINE directive?  Perhaps it also makes sense to move those checks close to the lines they expect to match? Another thought is to leave the test as is, but drop the line numbers(or just match any number) and have a small set of check in line with the code that checks the line numbers with the LINE directive and uses a new prefix. 

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


More information about the cfe-commits mailing list