Browse Source

Don't include `core/io/image.h` in `core/os/os.h`

`core/os/os.h` doesn't use `core/io/image.h`. It just brings
transitive dependencies. Lots of dependencies because `core/os/os.h`
is transitively included in almost every file of godot

Also added `core/io/image.h` into files^1 where `Ref<Image>` and `core/os/os.h`
were used to prevent obscure errors involving `Ref<Image>`

^1 except those which include `core/io/image_loader.h` or `core/io/image.h` by
corresponding .h file with the same name

Signed-off-by: Yevhen Babiichuk (DustDFG) <[email protected]>
Co-authored-by: A Thousand Ships <[email protected]>
Yevhen Babiichuk (DustDFG) 9 months ago
parent
commit
af6d260c17

+ 0 - 1
core/core_bind.h

@@ -32,7 +32,6 @@
 #define CORE_BIND_H
 #define CORE_BIND_H
 
 
 #include "core/debugger/engine_profiler.h"
 #include "core/debugger/engine_profiler.h"
-#include "core/io/image.h"
 #include "core/io/resource_loader.h"
 #include "core/io/resource_loader.h"
 #include "core/io/resource_saver.h"
 #include "core/io/resource_saver.h"
 #include "core/object/script_language.h"
 #include "core/object/script_language.h"

+ 1 - 0
core/extension/gdextension_interface.cpp

@@ -34,6 +34,7 @@
 #include "core/extension/gdextension.h"
 #include "core/extension/gdextension.h"
 #include "core/extension/gdextension_compat_hashes.h"
 #include "core/extension/gdextension_compat_hashes.h"
 #include "core/io/file_access.h"
 #include "core/io/file_access.h"
+#include "core/io/image.h"
 #include "core/io/xml_parser.h"
 #include "core/io/xml_parser.h"
 #include "core/object/class_db.h"
 #include "core/object/class_db.h"
 #include "core/object/script_language_extension.h"
 #include "core/object/script_language_extension.h"

+ 0 - 1
core/os/os.h

@@ -32,7 +32,6 @@
 #define OS_H
 #define OS_H
 
 
 #include "core/config/engine.h"
 #include "core/config/engine.h"
-#include "core/io/image.h"
 #include "core/io/logger.h"
 #include "core/io/logger.h"
 #include "core/io/remote_filesystem_client.h"
 #include "core/io/remote_filesystem_client.h"
 #include "core/os/time_enums.h"
 #include "core/os/time_enums.h"

+ 1 - 0
drivers/gles3/rasterizer_gles3.cpp

@@ -35,6 +35,7 @@
 
 
 #include "core/config/project_settings.h"
 #include "core/config/project_settings.h"
 #include "core/io/dir_access.h"
 #include "core/io/dir_access.h"
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "storage/texture_storage.h"
 #include "storage/texture_storage.h"
 
 

+ 1 - 0
drivers/gles3/storage/texture_storage.h

@@ -36,6 +36,7 @@
 #include "platform_gl.h"
 #include "platform_gl.h"
 
 
 #include "config.h"
 #include "config.h"
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "core/templates/rid_owner.h"
 #include "core/templates/rid_owner.h"
 #include "servers/rendering/renderer_compositor.h"
 #include "servers/rendering/renderer_compositor.h"

+ 1 - 0
editor/debugger/editor_profiler.cpp

@@ -30,6 +30,7 @@
 
 
 #include "editor_profiler.h"
 #include "editor_profiler.h"
 
 
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "editor/editor_settings.h"
 #include "editor/editor_settings.h"
 #include "editor/editor_string_names.h"
 #include "editor/editor_string_names.h"

+ 1 - 0
editor/debugger/editor_visual_profiler.cpp

@@ -30,6 +30,7 @@
 
 
 #include "editor_visual_profiler.h"
 #include "editor_visual_profiler.h"
 
 
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "editor/editor_settings.h"
 #include "editor/editor_settings.h"
 #include "editor/editor_string_names.h"
 #include "editor/editor_string_names.h"

+ 1 - 0
editor/editor_node.cpp

@@ -35,6 +35,7 @@
 #include "core/input/input.h"
 #include "core/input/input.h"
 #include "core/io/config_file.h"
 #include "core/io/config_file.h"
 #include "core/io/file_access.h"
 #include "core/io/file_access.h"
+#include "core/io/image.h"
 #include "core/io/resource_loader.h"
 #include "core/io/resource_loader.h"
 #include "core/io/resource_saver.h"
 #include "core/io/resource_saver.h"
 #include "core/object/class_db.h"
 #include "core/object/class_db.h"

