Fixed C99 correctness and signed/unsigned compiler warning SDL-1.2
authorSam Lantinga <slouken@libsdl.org>
Sat, 07 Jan 2012 02:07:38 -0500
branchSDL-1.2
changeset 6170 044ca54328b4
parent 6169 6f3e5ccfd585
child 6173 5619adf4736d
Fixed C99 correctness and signed/unsigned compiler warning
src/video/x11/SDL_x11video.c
--- a/src/video/x11/SDL_x11video.c	Tue Jun 28 22:26:47 2011 +0200
+++ b/src/video/x11/SDL_x11video.c	Sat Jan 07 02:07:38 2012 -0500
@@ -418,21 +418,18 @@
     }
 
 	{
+		pid_t pid = getpid();
 		char hostname[256];
 
-		if (gethostname(hostname, sizeof(hostname)) > -1) {
+		if (pid > 0 && gethostname(hostname, sizeof(hostname)) > -1) {
+			Atom _NET_WM_PID = XInternAtom(SDL_Display, "_NET_WM_PID", False);
+			Atom WM_CLIENT_MACHINE = XInternAtom(SDL_Display, "WM_CLIENT_MACHINE", False);
+			
 			hostname[sizeof(hostname)-1] = '\0';
-			pid_t pid = getpid();
-
-			if (pid > 0) {
-				Atom _NET_WM_PID = XInternAtom(SDL_Display, "_NET_WM_PID", False);
-				Atom WM_CLIENT_MACHINE = XInternAtom(SDL_Display, "WM_CLIENT_MACHINE", False);
-				
-				XChangeProperty(SDL_Display, WMwindow, _NET_WM_PID, XA_CARDINAL, 32,
-				                PropModeReplace, (unsigned char *)&pid, 1);
-				XChangeProperty(SDL_Display, WMwindow, WM_CLIENT_MACHINE, XA_STRING, 8,
-				                PropModeReplace, hostname, SDL_strlen(hostname));
-			}
+			XChangeProperty(SDL_Display, WMwindow, _NET_WM_PID, XA_CARDINAL, 32,
+					PropModeReplace, (unsigned char *)&pid, 1);
+			XChangeProperty(SDL_Display, WMwindow, WM_CLIENT_MACHINE, XA_STRING, 8,
+					PropModeReplace, (unsigned char *)hostname, SDL_strlen(hostname));
 		}
 	}