X-Git-Url: http://de.git.xonotic.org/?p=xonotic%2Fxonotic-data.pk3dir.git;a=blobdiff_plain;f=qcsrc%2Flib%2Flog.qh;h=16875c8bc3e233224cc484b49a8efa63ed5f4932;hp=aa4f87cb6637f68ebaebc8414e05b039b6de63a9;hb=5135af77c8666e85e6fa2e5664e7f53e09690b0d;hpb=af71335a8e418edc55126a63aa65afd9353f413c

diff --git a/qcsrc/lib/log.qh b/qcsrc/lib/log.qh
index aa4f87cb6..16875c8bc 100644
--- a/qcsrc/lib/log.qh
+++ b/qcsrc/lib/log.qh
@@ -2,32 +2,34 @@
 
 #include "progname.qh"
 
-#define assert(expr, ...) _assert(LOG_SEVERE, expr, __VA_ARGS__)
-#define devassert(...) MACRO_BEGIN if (autocvar_developer) assert(__VA_ARGS__); MACRO_END
+void print_assertfailed_severe(string expr);
+void print_assertfailed_fatal(string expr);
+
+#define assert(expr, ...) _assert(print_assertfailed_severe, expr, __VA_ARGS__)
+#define devassert(...) MACRO_BEGIN if (autocvar_developer > 0) assert(__VA_ARGS__); MACRO_END
 
 #define assert_once(expr, ...) \
-	MACRO_BEGIN { \
+	MACRO_BEGIN \
 		static bool __once; \
 		if (!__once) \
 		{ \
 			assert(expr, __VA_ARGS__); \
 			__once = true; \
 		} \
-	} MACRO_END
-#define devassert_once(...) MACRO_BEGIN if (autocvar_developer) assert_once(__VA_ARGS__); MACRO_END
+	MACRO_END
+#define devassert_once(...) MACRO_BEGIN if (autocvar_developer > 0) assert_once(__VA_ARGS__); MACRO_END
 
-#define demand(expr, ...) _assert(LOG_FATAL, expr, __VA_ARGS__)
-#define devdemand(...) MACRO_BEGIN if (autocvar_developer) demand(__VA_ARGS__); MACRO_END
+#define demand(expr, ...) _assert(print_assertfailed_fatal, expr, __VA_ARGS__)
+#define devdemand(...) MACRO_BEGIN if (autocvar_developer > 0) demand(__VA_ARGS__); MACRO_END
 
 #define _assert(f, expr, then) \
 	MACRO_BEGIN \
-	{ \
 		if (!(expr)) \
 		{ \
-			f("assertion failed: `" #expr "`\n"); \
+			f(#expr); \
 			then; \
 		} \
-	} MACRO_END
+	MACRO_END
 
 #define ASSERT_LESS(name, var, const) noref int name[(const - var + 1)];
 
@@ -39,13 +41,18 @@ string(string, string...) strcat1n = #115;
 #endif
 
 // would be nice if __FUNC__ could be concatenated at compile time
-#define __SOURCELOC__ (sprintf("^7%s^9" "(" "^9"__FILE__"^7"  ":"  "^9"STR(__LINE__)"^7" ")", __FUNC__))
+#if 0
+	// less work, bigger binary
+	#define __SOURCELOC__ (sprintf("^7%s^9(^9"__FILE__"^7:^9"STR(__LINE__)"^7)", __FUNC__))
+#else
+	#define __SOURCELOC__ (sprintf("^7%s^9(^9%s^7:^9%s^7)", __FUNC__, __FILE__, STR(__LINE__)))
+#endif
 
-#define _LOG_HEADER(level) "^9[::" "^7"PROGNAME"^9" "::" level"^9" "] ", __SOURCELOC__
+#define _LOG_HEADER(level) "^9[::^7"PROGNAME"^9::"level"^9] ", __SOURCELOC__
 #define _LOG(f, level, s) \
-	MACRO_BEGIN { \
+	MACRO_BEGIN \
 		f(strcat1n(_LOG_HEADER(level), "\n^7", s, "\n")); \
-	} MACRO_END
+	MACRO_END
 
 #define  LOG_FATAL(...) _LOG_FATAL(strcat1n(__VA_ARGS__))
 #define  LOG_FATALF(...) _LOG_FATAL(sprintf(__VA_ARGS__))
@@ -62,10 +69,13 @@ string(string, string...) strcat1n = #115;
 #define  LOG_INFO(...) _LOG_INFO(strcat1n(__VA_ARGS__))
 #define  LOG_INFOF(...) _LOG_INFO(sprintf(__VA_ARGS__))
 #define _LOG_INFO(s) \
-	MACRO_BEGIN { \
+	MACRO_BEGIN \
 		dprint(_LOG_HEADER("^5INFO")); \
-		print("\n^7", s); \
-	} MACRO_END
+		string __s = s; \
+		print("\n^7", __s); \
+		/* TODO: unconditionally add a newline when possible */ \
+		if (str2chr(__s, strlen(__s) - 1) != '\n') { print("\n"); } \
+	MACRO_END
 
 #define  LOG_TRACE(...) _LOG_TRACE(strcat1n(__VA_ARGS__))
 #define  LOG_TRACEF(...) _LOG_TRACE(sprintf(__VA_ARGS__))
@@ -76,9 +86,9 @@ string(string, string...) strcat1n = #115;
 #define _LOG_DEBUG(s) _LOG(dprint2, "^2DEBUG", s)
 
 #define dprint2(msg) \
-	MACRO_BEGIN { \
+	MACRO_BEGIN \
 		if (autocvar_developer > 1) dprint(msg); \
-	} MACRO_END
+	MACRO_END
 
 // TODO: this sucks, lets find a better way to do backtraces?
 #define _backtrace() builtin_remove(NULL)
@@ -94,7 +104,6 @@ noref bool autocvar_prvm_backtraceforwarnings;
 
 #define backtrace(msg) \
 	MACRO_BEGIN \
-	{ \
 		int dev = autocvar_developer; \
 		bool war = autocvar_prvm_backtraceforwarnings; \
 		bt_cvar_set("developer", "1"); \
@@ -104,4 +113,16 @@ noref bool autocvar_prvm_backtraceforwarnings;
 		print("\n--- CUT UNTIL HERE ---\n"); \
 		bt_cvar_set("developer", ftos(dev)); \
 		bt_cvar_set("prvm_backtraceforwarnings", ftos(war)); \
-	} MACRO_END
+	MACRO_END
+
+void print_assertfailed_severe(string expr)
+{
+	// this is inside a function to avoid expanding it on compilation everytime
+	LOG_SEVERE("assertion failed: `", expr, "`\n");
+}
+
+void print_assertfailed_fatal(string expr)
+{
+	// this is inside a function to avoid expanding it on compilation everytime
+	LOG_FATAL("assertion failed: `", expr, "`\n");
+}