asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

C#: Make property accessors internal

Browse source
This commit is contained in:
Raul Santos 2023-01-16 14:57:50 +01:00
parent 04a39ecd84
commit 47e355f965
No known key found for this signature in database
GPG key ID: B532473AE3A803E4
1 changed files with 8 additions and 30 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

38
modules/mono/editor/bindings_generator.cpp
View file

@ -1864,12 +1864,7 @@ Error BindingsGenerator::_generate_cs_property(const BindingsGenerator::TypeInte
p_output.append("\n" OPEN_BLOCK_L1); p_output.append("\n" OPEN_BLOCK_L1);
if (getter) { if (getter) {
p_output.append(INDENT2 "get\n" p_output.append(INDENT2 "get\n" OPEN_BLOCK_L2 INDENT3);
// TODO Remove this once we make accessor methods private/internal (they will no longer be marked as obsolete after that)
"#pragma warning disable CS0618 // Disable warning about obsolete method\n"
OPEN_BLOCK_L2 INDENT3);
p_output.append("return "); p_output.append("return ");
p_output.append(getter->proxy_name + "("); p_output.append(getter->proxy_name + "(");
@ -1884,21 +1879,11 @@ Error BindingsGenerator::_generate_cs_property(const BindingsGenerator::TypeInte
p_output.append(itos(p_iprop.index)); p_output.append(itos(p_iprop.index));
} }
} }
p_output.append(");\n" p_output.append(");\n" CLOSE_BLOCK_L2);
CLOSE_BLOCK_L2
// TODO Remove this once we make accessor methods private/internal (they will no longer be marked as obsolete after that)
"#pragma warning restore CS0618\n");
} }
if (setter) { if (setter) {
p_output.append(INDENT2 "set\n" p_output.append(INDENT2 "set\n" OPEN_BLOCK_L2 INDENT3);
// TODO Remove this once we make accessor methods private/internal (they will no longer be marked as obsolete after that)
"#pragma warning disable CS0618 // Disable warning about obsolete method\n"
OPEN_BLOCK_L2 INDENT3);
p_output.append(setter->proxy_name + "("); p_output.append(setter->proxy_name + "(");
if (p_iprop.index != -1) { if (p_iprop.index != -1) {
@ -1912,12 +1897,7 @@ Error BindingsGenerator::_generate_cs_property(const BindingsGenerator::TypeInte
p_output.append(itos(p_iprop.index) + ", "); p_output.append(itos(p_iprop.index) + ", ");
} }
} }
p_output.append("value);\n" p_output.append("value);\n" CLOSE_BLOCK_L2);
CLOSE_BLOCK_L2
// TODO Remove this once we make accessor methods private/internal (they will no longer be marked as obsolete after that)
"#pragma warning restore CS0618\n");
} }
p_output.append(CLOSE_BLOCK_L1); p_output.append(CLOSE_BLOCK_L1);
@ -3056,12 +3036,10 @@ bool BindingsGenerator::_populate_object_type_interfaces() {
HashMap<StringName, StringName>::Iterator accessor = accessor_methods.find(imethod.cname); HashMap<StringName, StringName>::Iterator accessor = accessor_methods.find(imethod.cname);
if (accessor) { if (accessor) {
const PropertyInterface *accessor_property = itype.find_property_by_name(accessor->value); // We only make internal an accessor method if it's in the same class as the property.
// It's easier this way, but also we don't know if an accessor method in a different class
// We only deprecate an accessor method if it's in the same class as the property. It's easier this way, but also // could have other purposes, so better leave those untouched.
// we don't know if an accessor method in a different class could have other purposes, so better leave those untouched. imethod.is_internal = true;
imethod.is_deprecated = true;
imethod.deprecation_message = imethod.proxy_name + " is deprecated. Use the " + accessor_property->proxy_name + " property instead.";
} }
if (itype.class_doc) { if (itype.class_doc) {