fix: limit URLs even shorter

Wine seems to empirically crash out on even shorter URLs, so limit it to
1500 as that seems fairly safe.

Author: zao

engine/system/win/sys_main.cpp

@@ -413,10 +413,13 @@ void PlatformOpenURL(const char* url)
 {
 #ifdef _WIN32
 	// There is a surprisingly low limit for the maximum URL length that ShellExecute and browsers can open.
-	// A common lower limit is 2083, so we'll refuse to open URLs smaller than this.
-	// Wine crashes violently if this is violated, 2k should be a safe enough limit while still allowing rather long links.
-	if (strlen(url) < 2048)
+	// A common lower limit on Windows is 2083, so we'll definitely refuse to open URLs longer than that.
+	// Wine on Linux has an indeterminate boundary above which it either crashes or silently does nothing,
+	// one one system it landed at around 1575 bytes before failure, so put the limit somewhat below that.
+	if (strlen(url) <= 1500)
+	{
 		ShellExecuteA(NULL, "open", url, NULL, NULL, SW_SHOWDEFAULT);
+	}
 #else
 #warning LV: URL opening not implemented on this OS.
 	// TODO(LV): Implement URL opening for other OSes.