[PATCH] D26139: Tests for strings conversions under libcpp-no-exceptions

Asiri Rathnayake via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 6 01:43:50 PST 2016


rmaprath added a comment.

In https://reviews.llvm.org/D26139#587587, @mclow.lists wrote:

> >   I think it might be better to add TEST_TRY and TEST_CATCH(...) macros defined like
>
> @rogfer01 said at the top that he didn't want to add "a magical TEST_TRY macro" - and I agree.  Someone tried that in another review, and I nixed it there.


I had a proposal for something along those lines in https://reviews.llvm.org/D14653, however those TEST_TRY and TEST_CATCH macros were trying to be "too smart".

I agree, having macros for `try` and `catch` sounds to me like it would set the wrong precedent.


https://reviews.llvm.org/D26139





More information about the cfe-commits mailing list