+ 1 - 0
editor/plugins/editor_preview_plugins.cpp

@@ -32,6 +32,7 @@
 
 
 #include "core/config/project_settings.h"
 #include "core/config/project_settings.h"
 #include "core/io/file_access_memory.h"
 #include "core/io/file_access_memory.h"
+#include "core/io/image.h"
 #include "core/io/resource_loader.h"
 #include "core/io/resource_loader.h"
 #include "core/object/script_language.h"
 #include "core/object/script_language.h"
 #include "core/os/os.h"
 #include "core/os/os.h"

+ 1 - 0
main/main.cpp

@@ -42,6 +42,7 @@
 #include "core/io/dir_access.h"
 #include "core/io/dir_access.h"
 #include "core/io/file_access_pack.h"
 #include "core/io/file_access_pack.h"
 #include "core/io/file_access_zip.h"
 #include "core/io/file_access_zip.h"
+#include "core/io/image.h"
 #include "core/io/image_loader.h"
 #include "core/io/image_loader.h"
 #include "core/io/ip.h"
 #include "core/io/ip.h"
 #include "core/io/resource_loader.h"
 #include "core/io/resource_loader.h"

+ 1 - 0
modules/basis_universal/image_compress_basisu.cpp

@@ -30,6 +30,7 @@
 
 
 #include "image_compress_basisu.h"
 #include "image_compress_basisu.h"
 
 
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "core/string/print_string.h"
 #include "core/string/print_string.h"
 #include "servers/rendering_server.h"
 #include "servers/rendering_server.h"

+ 1 - 0
modules/tga/image_loader_tga.cpp

@@ -32,6 +32,7 @@
 
 
 #include "core/error/error_macros.h"
 #include "core/error/error_macros.h"
 #include "core/io/file_access_memory.h"
 #include "core/io/file_access_memory.h"
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "core/string/print_string.h"
 #include "core/string/print_string.h"
 
 

+ 1 - 0
modules/theora/video_stream_theora.cpp

@@ -31,6 +31,7 @@
 #include "video_stream_theora.h"
 #include "video_stream_theora.h"
 
 
 #include "core/config/project_settings.h"
 #include "core/config/project_settings.h"
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "scene/resources/image_texture.h"
 #include "scene/resources/image_texture.h"
 
 

+ 1 - 0
modules/tinyexr/image_saver_tinyexr.cpp

@@ -31,6 +31,7 @@
 #include "image_saver_tinyexr.h"
 #include "image_saver_tinyexr.h"
 
 
 #include "core/math/math_funcs.h"
 #include "core/math/math_funcs.h"
+#include "core/os/os.h"
 
 
 #include <zlib.h> // Should come before including tinyexr.
 #include <zlib.h> // Should come before including tinyexr.
 
 

+ 1 - 1
modules/tinyexr/image_saver_tinyexr.h

@@ -31,7 +31,7 @@
 #ifndef IMAGE_SAVER_TINYEXR_H
 #ifndef IMAGE_SAVER_TINYEXR_H
 #define IMAGE_SAVER_TINYEXR_H
 #define IMAGE_SAVER_TINYEXR_H
 
 
-#include "core/os/os.h"
+#include "core/io/image.h"
 
 
 Error save_exr(const String &p_path, const Ref<Image> &p_img, bool p_grayscale);
 Error save_exr(const String &p_path, const Ref<Image> &p_img, bool p_grayscale);
 Vector<uint8_t> save_exr_buffer(const Ref<Image> &p_img, bool p_grayscale);
 Vector<uint8_t> save_exr_buffer(const Ref<Image> &p_img, bool p_grayscale);

+ 1 - 0
platform/android/export/export_plugin.h

@@ -35,6 +35,7 @@
 #include "godot_plugin_config.h"
 #include "godot_plugin_config.h"
 #endif // DISABLE_DEPRECATED
 #endif // DISABLE_DEPRECATED
 
 
+#include "core/io/image.h"
 #include "core/io/zip_io.h"
 #include "core/io/zip_io.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "editor/export/editor_export_platform.h"
 #include "editor/export/editor_export_platform.h"

+ 1 - 0
platform/macos/export/export_plugin.h

@@ -34,6 +34,7 @@
 #include "core/config/project_settings.h"
 #include "core/config/project_settings.h"
 #include "core/io/dir_access.h"
 #include "core/io/dir_access.h"
 #include "core/io/file_access.h"
 #include "core/io/file_access.h"
+#include "core/io/image.h"
 #include "core/io/marshalls.h"
 #include "core/io/marshalls.h"
 #include "core/io/resource_saver.h"
 #include "core/io/resource_saver.h"
 #include "core/os/os.h"
 #include "core/os/os.h"

