Replaced C-style casts with C++ stype casts in C++ codes.

cpp11/define.hpp

@@ -121,7 +121,7 @@ struct define {
 	void msgpack_object(msgpack::object* o, msgpack::zone* z) const
 	{
 		o->type = type::ARRAY;
-		o->via.array.ptr = (object*)z->malloc(sizeof(object)*sizeof...(Args));
+		o->via.array.ptr = static_cast<object*>(z->malloc(sizeof(object)*sizeof...(Args)));
 		o->via.array.size = sizeof...(Args);
 
 		define_imp<tuple<Args&...>, sizeof...(Args)>::object(o, z, a);

cpp11/tuple.hpp

@@ -108,7 +108,7 @@ inline void operator<< (
 		object::with_zone& o,
 		type::tuple<Args...>& v) {
 	o.type = type::ARRAY;
-	o.via.array.ptr = (object*)o.zone->malloc(sizeof(object)*sizeof...(Args));
+	o.via.array.ptr = static_cast<object*>(o.zone->malloc(sizeof(object)*sizeof...(Args)));
 	o.via.array.size = sizeof...(Args);
 	TupleToObjectWithZone<decltype(v), sizeof...(Args)>::convert(o, v);
 }

cpp11/zone.hpp

@@ -80,7 +80,7 @@ private:
 				nnext = nused * 2;
 			}
 			finalizer* tmp =
-				(finalizer*)::realloc(array_, sizeof(finalizer) * nnext);
+				static_cast<finalizer*>(::realloc(array_, sizeof(finalizer) * nnext));
 			if(!tmp) {
 				throw std::bad_alloc();
 			}
@@ -101,14 +101,14 @@ private:
 	struct chunk_list {
 		chunk_list(size_t chunk_size)
 		{
-			chunk* c = (chunk*)::malloc(sizeof(chunk) + chunk_size);
+			chunk* c = static_cast<chunk*>(::malloc(sizeof(chunk) + chunk_size));
 			if(!c) {
 				throw std::bad_alloc();
 			}
 
 			head_ = c;
 			free_ = chunk_size;
-			ptr_  = ((char*)c) + sizeof(chunk);
+			ptr_  = reinterpret_cast<char*>(c) + sizeof(chunk);
 			c->next_ = nullptr;
 		}
 		~chunk_list()
@@ -138,7 +138,7 @@ private:
 			}
 			head_->next_ = nullptr;
 			free_ = chunk_size;
-			ptr_  = ((char*)head_) + sizeof(chunk);
+			ptr_  = reinterpret_cast<char*>(head_) + sizeof(chunk);
 		}
 		size_t free_;
 		char* ptr_;
@@ -181,7 +181,7 @@ private:
 
 inline zone* zone::create(size_t chunk_size)
 {
-	zone* z = (zone*)::malloc(sizeof(zone) + chunk_size);
+	zone* z = static_cast<zone*>(::malloc(sizeof(zone) + chunk_size));
 	if (!z) {
 		return nullptr;
 	}
@@ -234,9 +234,9 @@ inline void* zone::malloc_expand(size_t size)
 		sz *= 2;
 	}
 
-	chunk* c = (chunk*)::malloc(sizeof(chunk) + sz);
+	chunk* c = static_cast<chunk*>(::malloc(sizeof(chunk) + sz));
 
-	char* ptr = ((char*)c) + sizeof(chunk);
+	char* ptr = reinterpret_cast<char*>(c) + sizeof(chunk);
 
 	c->next_  = cl->head_;
 	cl->head_ = c;

src/msgpack/sbuffer.hpp

@@ -98,7 +98,7 @@ private:
 			throw std::bad_alloc();
 		}
 	
-		data_ = (char*)tmp;
+		data_ = static_cast<char*>(tmp);
 		alloc_ = nsize;
 	}
 

src/msgpack/type/define.hpp.erb

