diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2019-01-10 12:45:57 +0100 |
|---|---|---|
| committer | Rémi Verschelde <rverschelde@gmail.com> | 2019-01-10 12:45:57 +0100 |
| commit | e5f1d19352dbaafab96538b91fad0ce11cf0d046 (patch) | |
| tree | c3cf72310e91ced0bd739b61d1d3197df78c1313 /drivers | |
| parent | 940a2c5ec3e13d57c6462b7070eb3101766c91db (diff) | |
Consistency in resource format saver/loader de-registration
Some used 'is_valid()' checks, others not. Validity is already checked in 'unref()',
and 'remove_resource_format_*()' has an ERR_FAIL condition on 'is_null()' already
(which shouldn't happen since we're only unregistering things that we previously
registered.
Also add missing GDCLASS statement in ResourceFormatLoaderVideoStreamGDNative,
missed in #20552 which was last amended before #19501 was merged.
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/register_driver_types.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/register_driver_types.cpp b/drivers/register_driver_types.cpp index e9e49203ea..20556d98af 100644 --- a/drivers/register_driver_types.cpp +++ b/drivers/register_driver_types.cpp @@ -59,10 +59,8 @@ void unregister_core_driver_types() { if (image_loader_png) memdelete(image_loader_png); - if (resource_saver_png.is_valid()) { - ResourceSaver::remove_resource_format_saver(resource_saver_png); - resource_saver_png.unref(); - } + ResourceSaver::remove_resource_format_saver(resource_saver_png); + resource_saver_png.unref(); } void register_driver_types() { |