base_cpp: removed cppcheck warnings

d59cff75 · Marcus Winter · 4a3b52b0 · d59cff75 · d59cff75
Commit d59cff75 authored Jan 19, 2015 by Marcus Winter
Show whitespace changes
Inline Side-by-side

Showing with 1 additions and 3 deletions

base_cpp/test/atomic_test.cc
+0 -2

base_cpp/test/mutex_test.cc
+1 -1

No files found.
--- a/base_cpp/test/atomic_test.cc
+++ b/base_cpp/test/atomic_test.cc
@@ -346,8 +346,6 @@ void AtomicTest::BasicTests() {
  //Boolean operations...
  // And
  i = 0;
-  i &= 0;
-  PT_EXPECT(i == 0);
  i &= 1;
  PT_EXPECT(i == 0);
  i = 1;

--- a/base_cpp/test/mutex_test.cc
+++ b/base_cpp/test/mutex_test.cc
@@ -93,10 +93,10 @@ void MutexTest::PostLockGuardCount() {
 }

 void MutexTest::TestUniqueLock() {
+  { // Test standard usage and releasing
 #ifdef EMBB_USE_EXCEPTIONS
    bool exception_thrown = false;
 #endif
-  { // Test standard usage and releasing
    UniqueLock<Mutex> lock(mutex_);
    PT_EXPECT_EQ(lock.OwnsLock(), true);