Skip to content

Commit 9dbf9aa

Browse files
committed
Merge pull request godotengine#96511 from lawnjelly/fix_namespace_conflict
Fix SCU global namespace conflict in `resource_format_text.cpp`
2 parents d1c27f2 + 543ab3d commit 9dbf9aa

File tree

2 files changed

+17
-10
lines changed

2 files changed

+17
-10
lines changed

scene/resources/resource_format_text.cpp

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -35,17 +35,12 @@
3535
#include "core/io/missing_resource.h"
3636
#include "core/object/script_language.h"
3737

38-
// Version 2: Changed names for Basis, AABB, Vectors, etc.
39-
// Version 3: New string ID for ext/subresources, breaks forward compat.
40-
// Version 4: PackedByteArray can be base64 encoded, and PackedVector4Array was added.
41-
#define FORMAT_VERSION 4
42-
// For compat, save as version 3 if not using PackedVector4Array or no big PackedByteArray.
43-
#define FORMAT_VERSION_COMPAT 3
44-
45-
#define _printerr() ERR_PRINT(String(res_path + ":" + itos(lines) + " - Parse Error: " + error_text).utf8().get_data());
46-
4738
///
4839

40+
void ResourceLoaderText::_printerr() {
41+
ERR_PRINT(String(res_path + ":" + itos(lines) + " - Parse Error: " + error_text).utf8().get_data());
42+
}
43+
4944
Ref<Resource> ResourceLoaderText::get_resource() {
5045
return resource;
5146
}
@@ -1734,7 +1729,7 @@ Error ResourceFormatSaverTextInstance::save(const String &p_path, const Ref<Reso
17341729
if (load_steps > 1) {
17351730
title += "load_steps=" + itos(load_steps) + " ";
17361731
}
1737-
title += "format=" + itos(use_compat ? FORMAT_VERSION_COMPAT : FORMAT_VERSION) + "";
1732+
title += "format=" + itos(use_compat ? ResourceLoaderText::FORMAT_VERSION_COMPAT : ResourceLoaderText::FORMAT_VERSION) + "";
17381733

17391734
ResourceUID::ID uid = ResourceSaver::get_resource_id_for_path(local_path, true);
17401735

scene/resources/resource_format_text.h

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,17 @@
3838
#include "scene/resources/packed_scene.h"
3939

4040
class ResourceLoaderText {
41+
public:
42+
enum {
43+
// Version 2: Changed names for Basis, AABB, Vectors, etc.
44+
// Version 3: New string ID for ext/subresources, breaks forward compat.
45+
// Version 4: PackedByteArray can be base64 encoded, and PackedVector4Array was added.
46+
FORMAT_VERSION = 4,
47+
// For compat, save as version 3 if not using PackedVector4Array or no big PackedByteArray.
48+
FORMAT_VERSION_COMPAT = 3,
49+
};
50+
51+
private:
4152
bool translation_remapped = false;
4253
String local_path;
4354
String res_path;
@@ -100,6 +111,7 @@ class ResourceLoaderText {
100111

101112
static Error _parse_sub_resource_dummy(DummyReadData *p_data, VariantParser::Stream *p_stream, Ref<Resource> &r_res, int &line, String &r_err_str);
102113
static Error _parse_ext_resource_dummy(DummyReadData *p_data, VariantParser::Stream *p_stream, Ref<Resource> &r_res, int &line, String &r_err_str);
114+
void _printerr();
103115

104116
VariantParser::ResourceParser rp;
105117

0 commit comments

Comments
 (0)