HazardyKnusperkeks wrote:

> (Why drop the formatting test? Seems good to have an integration test for 
> this, no?)

Because the bug was a misannotation. The formatting, based on the annotations, 
should already be covered. Of course one can always add more tests, and I'm not 
demanding to remove it, I simply suggest it. The more test cases there are, the 
longer the duration of building and running the tests for all (clang-format) 
developers and CIs.

https://github.com/llvm/llvm-project/pull/124634
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to