EricWF requested changes to this revision.
EricWF added a comment.
This revision now requires changes to proceed.

One inline change requested.


================
Comment at: 
test/std/localization/locale.categories/category.numeric/locale.nm.put/facet.num.put.members/put_long_double.pass.cpp:24421
@@ -24420,2 +24420,3 @@
     std::locale lc = std::locale::classic();
     std::locale lg(lc, new my_numpunct);
+    ((void)str); // Prevent unused warning
----------------
Isn't this unused as well on non-apple platforms?

================
Comment at: 
test/std/localization/locale.categories/category.numeric/locale.nm.put/facet.num.put.members/put_long_double.pass.cpp:24422
@@ -24421,2 +24421,3 @@
     std::locale lg(lc, new my_numpunct);
+    ((void)str); // Prevent unused warning
 #ifdef __APPLE__
----------------
Move  `str` inside the `#ifdef __APPLE__` instead.

================
Comment at: test/std/numerics/rand/rand.device/eval.pass.cpp:33
@@ -32,3 +32,3 @@
     }
-    catch (const std::system_error& e)
+    catch (const std::system_error&)
     {
----------------
This looks OK, but I noticed that *technically* the spec only says the 
exception type is derived from `std::exception`.

However if we can keep testing this exception without bothering anybody I say 
we do.


http://reviews.llvm.org/D19625



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

Reply via email to