[libcxx] r220154 - Fix unused variables in tests to placate scan-build. Patch from Steve MacKenzie.

Eric Fiselier eric at efcs.ca
Sat Oct 18 17:10:15 PDT 2014


Author: ericwf
Date: Sat Oct 18 19:10:15 2014
New Revision: 220154

URL: http://llvm.org/viewvc/llvm-project?rev=220154&view=rev
Log:
Fix unused variables in tests to placate scan-build. Patch from Steve MacKenzie.

Modified:
    libcxx/trunk/test/atomics/atomics.types.generic/bool.pass.cpp
    libcxx/trunk/test/containers/associative/multimap/multimap.modifiers/erase_key.pass.cpp
    libcxx/trunk/test/containers/sequences/array/array.data/data.pass.cpp
    libcxx/trunk/test/containers/sequences/array/array.data/data_const.pass.cpp
    libcxx/trunk/test/depr/depr.c.headers/tgmath_h.pass.cpp

Modified: libcxx/trunk/test/atomics/atomics.types.generic/bool.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/atomics/atomics.types.generic/bool.pass.cpp?rev=220154&r1=220153&r2=220154&view=diff
==============================================================================
--- libcxx/trunk/test/atomics/atomics.types.generic/bool.pass.cpp (original)
+++ libcxx/trunk/test/atomics/atomics.types.generic/bool.pass.cpp Sat Oct 18 19:10:15 2014
@@ -68,6 +68,7 @@ int main()
         std::atomic_init(&obj, true);
         assert(obj == true);
         bool b0 = obj.is_lock_free();
+        (void)b0; // to placate scan-build
         obj.store(false);
         assert(obj == false);
         obj.store(true, std::memory_order_release);
@@ -123,6 +124,7 @@ int main()
         std::atomic_init(&obj, true);
         assert(obj == true);
         bool b0 = obj.is_lock_free();
+        (void)b0; // to placate scan-build
         obj.store(false);
         assert(obj == false);
         obj.store(true, std::memory_order_release);
@@ -178,6 +180,7 @@ int main()
         std::atomic_init(&obj, true);
         assert(obj == true);
         bool b0 = obj.is_lock_free();
+        (void)b0; // to placate scan-build
         obj.store(false);
         assert(obj == false);
         obj.store(true, std::memory_order_release);

Modified: libcxx/trunk/test/containers/associative/multimap/multimap.modifiers/erase_key.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/containers/associative/multimap/multimap.modifiers/erase_key.pass.cpp?rev=220154&r1=220153&r2=220154&view=diff
==============================================================================
--- libcxx/trunk/test/containers/associative/multimap/multimap.modifiers/erase_key.pass.cpp (original)
+++ libcxx/trunk/test/containers/associative/multimap/multimap.modifiers/erase_key.pass.cpp Sat Oct 18 19:10:15 2014
@@ -71,6 +71,7 @@ int main()
         assert(next(m.begin(), 5)->second == 2);
 
         i = m.erase(3);
+        assert(i == 3);
         assert(m.size() == 3);
         assert(next(m.begin(), 0)->first == 1);
         assert(next(m.begin(), 0)->second == 1);
@@ -135,6 +136,7 @@ int main()
         assert(next(m.begin(), 5)->second == 2);
 
         i = m.erase(3);
+        assert(i == 3);
         assert(m.size() == 3);
         assert(next(m.begin(), 0)->first == 1);
         assert(next(m.begin(), 0)->second == 1);

Modified: libcxx/trunk/test/containers/sequences/array/array.data/data.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/containers/sequences/array/array.data/data.pass.cpp?rev=220154&r1=220153&r2=220154&view=diff
==============================================================================
--- libcxx/trunk/test/containers/sequences/array/array.data/data.pass.cpp (original)
+++ libcxx/trunk/test/containers/sequences/array/array.data/data.pass.cpp Sat Oct 18 19:10:15 2014
@@ -30,5 +30,6 @@ int main()
         typedef std::array<T, 0> C;
         C c = {};
         T* p = c.data();
+        (void)p; // to placate scan-build
     }
 }

Modified: libcxx/trunk/test/containers/sequences/array/array.data/data_const.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/containers/sequences/array/array.data/data_const.pass.cpp?rev=220154&r1=220153&r2=220154&view=diff
==============================================================================
--- libcxx/trunk/test/containers/sequences/array/array.data/data_const.pass.cpp (original)
+++ libcxx/trunk/test/containers/sequences/array/array.data/data_const.pass.cpp Sat Oct 18 19:10:15 2014
@@ -30,5 +30,6 @@ int main()
         typedef std::array<T, 0> C;
         const C c = {};
         const T* p = c.data();
+        (void)p; // to placate scan-build
     }
 }

Modified: libcxx/trunk/test/depr/depr.c.headers/tgmath_h.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/depr/depr.c.headers/tgmath_h.pass.cpp?rev=220154&r1=220153&r2=220154&view=diff
==============================================================================
--- libcxx/trunk/test/depr/depr.c.headers/tgmath_h.pass.cpp (original)
+++ libcxx/trunk/test/depr/depr.c.headers/tgmath_h.pass.cpp Sat Oct 18 19:10:15 2014
@@ -19,4 +19,5 @@ int main()
 {
     std::complex<double> cd;
     double x = sin(1.0);
+    (void)x; // to placate scan-build
 }





More information about the cfe-commits mailing list