+ 1 - 0
platform/windows/display_server_windows.h

@@ -38,6 +38,7 @@
 
 
 #include "core/config/project_settings.h"
 #include "core/config/project_settings.h"
 #include "core/input/input.h"
 #include "core/input/input.h"
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "drivers/unix/ip_unix.h"
 #include "drivers/unix/ip_unix.h"
 #include "drivers/wasapi/audio_driver_wasapi.h"
 #include "drivers/wasapi/audio_driver_wasapi.h"

+ 1 - 0
platform/windows/native_menu_windows.h

@@ -31,6 +31,7 @@
 #ifndef NATIVE_MENU_WINDOWS_H
 #ifndef NATIVE_MENU_WINDOWS_H
 #define NATIVE_MENU_WINDOWS_H
 #define NATIVE_MENU_WINDOWS_H
 
 
+#include "core/io/image.h"
 #include "core/templates/hash_map.h"
 #include "core/templates/hash_map.h"
 #include "core/templates/rid_owner.h"
 #include "core/templates/rid_owner.h"
 #include "servers/display/native_menu.h"
 #include "servers/display/native_menu.h"

+ 1 - 0
scene/theme/default_theme.cpp

@@ -30,6 +30,7 @@
 
 
 #include "default_theme.h"
 #include "default_theme.h"
 
 
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "default_font.gen.h"
 #include "default_font.gen.h"
 #include "default_theme_icons.gen.h"
 #include "default_theme_icons.gen.h"

+ 1 - 1
servers/display_server.h

@@ -32,6 +32,7 @@
 #define DISPLAY_SERVER_H
 #define DISPLAY_SERVER_H
 
 
 #include "core/input/input.h"
 #include "core/input/input.h"
+#include "core/io/image.h"
 #include "core/io/resource.h"
 #include "core/io/resource.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "core/variant/callable.h"
 #include "core/variant/callable.h"
@@ -39,7 +40,6 @@
 #include "display/native_menu.h"
 #include "display/native_menu.h"
 
 
 class Texture2D;
 class Texture2D;
-class Image;
 
 
 class DisplayServer : public Object {
 class DisplayServer : public Object {
 	GDCLASS(DisplayServer, Object)
 	GDCLASS(DisplayServer, Object)

+ 1 - 0
servers/movie_writer/movie_writer.h

@@ -31,6 +31,7 @@
 #ifndef MOVIE_WRITER_H
 #ifndef MOVIE_WRITER_H
 #define MOVIE_WRITER_H
 #define MOVIE_WRITER_H
 
 
+#include "core/io/image.h"
 #include "core/templates/local_vector.h"
 #include "core/templates/local_vector.h"
 #include "servers/audio/audio_driver_dummy.h"
 #include "servers/audio/audio_driver_dummy.h"
 #include "servers/audio_server.h"
 #include "servers/audio_server.h"

+ 1 - 0
servers/rendering/renderer_rd/renderer_compositor_rd.h

@@ -31,6 +31,7 @@
 #ifndef RENDERER_COMPOSITOR_RD_H
 #ifndef RENDERER_COMPOSITOR_RD_H
 #define RENDERER_COMPOSITOR_RD_H
 #define RENDERER_COMPOSITOR_RD_H
 
 
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "servers/rendering/renderer_compositor.h"
 #include "servers/rendering/renderer_compositor.h"
 #include "servers/rendering/renderer_rd/environment/fog.h"
 #include "servers/rendering/renderer_rd/environment/fog.h"

+ 1 - 0
servers/rendering/renderer_rd/renderer_scene_render_rd.cpp

@@ -31,6 +31,7 @@
 #include "renderer_scene_render_rd.h"
 #include "renderer_scene_render_rd.h"
 
 
 #include "core/config/project_settings.h"
 #include "core/config/project_settings.h"
+#include "core/io/image.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "renderer_compositor_rd.h"
 #include "renderer_compositor_rd.h"
 #include "servers/rendering/renderer_rd/environment/fog.h"
 #include "servers/rendering/renderer_rd/environment/fog.h"

+ 1 - 0
servers/text_server.h

@@ -31,6 +31,7 @@
 #ifndef TEXT_SERVER_H
 #ifndef TEXT_SERVER_H
 #define TEXT_SERVER_H
 #define TEXT_SERVER_H
 
 
+#include "core/io/image.h"
 #include "core/object/ref_counted.h"
 #include "core/object/ref_counted.h"
 #include "core/os/os.h"
 #include "core/os/os.h"
 #include "core/templates/rid.h"
 #include "core/templates/rid.h"