Main: Use ImageLoader directly instead of Image::load

Image::load is now issuing warnings (since ef50957) to prevent users
from using it to load images at runtime which would be included in
their exported game.

So we now use ImageLoader explicitly instead for the custom-handled
cases in Main.

Fixes #21072, supersedes #22321.
This commit is contained in:
Rémi Verschelde 2018-09-30 17:21:07 +02:00
parent d3f01a2281
commit fc8b73a935

View file

@ -34,6 +34,7 @@
#include "core/io/file_access_network.h" #include "core/io/file_access_network.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_loader.h"
#include "core/io/ip.h" #include "core/io/ip.h"
#include "core/io/resource_loader.h" #include "core/io/resource_loader.h"
#include "core/io/stream_peer_ssl.h" #include "core/io/stream_peer_ssl.h"
@ -1124,9 +1125,8 @@ Error Main::setup2(Thread::ID p_main_tid_override) {
boot_logo_path = boot_logo_path.strip_edges(); boot_logo_path = boot_logo_path.strip_edges();
if (boot_logo_path != String()) { if (boot_logo_path != String()) {
print_line("Boot splash path: " + boot_logo_path);
boot_logo.instance(); boot_logo.instance();
Error err = boot_logo->load(boot_logo_path); Error err = ImageLoader::load_image(boot_logo_path, boot_logo);
if (err) if (err)
ERR_PRINTS("Non-existing or invalid boot splash at: " + boot_logo_path + ". Loading default splash."); ERR_PRINTS("Non-existing or invalid boot splash at: " + boot_logo_path + ". Loading default splash.");
} }
@ -1707,7 +1707,7 @@ bool Main::start() {
if (iconpath != "") { if (iconpath != "") {
Ref<Image> icon; Ref<Image> icon;
icon.instance(); icon.instance();
if (icon->load(iconpath) == OK) { if (ImageLoader::load_image(iconpath, icon) == OK) {
OS::get_singleton()->set_icon(icon); OS::get_singleton()->set_icon(icon);
hasicon = true; hasicon = true;
} }