Fixed two major bugs in paxos.cc.

[invirt/third/libt4.git] / threaded_log.h

diff --git a/threaded_log.h b/threaded_log.h
index 706e2b7..6630a86 100644 (file)
--- a/threaded_log.h
+++ b/threaded_log.h
@@ -11,8 +11,7 @@ extern int next_instance_num;
 extern char log_thread_prefix;
 
 namespace std {
-    // This is an awful hack.  But sticking this in std:: makes it possible for
-    // ostream_iterator to use it.
+    // Sticking this in std:: makes it possible for ostream_iterator to use it.
     template <class A, class B>
     ostream & operator<<(ostream &o, const pair<A,B> &d) {
         return o << "<" << d.first << "," << d.second << ">";
@@ -20,13 +19,13 @@ namespace std {
 }
 
 template <class A>
-typename enable_if<is_iterable<A>::value && !is_same<A,string>::value, ostream>::type &
+typename enable_if<is_const_iterable<A>::value && !is_same<A,string>::value, ostream>::type &
 operator<<(ostream &o, const A &a) {
     return o << "[" << implode(a, ", ") << "]";
 }
 
 #define LOG_PREFIX { \
-    auto _thread_ = std::this_thread::get_id(); \
+    auto _thread_ = this_thread::get_id(); \
     int _tid_ = thread_name_map[_thread_]; \
     if (_tid_==0) \
         _tid_ = thread_name_map[_thread_] = ++next_thread_num; \
@@ -39,7 +38,7 @@ operator<<(ostream &o, const A &a) {
     int _self_ = instance_name_map[this]; \
     if (_self_==0) \
         _self_ = instance_name_map[this] = ++next_instance_num; \
-    cerr << "#" << setw(2) << _self_; \
+    cerr << "#" << left << setw(2) << _self_ << " "; \
 }
 
 #define LOG_NONMEMBER(_x_) { \