EricWF added inline comments.

================
Comment at: test/std/strings/string.conversions/stold.pass.cpp:39
     assert(idx == 2);
+#ifndef TEST_HAS_NO_EXCEPTIONS
     try
----------------
Hmm. All of these `#ifdef`s get ugly fast. I think it might be better to add 
`TEST_TRY` and `TEST_CATCH(...)` macros defined like:

```
#ifndef TEST_HAS_NO_EXCEPTIONS
#define TEST_TRY try
#define TEST_CATCH(...) catch(__VA_ARGS__)
#else
#define TEST_TRY
#define TEST_CATCH(...) if (false)
#endif
```

What do you think? Feel free to add those macros to `test_macros.h` if you like 
the idea.



https://reviews.llvm.org/D26139



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

Reply via email to