From e076405f30be922d152dee102879668d68538c7e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20Verschelde?= Date: Mon, 19 Feb 2024 01:19:21 +0100 Subject: [PATCH] GDExtension: Fix `-Wtype-limits` warning in `compatibility_maximum` patch check And cleanup some dead code in `version.h`, we now always define `VERSION_PATCH`. --- core/extension/gdextension.cpp | 6 +++++- core/version.h | 5 ----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/core/extension/gdextension.cpp b/core/extension/gdextension.cpp index 029f52d5a63..60487c5a525 100644 --- a/core/extension/gdextension.cpp +++ b/core/extension/gdextension.cpp @@ -929,9 +929,13 @@ Error GDExtensionResourceLoader::load_gdextension_resource(const String &p_path, compatible = VERSION_MAJOR < compatibility_maximum[0]; } else if (VERSION_MINOR != compatibility_maximum[1]) { compatible = VERSION_MINOR < compatibility_maximum[1]; - } else { + } +#if VERSION_PATCH + // #if check to avoid -Wtype-limits warning when 0. + else { compatible = VERSION_PATCH <= compatibility_maximum[2]; } +#endif if (!compatible) { ERR_PRINT(vformat("GDExtension only compatible with Godot version %s or earlier: %s", compat_string, p_path)); diff --git a/core/version.h b/core/version.h index abb81312ac9..05cc25bc1cc 100644 --- a/core/version.h +++ b/core/version.h @@ -47,13 +47,8 @@ // forward-compatible. // Example: "3.1" #define VERSION_BRANCH _MKSTR(VERSION_MAJOR) "." _MKSTR(VERSION_MINOR) -#if VERSION_PATCH // Example: "3.1.4" #define VERSION_NUMBER VERSION_BRANCH "." _MKSTR(VERSION_PATCH) -#else // patch is 0, we don't include it in the "pretty" version number. -// Example: "3.1" instead of "3.1.0" -#define VERSION_NUMBER VERSION_BRANCH -#endif // VERSION_PATCH // Version number encoded as hexadecimal int with one byte for each number, // for easy comparison from code.