engine: fix -Wincompatible-pointer-types, add it to Werror list

engine/client/ref_common.c

@@ -612,7 +612,7 @@ void R_CollectRendererNames( void )
 		pfn = COM_GetProcAddress( dll, GET_REF_HUMANREADABLE_NAME );
 		if( !pfn ) // just in case
 		{
-			Con_Reportf( "R_CollectRendererNames: can't find GetHumanReadableName export in %s\n", temp, COM_GetLibraryError() );
+			Con_Reportf( "R_CollectRendererNames: can't find GetHumanReadableName export in %s\n", temp );
 			Q_strncpy( ref.readableNames[i], renderers[i], sizeof( ref.readableNames[i] ));
 		}
 		else

engine/common/cvar.c

@@ -595,7 +595,7 @@ void Cvar_Set( const char *var_name, const char *value )
 	if( !var_name )
 	{
 		// there is an error in C code if this happens
-		Con_Printf( "Cvar_Set: passed NULL variable name\n", var_name );
+		Con_Printf( "Cvar_Set: passed NULL variable name\n" );
 		return;
 	}
 
@@ -649,7 +649,7 @@ float Cvar_VariableValue( const char *var_name )
 	if( !var_name )
 	{
 		// there is an error in C code if this happens
-		Con_Printf( "Cvar_VariableValue: passed NULL variable name\n", var_name );
+		Con_Printf( "Cvar_VariableValue: passed NULL variable name\n" );
 		return 0.0f;
 	}
 
@@ -686,7 +686,7 @@ const char *Cvar_VariableString( const char *var_name )
 	if( !var_name )
 	{
 		// there is an error in C code if this happens
-		Con_Printf( "Cvar_VariableString: passed NULL variable name\n", var_name );
+		Con_Printf( "Cvar_VariableString: passed NULL variable name\n" );
 		return "";
 	}
 

engine/common/lib_common.c

@@ -116,7 +116,7 @@ static void COM_GenerateServerLibraryPath( char *out, size_t size )
 
 #else
 	string dllpath;
-	char *ext;
+	const char *ext;
 
 #if XASH_WIN32
 	Q_strncpy( dllpath, GI->game_dll, sizeof( dllpath ) );

engine/common/mod_bmodel.c

@@ -65,14 +65,14 @@ typedef struct
 		dleaf_t		*leafs;
 		dleaf32_t		*leafs32;
 	};
-	int			numleafs;
+	size_t			numleafs;
 
 	union
 	{
 		dclipnode_t	*clipnodes;
 		dclipnode32_t	*clipnodes32;
 	};
-	int			numclipnodes;
+	size_t			numclipnodes;
 
 	dtexinfo_t		*texinfo;
 	size_t			numtexinfo;

engine/common/net_ws.c

@@ -97,7 +97,7 @@ static int ioctl_stub( int d, unsigned long r, ... )
 #define closesocket close
 #define SOCKET int
 #define INVALID_SOCKET -1
-typedef size_t WSAsize_t;
+typedef socklen_t WSAsize_t;
 #else // WIN32
 typedef int WSAsize_t; // for some reason, MS has signed size. We won't exceed 2^32, probably, so just typedef it
 #endif

ref_gl/gl_sprite.c

@@ -87,7 +87,7 @@ static const dframetype_t *R_SpriteLoadFrame( model_t *mod, const void *pin, msp
 	pspriteframe->gl_texturenum = gl_texturenum;
 	*ppframe = pspriteframe;
 
-	return ( const dspriteframe_t* )(( const byte* )pin + sizeof( dspriteframe_t ) + pinframe.width * pinframe.height * bytes );
+	return ( const dframetype_t* )(( const byte* )pin + sizeof( dspriteframe_t ) + pinframe.width * pinframe.height * bytes );
 }
 
 /*

wscript

@@ -235,6 +235,7 @@ def configure(conf):
 		'-Werror=duplicated-branches', # BEWARE: buggy
 		'-Werror=bool-compare',
 		'-Werror=bool-operation',
+		'-Werror=incompatible-pointer-types',
 #		'-Wdouble-promotion', # disable warning flood
 		'-Wstrict-aliasing',
 	]