xazax.hun added a comment. In https://reviews.llvm.org/D38171#927046, @alexfh wrote:
> And, btw, sorry for the long delay. I've been on travelling / on vacation for > the last few weeks. No problem. Will you have some time to think about the overall concept? Implementation and test wise it looks good to me. I think this patch is a step in a good direction but of course, it is important for the community to agree on the approach. ================ Comment at: test/clang-tidy/warning-check-aliases.cpp:10 + try { + + } catch (B &X) { ---------------- There are some redundant empty lines. https://reviews.llvm.org/D38171 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits