From: Leandro Lucarella <llucarella@integratech.com.ar>
Date: Fri, 21 Sep 2007 16:28:44 +0000 (-0300)
Subject: Avoid a weird gcc warning when compiling with optimizations.
X-Git-Tag: 1.0~7
X-Git-Url: https://git.llucax.com/software/eventxx.git/commitdiff_plain/a40aa4e85b7652886bda0669191a2185bfcbfc2d

Avoid a weird gcc warning when compiling with optimizations.

When compiling with -O (or greater) with gcc (4.1.2 at least), there where
some warnings about "dereferencing type-punned pointer will break
strict-aliasing rules". A temporary varible to hold the pointer in the
ugly cast fix it.
---

diff --git a/eventxx b/eventxx
index 9ac878d..7b4d950 100644
--- a/eventxx
+++ b/eventxx
@@ -201,7 +201,9 @@ inline
 type operator| (const type& t1, const type& t2)
 {
 	int r = static_cast< int >(t1) | static_cast< int >(t2);
-	return *reinterpret_cast< type* >(&r);
+	int* pr = &r; // Avoid some weird warning about dereferencing
+	              // type-punned pointer will break strict-aliasing rules
+	return *reinterpret_cast< type* >(pr);
 }
 
 
@@ -320,7 +322,10 @@ struct event: basic_event
 			F& handler = *reinterpret_cast< F* >(h);
 			// Hackish, but this way the handler can get a clean
 			// event type
-			handler(fd, *reinterpret_cast< type* >(&ev));
+			short* pev = &ev; // Avoid some weird warning about
+			                  // dereferencing type-punned pointer
+					  // will break strict-aliasing rules
+			handler(fd, *reinterpret_cast< type* >(pev));
 		}
 
 }; // struct event< F >