Fixup some warnings pointed out by gcc

Disable Q_strnlen until its needed due to warnings even though standard library version of strnlen does exactly this style of return.

Author: ensiform

code/api/shared/q_string.c

@@ -433,7 +433,7 @@ void Q_strcat_fn( char *dest, const int size, const char *src, const char *func,
 	Q_strncpyz_fn( dest + l1, src, size - l1, func, file, line );
 }
 
-size_t Q_strnlen_fn(const char *str, size_t strsz, const char *func, const char *file, const int line ) {
+/*size_t Q_strnlen_fn(const char* str, size_t strsz, const char* func, const char* file, const int line) {
 	const char *p;
 	if (!str) {
 		Lib_Error("Q_strnlen: NULL str (%s, %s:%i)", func, file, line);
@@ -442,7 +442,7 @@ size_t Q_strnlen_fn(const char *str, size_t strsz, const char *func, const char
 
 	p = memchr(str, 0, strsz);
 	return p ? p - str : strsz;
-}
+}*/
 
 char *Q_stradd( char *dst, const char *src )
 {

code/api/shared/q_string.h

@@ -38,7 +38,7 @@ If you have questions concerning this license or the applicable additional terms
 #pragma once
 
 #include "q_primitives.h"
-#include <stddef.h>
+#include <stddef.h> // Q_vsnprintf or Q_strnlen needs size_t
 
 #if defined(__cplusplus)
 extern "C" {
@@ -72,10 +72,10 @@ char *Q_strupr( char *s1 );
 // buffer size safe library replacements
 #define Q_strncpyz(dest, src, destsize) Q_strncpyz_fn(dest, src, destsize, __func__, RELATIVE_FILENAME, __LINE__)
 #define Q_strcat(dest, size, src) Q_strcat_fn(dest, size, src, __func__, RELATIVE_FILENAME, __LINE__)
-#define Q_strnlen(str, strsz ) Q_strnlen_fn(str, strsz, __func__, RELATIVE_FILENAME, __LINE__)
+//#define Q_strnlen(str, strsz ) Q_strnlen_fn(str, strsz, __func__, RELATIVE_FILENAME, __LINE__)
 void Q_strncpyz_fn( char *dest, const char *src, const int destsize, const char *func, const char *file, int line );
 void Q_strcat_fn( char *dest, const int size, const char *src, const char *func, const char *file, int line );
-size_t Q_strnlen_fn(const char *str, size_t strsz, const char *func, const char *file, int line );
+//size_t Q_strnlen_fn(const char *str, size_t strsz, const char *func, const char *file, int line );
 
 const char *Q_stristr( const char *s, const char *find);
 

code/cgame/cg_ents.c

@@ -1571,7 +1571,7 @@ static void CG_InterpolateEntityPosition( centity_t *cent ) {
 CG_InterpolateEntityPosition
 =============================
 */
-static void CG_InterpolateEntityAngle( centity_t *cent ) {
+/*static void CG_InterpolateEntityAngle(centity_t* cent) {
 	vec3_t		current, next;
 	float		f;
 
@@ -1583,7 +1583,7 @@ static void CG_InterpolateEntityAngle( centity_t *cent ) {
 	cent->lerpAngles[0] = LerpAngle( current[0], next[0], f );
 	cent->lerpAngles[1] = LerpAngle( current[1], next[1], f );
 	cent->lerpAngles[2] = LerpAngle( current[2], next[2], f );
-}
+}*/
 
 
 /*

code/game/g_main.c

@@ -2227,34 +2227,35 @@ void G_RunEntities(void) {
 			continue;
 
 		switch (ent->s.eType) {
-		case ET_MISSILE: {
-			if (ent->antilag_time) {
-				int snap_ms = 1000 / sv_fps.integer;
-				int prestep = ent->s.pos.trTime - projectile_now;
-				int steps = (projectile_now - ent->antilag_time) / snap_ms, rs = 0;
-				int ot = ent->s.pos.trTime - prestep;
-				ent->s.pos.trTime = ent->antilag_time + prestep;
-
-				while (steps > 0) {
-					int now = projectile_now - steps * snap_ms;
-					G_TimeShiftAllClients(now, ent->parent);
-					G_RunMissile(ent, now);
-
-					if (!ent->inuse)
-						break;
-
-					rs = 1;
-					steps--;
+		case ET_MISSILE:
+			{
+				if (ent->antilag_time) {
+					int snap_ms = 1000 / sv_fps.integer;
+					int prestep = ent->s.pos.trTime - projectile_now;
+					int steps = (projectile_now - ent->antilag_time) / snap_ms, rs = 0;
+					//int ot = ent->s.pos.trTime - prestep;
+					ent->s.pos.trTime = ent->antilag_time + prestep;
+
+					while (steps > 0) {
+						int now = projectile_now - steps * snap_ms;
+						G_TimeShiftAllClients(now, ent->parent);
+						G_RunMissile(ent, now);
+
+						if (!ent->inuse)
+							break;
+
+						rs = 1;
+						steps--;
+					}
+					ent->antilag_time = 0;
+					if (rs)
+						G_TimeShiftAllClients( projectile_now, NULL );
 				}
-				ent->antilag_time = 0;
-				if (rs)
-					G_TimeShiftAllClients( projectile_now, NULL );
-			}
 
-			if (ent->inuse)
-				G_RunMissile( ent, level.time );
-			break;
-										 }
+				if (ent->inuse)
+					G_RunMissile( ent, level.time );
+				break;
+			}
 		}
 	}