]> git.ipfire.org Git - thirdparty/vala.git/commitdiff
codegen: Don't use G_GNUC_INTERNAL on implicit type specific fields
authorRico Tzschichholz <ricotz@ubuntu.com>
Tue, 27 Apr 2021 08:24:55 +0000 (10:24 +0200)
committerRico Tzschichholz <ricotz@ubuntu.com>
Wed, 28 Apr 2021 06:50:13 +0000 (08:50 +0200)
The actual field was never handled that way, so don't apply it at all
when --hide-internal is passed.

Introduced by 970f58989a2863faca11e30fdbcf4da1273a6acd which claimed to
apply it to methods only.

Fixes https://gitlab.gnome.org/GNOME/vala/issues/1178

codegen/valaccodebasemodule.vala

index c00ddc7be18131199fdd5eb94381f3a4d1a8da9f..f47bed2743972cab5d8202fffa0e05fd69277d61 100644 (file)
@@ -1113,8 +1113,6 @@ public abstract class Vala.CCodeBaseModule : CodeGenerator {
 
                        if (f.is_private_symbol ()) {
                                flock.modifiers = CCodeModifiers.STATIC;
-                       } else if (context.hide_internal && f.is_internal_symbol ()) {
-                               flock.modifiers = CCodeModifiers.INTERNAL;
                        } else {
                                flock.modifiers = CCodeModifiers.EXTERN;
                        }
@@ -1132,8 +1130,6 @@ public abstract class Vala.CCodeBaseModule : CodeGenerator {
                                        cdecl.add_declarator (new CCodeVariableDeclarator (get_variable_array_length_cname (f, dim)));
                                        if (f.is_private_symbol ()) {
                                                cdecl.modifiers = CCodeModifiers.STATIC;
-                                       } else if (context.hide_internal && f.is_internal_symbol ()) {
-                                               cdecl.modifiers = CCodeModifiers.INTERNAL;
                                        } else {
                                                cdecl.modifiers = CCodeModifiers.EXTERN;
                                        }
@@ -1149,8 +1145,6 @@ public abstract class Vala.CCodeBaseModule : CodeGenerator {
                                cdecl.add_declarator (new CCodeVariableDeclarator (get_ccode_delegate_target_name (f)));
                                if (f.is_private_symbol ()) {
                                        cdecl.modifiers = CCodeModifiers.STATIC;
-                               } else if (context.hide_internal && f.is_internal_symbol ()) {
-                                       cdecl.modifiers = CCodeModifiers.INTERNAL;
                                } else {
                                        cdecl.modifiers = CCodeModifiers.EXTERN;
                                }
@@ -1161,8 +1155,6 @@ public abstract class Vala.CCodeBaseModule : CodeGenerator {
                                        cdecl.add_declarator (new CCodeVariableDeclarator (get_ccode_delegate_target_destroy_notify_name (f)));
                                        if (f.is_private_symbol ()) {
                                                cdecl.modifiers = CCodeModifiers.STATIC;
-                                       } else if (context.hide_internal && f.is_internal_symbol ()) {
-                                               cdecl.modifiers = CCodeModifiers.INTERNAL;
                                        } else {
                                                cdecl.modifiers = CCodeModifiers.EXTERN;
                                        }