Browse Source

[trac602] addressed jinmei's comments

Jelte Jansen 14 years ago
parent
commit
27025d831a
3 changed files with 9 additions and 10 deletions
  1. 0 1
      configure.ac
  2. 1 1
      src/lib/nsas/hash_table.h
  3. 8 8
      src/lib/nsas/locks.h

+ 0 - 1
configure.ac

@@ -471,7 +471,6 @@ if test "${use_boost_threads}" = "yes" ; then
     boost::mutex m;
     ],
         [ AC_MSG_RESULT(yes (without libboost_thread)) ],
-    
         # there is one specific problem with SunStudio 5.10
         # where including boost/thread causes a compilation failure
         # There is a workaround in boost but it checks the version not being 5.10

+ 1 - 1
src/lib/nsas/hash_table.h

@@ -121,7 +121,7 @@ public:
     isc::locks::scoped_lock<typename HashTableSlot<T>::mutex_type>
     scoped_lock;                    ///< Type for a scope-limited write-lock
     //@}
-    
+
     /// \brief Constructor
     ///
     /// Initialises the hash table.

+ 8 - 8
src/lib/nsas/locks.h

@@ -33,7 +33,7 @@
 /// (TODO2)
 
 #ifndef __LOCKS_
-#define __LOCKS_ 
+#define __LOCKS_
 
 #ifndef USE_BOOST_THREADS
 
@@ -52,16 +52,16 @@ class upgradable_mutex {
 template <typename T> 
 class sharable_lock {
 public:
-    sharable_lock(T mtype) { (void) mtype; }
+    sharable_lock(T) { }
 };
 
-template <typename T> 
+template <typename T>
 class scoped_lock {
 public:
-    scoped_lock(T mtype) { (void) mtype; }
-    
-    void lock() {};
-    void unlock() {};
+    scoped_lock(T) { }
+
+    void lock() {}
+    void unlock() {}
 };
 
 }
@@ -104,7 +104,7 @@ public:
 template <class T> 
 struct scoped_lock : public boost::interprocess::scoped_lock<T> {
 public:
-    scoped_lock(T& mtype) : boost::interprocess::scoped_lock<T>(mtype) { (void) mtype; }
+    scoped_lock(T& mtype) : boost::interprocess::scoped_lock<T>(mtype) { }
 };
 
 }