Merge pull request #4608 from rmosolgo/c-parser-none-fix

GraphQL::CParser: remove dependency on Nodes::NONE
rmosolgo · Sep 1, 2023 · 6e9e906 · 6e9e906
2 parents 1e60a9f + 1ff9d88
commit 6e9e906
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/graphql-c_parser/ext/graphql_c_parser_ext/parser.c b/graphql-c_parser/ext/graphql_c_parser_ext/parser.c
@@ -3278,7 +3278,11 @@ void initialize_node_class_variables() {
   VALUE mGraphQL = rb_const_get_at(rb_cObject, rb_intern("GraphQL"));
   VALUE mGraphQLLanguage = rb_const_get_at(mGraphQL, rb_intern("Language"));
   VALUE mGraphQLLanguageNodes = rb_const_get_at(mGraphQLLanguage, rb_intern("Nodes"));
-  GraphQL_Language_Nodes_NONE = rb_const_get_at(mGraphQLLanguageNodes, rb_intern("NONE"));
+
+  GraphQL_Language_Nodes_NONE = rb_ary_new();
+  rb_global_variable(&GraphQL_Language_Nodes_NONE);
+  rb_ary_freeze(GraphQL_Language_Nodes_NONE);
+
   r_string_query = rb_str_new_cstr("query");
   rb_global_variable(&r_string_query);
   rb_str_freeze(r_string_query);

diff --git a/graphql-c_parser/ext/graphql_c_parser_ext/parser.y b/graphql-c_parser/ext/graphql_c_parser_ext/parser.y
@@ -907,7 +907,11 @@ void initialize_node_class_variables() {
   VALUE mGraphQL = rb_const_get_at(rb_cObject, rb_intern("GraphQL"));
   VALUE mGraphQLLanguage = rb_const_get_at(mGraphQL, rb_intern("Language"));
   VALUE mGraphQLLanguageNodes = rb_const_get_at(mGraphQLLanguage, rb_intern("Nodes"));
-  GraphQL_Language_Nodes_NONE = rb_const_get_at(mGraphQLLanguageNodes, rb_intern("NONE"));
+
+  GraphQL_Language_Nodes_NONE = rb_ary_new();
+  rb_global_variable(&GraphQL_Language_Nodes_NONE);
+  rb_ary_freeze(GraphQL_Language_Nodes_NONE);
+
   r_string_query = rb_str_new_cstr("query");
   rb_global_variable(&r_string_query);
   rb_str_freeze(r_string_query);