thieta added a comment.

In D125961#3527447 <https://reviews.llvm.org/D125961#3527447>, @curdeius wrote:

> Thanks for the patch!
> Could you please add a unit test in unittest/Format/FormatTest.cpp instead of 
> in lit-based test/?
> Is there a bug report that your patch fixes?

Sure - I didn't know clang-format used unittests instead. Maybe there should be 
some documentation around this for new people used to adding lit tests?

I never filed a bug for it - I fixed it instead.

Thanks


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125961

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to