From: Leandro Lucarella <llucarella@integratech.com.ar>
Date: Wed, 27 Dec 2006 19:01:45 +0000 (+0000)
Subject: Casting fixes.
X-Git-Tag: 0.1~22
X-Git-Url: https://git.llucax.com/software/eventxx.git/commitdiff_plain/b2d7232b16208cdad0c31ea507003b7cefe685f0?ds=inline;hp=ffa2e050e8cc072fe47d37e4fe7610c417f4c32f

Casting fixes.
When using function pointers (other than the libevent C API style callbacks),
the implicit cast doesn't work, so reinterpret_cast< void* > is used.
---

diff --git a/event b/event
index b1272dc..a4577bd 100644
--- a/event
+++ b/event
@@ -292,7 +292,7 @@ struct event: basic_event
 	event(int fd, short ev, F& handler, int priority = DEFAULT_PRIORITY)
 		throw(invalid_priority)
 	{
-		event_set(this, fd, ev, &wrapper, &handler);
+		event_set(this, fd, ev, &wrapper, reinterpret_cast< void* >(&handler));
 		if (priority != DEFAULT_PRIORITY
 				&& event_priority_set(this, priority))
 			throw invalid_priority();
@@ -302,7 +302,7 @@ struct event: basic_event
 		event() {}
 		static void wrapper(int fd, short ev, void* h)
 		{
-			F& handler = *static_cast< F* >(h);
+			F& handler = *reinterpret_cast< F* >(h);
 			handler(fd, ev);
 		}
 
@@ -365,7 +365,8 @@ struct timer: event< F >
 	timer(F& handler, int priority = DEFAULT_PRIORITY)
 		throw(invalid_priority)
 	{
-		evtimer_set(this, &event< F >::wrapper, &handler);
+		evtimer_set(this, &event< F >::wrapper,
+			reinterpret_cast< void* >(&handler));
 		if (priority != DEFAULT_PRIORITY
 				&& event_priority_set(this, priority))
 			throw invalid_priority();
@@ -427,7 +428,8 @@ struct signal: event< F >
 	signal(int signum, F& handler, int priority = DEFAULT_PRIORITY)
 		throw(invalid_priority)
 	{
-		signal_set(this, signum, &event< F >::wrapper, &handler);
+		signal_set(this, signum, &event< F >::wrapper,
+			reinterpret_cast< void* >(&handler));
 		if (priority != DEFAULT_PRIORITY
 				&& event_priority_set(this, priority))
 			throw invalid_priority();
@@ -563,7 +565,8 @@ struct dispatcher
 	template < typename F >
 	void add_once(int fd, short ev, F& handler)
 	{
-		internal::event_once(fd, ev, &dispatcher::wrapper< F >, &handler, 0);
+		internal::event_once(fd, ev, &dispatcher::wrapper< F >,
+				reinterpret_cast< void* >(&handler), 0);
 	}
 
 	/**
@@ -596,7 +599,9 @@ struct dispatcher
 	template < typename F >
 	void add_once(int fd, short ev, F& handler, const time& to)
 	{
-		internal::event_once(fd, ev, &dispatcher::wrapper< F >, &handler, const_cast< time* >(&to)); // XXX HACK libevent don't use const
+		internal::event_once(fd, ev, &dispatcher::wrapper< F >,
+				reinterpret_cast< void* >(&handler),
+				const_cast< time* >(&to)); // XXX HACK libevent don't use const
 	}
 
 	/**
@@ -628,7 +633,9 @@ struct dispatcher
 	template < typename F >
 	void add_once_timer(F& handler, const time& to)
 	{
-		internal::event_once(-1, EV_TIMEOUT, &dispatcher::wrapper< F >, &handler, const_cast< time* >(&to)); // XXX HACK libevent don't use const
+		internal::event_once(-1, EV_TIMEOUT, &dispatcher::wrapper< F >,
+				reinterpret_cast< void* >(&handler),
+				const_cast< time* >(&to)); // XXX HACK libevent don't use const
 	}
 
 	/**
@@ -694,7 +701,7 @@ struct dispatcher
 		template < typename F >
 		static void wrapper(int fd, short ev, void* h)
 		{
-			F& handler = *static_cast< F* >(h);
+			F& handler = *reinterpret_cast< F* >(h);
 			handler(fd, ev);
 		}