Fix & make linux conditionals uniform (#4278)

Rogier-5 · est31 · commit 1dfd977ec433 · 2016-07-04T21:00:57.000+02:00
The source used a hodge-podge of different combinations of different
macros to check for linux: 'linux', '__linux', '__linux__'.

As '__linux__' is standard (Posix), and the others are not, the source
now uniformly uses __linux__. If either linux or __linux are defined,
it is made sure that __linux__ is defined as well.
diff --git a/src/guiChatConsole.cpp b/src/guiChatConsole.cpp
@@ -630,7 +630,7 @@ bool GUIChatConsole::OnEvent(const SEvent& event)
 		}
 		else if(event.KeyInput.Char != 0 && !event.KeyInput.Control)
 		{
-			#if (defined(linux) || defined(__linux))
+			#if (defined(__linux__))
 				wchar_t wc = L'_';
 				mbtowc( &wc, (char *) &event.KeyInput.Char, sizeof(event.KeyInput.Char) );
 				prompt.input(wc);
diff --git a/src/intlGUIEditBox.cpp b/src/intlGUIEditBox.cpp
@@ -271,7 +271,7 @@ bool intlGUIEditBox::OnEvent(const SEvent& event)
 			break;
 		case EET_KEY_INPUT_EVENT:
         {
-#if (defined(linux) || defined(__linux) || defined(__FreeBSD__))
+#if (defined(__linux__) || defined(__FreeBSD__))
             // ################################################################
 			// ValkaTR:
             // This part is the difference from the original intlGUIEditBox
diff --git a/src/porting.cpp b/src/porting.cpp
@@ -258,7 +258,7 @@ bool getCurrentExecPath(char *buf, size_t len)
 
 
 //// Linux
-#elif defined(linux) || defined(__linux) || defined(__linux__)
+#elif defined(__linux__)
 
 bool getCurrentExecPath(char *buf, size_t len)
 {
@@ -374,7 +374,7 @@ bool setSystemPaths()
 
 
 //// Linux
-#elif defined(linux) || defined(__linux)
+#elif defined(__linux__)
 
 bool setSystemPaths()
 {
diff --git a/src/porting.h b/src/porting.h
@@ -60,7 +60,11 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 	#include <unistd.h>
 	#include <stdint.h> //for uintptr_t
 
-	#if (defined(linux) || defined(__linux) || defined(__GNU__)) && !defined(_GNU_SOURCE)
+	// Use standard Posix macro for Linux
+	#if (defined(linux) || defined(__linux)) && !defined(__linux__)
+		#define __linux__ 
+	#endif
+	#if (defined(__linux__) || defined(__GNU__)) && !defined(_GNU_SOURCE)
 		#define _GNU_SOURCE
 	#endif
 
@@ -321,7 +325,7 @@ inline const char *getPlatformName()
 	return
 #if defined(ANDROID)
 	"Android"
-#elif defined(linux) || defined(__linux) || defined(__linux__)
+#elif defined(__linux__)
 	"Linux"
 #elif defined(_WIN32) || defined(_WIN64)
 	"Windows"
diff --git a/src/threading/thread.cpp b/src/threading/thread.cpp
@@ -54,7 +54,7 @@ DEALINGS IN THE SOFTWARE.
 
 
 // for setName
-#if defined(linux) || defined(__linux)
+#if defined(__linux__)
 	#include <sys/prctl.h>
 #elif defined(__FreeBSD__) || defined(__OpenBSD__)
 	#include <pthread_np.h>
@@ -70,7 +70,7 @@ DEALINGS IN THE SOFTWARE.
 // for bindToProcessor
 #if __FreeBSD_version >= 702106
 	typedef cpuset_t cpu_set_t;
-#elif defined(__linux) || defined(linux)
+#elif defined(__linux__)
 	#include <sched.h>
 #elif defined(__sun) || defined(sun)
 	#include <sys/types.h>
@@ -261,7 +261,7 @@ DWORD WINAPI Thread::threadProc(LPVOID param)
 
 void Thread::setName(const std::string &name)
 {
-#if defined(linux) || defined(__linux)
+#if defined(__linux__)
 
 	// It would be cleaner to do this with pthread_setname_np,
 	// which was added to glibc in version 2.12, but some major
@@ -363,7 +363,7 @@ bool Thread::bindToProcessor(unsigned int proc_number)
 
 	return SetThreadAffinityMask(getThreadHandle(), 1 << proc_number);
 
-#elif __FreeBSD_version >= 702106 || defined(__linux) || defined(linux)
+#elif __FreeBSD_version >= 702106 || defined(__linux__)
 
 	cpu_set_t cpuset;
 

Original file line number	Diff line number	Diff line change
`@@ -630,7 +630,7 @@ bool GUIChatConsole::OnEvent(const SEvent& event)`
`630`	`630`	`}`
`631`	`631`	`else if(event.KeyInput.Char != 0 && !event.KeyInput.Control)`
`632`	`632`	`{`
`633`		`- #if (defined(linux) \|\| defined(__linux))`
	`633`	`+ #if (defined(__linux__))`
`634`	`634`	`wchar_t wc = L'_';`
`635`	`635`	`mbtowc( &wc, (char *) &event.KeyInput.Char, sizeof(event.KeyInput.Char) );`
`636`	`636`	`prompt.input(wc);`
Original file line number	Diff line number	Diff line change
`@@ -271,7 +271,7 @@ bool intlGUIEditBox::OnEvent(const SEvent& event)`
`271`	`271`	`break;`
`272`	`272`	`case EET_KEY_INPUT_EVENT:`
`273`	`273`	`{`
`274`		`-#if (defined(linux) \|\| defined(__linux) \|\| defined(__FreeBSD__))`
	`274`	`+#if (defined(__linux__) \|\| defined(__FreeBSD__))`
`275`	`275`	`// ################################################################`
`276`	`276`	`// ValkaTR:`
`277`	`277`	`// This part is the difference from the original intlGUIEditBox`
Original file line number	Diff line number	Diff line change
`@@ -258,7 +258,7 @@ bool getCurrentExecPath(char *buf, size_t len)`
`258`	`258`
`259`	`259`
`260`	`260`	`//// Linux`
`261`		`-#elif defined(linux) \|\| defined(__linux) \|\| defined(__linux__)`
	`261`	`+#elif defined(__linux__)`
`262`	`262`
`263`	`263`	`bool getCurrentExecPath(char *buf, size_t len)`
`264`	`264`	`{`
`@@ -374,7 +374,7 @@ bool setSystemPaths()`
`374`	`374`
`375`	`375`
`376`	`376`	`//// Linux`
`377`		`-#elif defined(linux) \|\| defined(__linux)`
	`377`	`+#elif defined(__linux__)`
`378`	`378`
`379`	`379`	`bool setSystemPaths()`
`380`	`380`	`{`