Include version.hpp in defs.hpp`` instead

Author: 60k41p

binding_generator.py

@@ -699,11 +699,6 @@ def generate_builtin_class_header(builtin_api, size, used_classes, fully_used_cl
     result.append("#include <godot_cpp/core/defs.hpp>")
     result.append("")
 
-    # Fixes the "use of undeclared identifier 'set_typed'" compile error when typed_dictionary.hpp & dictionary.hpp are both included
-    if class_name == "Dictionary":
-        result.append("#include <godot_cpp/core/version.hpp>")
-        result.append("")
-
     # Special cases.
     if class_name == "String":
         result.append("#include <godot_cpp/classes/global_constants.hpp>")

include/godot_cpp/core/defs.hpp

@@ -35,6 +35,12 @@
 #include <type_traits>
 #include <utility>
 
+// version.hpp is generated, and won't exist in a clean checkout until the first build or after a cleanup.
+// __has_include (available since C++17) prevents errors in the IDE when it hasn't been created yet.
+#if __has_include(<godot_cpp/core/version.hpp>)
+#include <godot_cpp/core/version.hpp>
+#endif
+
 namespace godot {
 
 #if !defined(GDE_EXPORT)