@@ -118,7 +118,7 @@ struct define<A0<%1.upto(i) {|j|%>, A<%=j%><%}%>> {
 	void msgpack_object(msgpack::object* o, msgpack::zone* z) const
 	{
 		o->type = type::ARRAY;
-		o->via.array.ptr = (object*)z->malloc(sizeof(object)*<%=i+1%>);
+		o->via.array.ptr = static_cast<object*>(z->malloc(sizeof(object)*<%=i+1%>));
 		o->via.array.size = <%=i+1%>;
 		<%0.upto(i) {|j|%>
 		o->via.array.ptr[<%=j%>] = object(a<%=j%>, z);<%}%>

src/msgpack/type/deque.hpp

@@ -57,7 +57,7 @@ inline void operator<< (object::with_zone& o, const std::deque<T>& v)
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();

src/msgpack/type/list.hpp

@@ -57,7 +57,7 @@ inline void operator<< (object::with_zone& o, const std::list<T>& v)
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();

src/msgpack/type/map.hpp

@@ -78,7 +78,7 @@ inline void operator<< (object::with_zone& o, const type::assoc_vector<K,V>& v)
 		o.via.map.ptr  = NULL;
 		o.via.map.size = 0;
 	} else {
-		object_kv* p = (object_kv*)o.zone->malloc(sizeof(object_kv)*v.size());
+		object_kv* p = static_cast<object_kv*>(o.zone->malloc(sizeof(object_kv)*v.size()));
 		object_kv* const pend = p + v.size();
 		o.via.map.ptr  = p;
 		o.via.map.size = v.size();
@@ -134,7 +134,7 @@ inline void operator<< (object::with_zone& o, const std::map<K,V>& v)
 		o.via.map.ptr  = NULL;
 		o.via.map.size = 0;
 	} else {
-		object_kv* p = (object_kv*)o.zone->malloc(sizeof(object_kv)*v.size());
+		object_kv* p = static_cast<object_kv*>(o.zone->malloc(sizeof(object_kv)*v.size()));
 		object_kv* const pend = p + v.size();
 		o.via.map.ptr  = p;
 		o.via.map.size = v.size();
@@ -184,7 +184,7 @@ inline void operator<< (object::with_zone& o, const std::multimap<K,V>& v)
 		o.via.map.ptr  = NULL;
 		o.via.map.size = 0;
 	} else {
-		object_kv* p = (object_kv*)o.zone->malloc(sizeof(object_kv)*v.size());
+		object_kv* p = static_cast<object_kv*>(o.zone->malloc(sizeof(object_kv)*v.size()));
 		object_kv* const pend = p + v.size();
 		o.via.map.ptr  = p;
 		o.via.map.size = v.size();

src/msgpack/type/pair.hpp

@@ -47,7 +47,7 @@ template <typename T1, typename T2>
 inline void operator<< (object::with_zone& o, const std::pair<T1, T2>& v)
 {
 	o.type = type::ARRAY;
-	object* p = (object*)o.zone->malloc(sizeof(object)*2);
+	object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*2));
 	o.via.array.ptr = p;
 	o.via.array.size = 2;
 	p[0] = object(v.first, o.zone);

src/msgpack/type/set.hpp

@@ -56,7 +56,7 @@ inline void operator<< (object::with_zone& o, const std::set<T>& v)
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();
@@ -102,7 +102,7 @@ inline void operator<< (object::with_zone& o, const std::multiset<T>& v)
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();

src/msgpack/type/string.hpp

@@ -42,7 +42,7 @@ inline packer<Stream>& operator<< (packer<Stream>& o, const std::string& v)
 inline void operator<< (object::with_zone& o, const std::string& v)
 {
 	o.type = type::RAW;
-	char* ptr = (char*)o.zone->malloc(v.size());
+	char* ptr = static_cast<char*>(o.zone->malloc(v.size()));
 	o.via.raw.ptr = ptr;
 	o.via.raw.size = (uint32_t)v.size();
 	memcpy(ptr, v.data(), v.size());

src/msgpack/type/tr1/unordered_map.hpp

@@ -77,7 +77,7 @@ inline void operator<< (object::with_zone& o, const MSGPACK_STD_TR1::unordered_m
 		o.via.map.ptr  = NULL;
 		o.via.map.size = 0;
 	} else {
-		object_kv* p = (object_kv*)o.zone->malloc(sizeof(object_kv)*v.size());
+		object_kv* p = static_cast<object_kv*>(o.zone->malloc(sizeof(object_kv)*v.size()));
 		object_kv* const pend = p + v.size();
 		o.via.map.ptr  = p;
 		o.via.map.size = v.size();
@@ -127,7 +127,7 @@ inline void operator<< (object::with_zone& o, const MSGPACK_STD_TR1::unordered_m
 		o.via.map.ptr  = NULL;
 		o.via.map.size = 0;
 	} else {
-		object_kv* p = (object_kv*)o.zone->malloc(sizeof(object_kv)*v.size());
+		object_kv* p = static_cast<object_kv*>(o.zone->malloc(sizeof(object_kv)*v.size()));
 		object_kv* const pend = p + v.size();
 		o.via.map.ptr  = p;
 		o.via.map.size = v.size();

src/msgpack/type/tr1/unordered_set.hpp

@@ -74,7 +74,7 @@ inline void operator<< (object::with_zone& o, const MSGPACK_STD_TR1::unordered_s
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();
@@ -120,7 +120,7 @@ inline void operator<< (object::with_zone& o, const MSGPACK_STD_TR1::unordered_m
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();

src/msgpack/type/tuple.hpp.erb

@@ -184,7 +184,7 @@ inline void operator<< (
 		object::with_zone& o,
 		const type::tuple<A0<%1.upto(i) {|j|%>, A<%=j%><%}%>>& v) {
 	o.type = type::ARRAY;
-	o.via.array.ptr = (object*)o.zone->malloc(sizeof(object)*<%=i+1%>);
+	o.via.array.ptr = static_cast<object*>(o.zone->malloc(sizeof(object)*<%=i+1%>));
 	o.via.array.size = <%=i+1%>;
 	<%0.upto(i) {|j|%>
 	o.via.array.ptr[<%=j%>] = object(v.template get<<%=j%>>(), o.zone);<%}%>

src/msgpack/type/vector.hpp

@@ -61,7 +61,7 @@ inline void operator<< (object::with_zone& o, const std::vector<T>& v)
 		o.via.array.ptr = NULL;
 		o.via.array.size = 0;
 	} else {
-		object* p = (object*)o.zone->malloc(sizeof(object)*v.size());
+		object* p = static_cast<object*>(o.zone->malloc(sizeof(object)*v.size()));
 		object* const pend = p + v.size();
 		o.via.array.ptr = p;
 		o.via.array.size = v.size();

src/msgpack/vrefbuffer.hpp

@@ -62,8 +62,8 @@ public:
 		size_t nfirst = (sizeof(iovec) < 72/2) ?
 			72 / sizeof(iovec) : 8;
 
-		iovec* array = (iovec*)::malloc(
+		iovec* array = static_cast<iovec*>(::malloc(
-			sizeof(iovec) * nfirst);
+			sizeof(iovec) * nfirst));
 		if(!array) {
 			throw std::bad_alloc();
 		}
@@ -72,7 +72,7 @@ public:
 		end_   = array + nfirst;
 		array_ = array;
 
-		chunk* c = (chunk*)::malloc(sizeof(chunk) + chunk_size);
+		chunk* c = static_cast<chunk*>(::malloc(sizeof(chunk) + chunk_size));
 		if(!c) {
 			::free(array);
 			throw std::bad_alloc();
@@ -80,7 +80,7 @@ public:
 		inner_buffer* const ib = &inner_buffer_;
 
 		ib->free = chunk_size;
-		ib->ptr	 = ((char*)c) + sizeof(chunk);
+		ib->ptr	 = reinterpret_cast<char*>(c) + sizeof(chunk);
 		ib->head = c;
 		c->next = nullptr;
 
@@ -117,8 +117,8 @@ public:
 			const size_t nused = tail_ - array_;
 			const size_t nnext = nused * 2;
 
-			iovec* nvec = (iovec*)::realloc(
+			iovec* nvec = static_cast<iovec*>(::realloc(
-				array_, sizeof(iovec)*nnext);
+				array_, sizeof(iovec)*nnext));
 			if(!nvec) {
 				throw std::bad_alloc();
 			}
@@ -143,7 +143,7 @@ public:
 				sz = len;
 			}
 
-			chunk* c = (chunk*)::malloc(sizeof(chunk) + sz);
+			chunk* c = static_cast<chunk*>(::malloc(sizeof(chunk) + sz));
 			if(!c) {
 				throw std::bad_alloc();
 			}
@@ -151,7 +151,7 @@ public:
 			c->next = ib->head;
 			ib->head = c;
 			ib->free = sz;
-			ib->ptr	 = ((char*)c) + sizeof(chunk);
+			ib->ptr	 = reinterpret_cast<char*>(c) + sizeof(chunk);
 		}
 
 		char* m = ib->ptr;
@@ -160,7 +160,9 @@ public:
 		ib->ptr	 += len;
 
 		if(tail_ != array_ && m ==
-			(const char*)((tail_ - 1)->iov_base) + (tail_ - 1)->iov_len) {
+			static_cast<const char*>(
+				const_cast<const void *>((tail_ - 1)->iov_base)
+			) + (tail_ - 1)->iov_len) {
 			(tail_ - 1)->iov_len += len;
 			return;
 		} else {
@@ -182,7 +184,7 @@ public:
 	{
 		size_t sz = chunk_size_;
 
-		chunk* empty = (chunk*)::malloc(sizeof(chunk) + sz);
+		chunk* empty = static_cast<chunk*>(::malloc(sizeof(chunk) + sz));
 		if(!empty) {
 			throw std::bad_alloc();
 		}
@@ -198,8 +200,8 @@ public:
 				nnext *= 2;
 			}
 
-			iovec* nvec = (iovec*)::realloc(
+			iovec* nvec = static_cast<iovec*>(::realloc(
-				to->array_, sizeof(iovec)*nnext);
+				to->array_, sizeof(iovec)*nnext));
 			if(!nvec) {
 				::free(empty);
 				throw std::bad_alloc();
@@ -233,7 +235,7 @@ public:
 
 		ib->head = empty;
 		ib->free = sz;
-		ib->ptr	 = ((char*)empty) + sizeof(chunk);
+		ib->ptr	 = reinterpret_cast<char*>(empty) + sizeof(chunk);
 
 	}
 
@@ -251,7 +253,7 @@ public:
 		c = ib->head;
 		c->next = nullptr;
 		ib->free = chunk_size_;
-		ib->ptr	 = ((char*)c) + sizeof(chunk);
+		ib->ptr	 = reinterpret_cast<char*>(c) + sizeof(chunk);
 
 		tail_ = array_;
 	}

src/msgpack/zbuffer.hpp

@@ -57,7 +57,7 @@ public:
 public:
 	void write(const char* buf, size_t len)
 	{
-		stream_.next_in = (Bytef*)buf;
+		stream_.next_in = reinterpret_cast<Bytef*>(const_cast<char*>(buf));
 		stream_.avail_in = len;
 
 		do {
@@ -115,8 +115,8 @@ public:
 
 	void reset_buffer()
 	{
-		stream_.avail_out += (char*)stream_.next_out - data_;
+		stream_.avail_out += reinterpret_cast<char*>(stream_.next_out) - data_;
-		stream_.next_out = (Bytef*)data_;
+		stream_.next_out = reinterpret_cast<Bytef*>(data_);
 	}
 
 	char* release_buffer()
@@ -131,17 +131,17 @@ public:
 private:
 	bool expand()
 	{
-		size_t used = (char*)stream_.next_out - data_;
+		size_t used = reinterpret_cast<char*>(stream_.next_out) - data_;
 		size_t csize = used + stream_.avail_out;
 		size_t nsize = (csize == 0) ? init_size_ : csize * 2;
 
-		char* tmp = (char*)::realloc(data_, nsize);
+		char* tmp = static_cast<char*>(::realloc(data_, nsize));
 		if(tmp == NULL) {
 			return false;
 		}
 
 		data_ = tmp;
-		stream_.next_out  = (Bytef*)(tmp + used);
+		stream_.next_out  = reinterpret_cast<Bytef*>(tmp + used);
 		stream_.avail_out = nsize - used;
 
 		return true;

src/msgpack/zone.hpp.erb

@@ -80,7 +80,7 @@ class zone {
                 nnext = nused * 2;
             }
             finalizer* tmp =
-                (finalizer*)::realloc(array_, sizeof(finalizer) * nnext);
+                static_cast<finalizer*>(::realloc(array_, sizeof(finalizer) * nnext));
             if(!tmp) {
                 throw std::bad_alloc();
             }
@@ -101,14 +101,14 @@ class zone {
     struct chunk_list {
         chunk_list(size_t chunk_size)
         {
-            chunk* c = (chunk*)::malloc(sizeof(chunk) + chunk_size);
+            chunk* c = static_cast<chunk*>(::malloc(sizeof(chunk) + chunk_size));
             if(!c) {
                 throw std::bad_alloc();
             }
             
             head_ = c;
             free_ = chunk_size;
-            ptr_  = ((char*)c) + sizeof(chunk);
+            ptr_  = reinterpret_cast<char*>(c) + sizeof(chunk);
             c->next_ = nullptr;
         }
         ~chunk_list()
@@ -138,7 +138,7 @@ class zone {
             }
             head_->next_ = nullptr;
             free_ = chunk_size;
-            ptr_  = ((char*)head_) + sizeof(chunk);
+            ptr_  = reinterpret_cast<char*>(head_) + sizeof(chunk);
         }
         size_t free_;
         char* ptr_;
@@ -183,7 +183,7 @@ private:
 
 inline zone* zone::create(size_t chunk_size)
 {
-    zone* z = (zone*)::malloc(sizeof(zone) + chunk_size);
+    zone* z = static_cast<zone*>(::malloc(sizeof(zone) + chunk_size));
     if (!z) {
         return nullptr;
     }
@@ -236,9 +236,9 @@ inline void* zone::malloc_expand(size_t size)
         sz *= 2;
     }
 
-    chunk* c = (chunk*)::malloc(sizeof(chunk) + sz);
+    chunk* c = static_cast<chunk*>(::malloc(sizeof(chunk) + sz));
 
-    char* ptr = ((char*)c) + sizeof(chunk);
+    char* ptr = reinterpret_cast<char*>(c) + sizeof(chunk);
 
     c->next_  = cl->head_;
     cl->head_ = c;