From cebbe18eed2e09843e9c24be9a8ea9db6d241050 Mon Sep 17 00:00:00 2001
From: yui-knk <spiketeika@gmail.com>
Date: Thu, 4 Apr 2024 23:53:47 +0900
Subject: [PATCH] Remove needless check

`nodetype_markable_p` always returns `false` then
`rb_ast_node_type_change` never calls `rb_bug`.
---
 node.c | 22 ----------------------
 node.h |  3 ---
 2 files changed, 25 deletions(-)

diff --git a/node.c b/node.c
index c66770307e..8ef7bb7c46 100644
--- a/node.c
+++ b/node.c
@@ -283,25 +283,6 @@ rb_ast_newnode(rb_ast_t *ast, enum node_type type, size_t size, size_t alignment
     return ast_newnode_in_bucket(ast, bucket, size, alignment);
 }
 
-#if RUBY_DEBUG
-RBIMPL_ATTR_PURE()
-static bool
-nodetype_markable_p(enum node_type type)
-{
-    return false;
-}
-
-void
-rb_ast_node_type_change(NODE *n, enum node_type type)
-{
-    enum node_type old_type = nd_type(n);
-    if (nodetype_markable_p(old_type) != nodetype_markable_p(type)) {
-        rb_bug("node type changed: %s -> %s",
-               ruby_node_name(old_type), ruby_node_name(type));
-    }
-}
-#endif
-
 rb_ast_id_table_t *
 rb_ast_new_local_table(rb_ast_t *ast, int size)
 {
@@ -421,8 +402,5 @@ rb_ast_dispose(rb_ast_t *ast)
 VALUE
 rb_node_set_type(NODE *n, enum node_type t)
 {
-#if RUBY_DEBUG
-    rb_ast_node_type_change(n, t);
-#endif
     return nd_init_type(n, t);
 }
diff --git a/node.h b/node.h
index 2cc304125e..d5522c82ec 100644
--- a/node.h
+++ b/node.h
@@ -53,9 +53,6 @@ rb_ast_t *rb_ast_new(void);
 #endif
 size_t rb_ast_memsize(const rb_ast_t*);
 void rb_ast_dispose(rb_ast_t*);
-#if RUBY_DEBUG
-void rb_ast_node_type_change(NODE *n, enum node_type type);
-#endif
 const char *ruby_node_name(int node);
 void rb_node_init(NODE *n, enum node_type type);