1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't set T_TYPES of NODE

Browse Source 
T_TYPES was needed once Ripper jumbled NODEs and other type
objects. However such hack was already removed.
Therefore don't need to set T_TYPES of NODE.
This commit is contained in:
yui-knk 2024-04-06 18:51:20 +09:00 committed by Yuichiro Kaneko
parent 4dd9e5cf74
commit 6f7e8e278f
2 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
node.c
View File

@ -15,7 +15,6 @@
#include "node.h" #include "node.h"
#include "rubyparser.h" #include "rubyparser.h"
#include "internal/parse.h" #include "internal/parse.h"
#define T_NODE 0x1b
#else #else
@ -87,16 +86,10 @@ rb_node_buffer_new(void)
typedef void node_itr_t(rb_ast_t *ast, void *ctx, NODE *node); typedef void node_itr_t(rb_ast_t *ast, void *ctx, NODE *node);
static void iterate_node_values(rb_ast_t *ast, node_buffer_list_t *nb, node_itr_t * func, void *ctx); static void iterate_node_values(rb_ast_t *ast, node_buffer_list_t *nb, node_itr_t * func, void *ctx);
/* Setup NODE structure.
* NODE is not an object managed by GC, but it imitates an object
* so that it can work with `RB_TYPE_P(obj, T_NODE)`.
* This dirty hack is needed because Ripper jumbles NODEs and other type
* objects.
*/
void void
rb_node_init(NODE *n, enum node_type type) rb_node_init(NODE *n, enum node_type type)
{ {
RNODE(n)->flags = T_NODE; RNODE(n)->flags = 0;
nd_init_type(RNODE(n), type); nd_init_type(RNODE(n), type);
RNODE(n)->nd_loc.beg_pos.lineno = 0; RNODE(n)->nd_loc.beg_pos.lineno = 0;
RNODE(n)->nd_loc.beg_pos.column = 0; RNODE(n)->nd_loc.beg_pos.column = 0;

2
rubyparser.h
View File

@ -1178,7 +1178,7 @@ typedef struct RNode_ERROR {
#define RNODE_ENCODING(node) ((struct RNode_ENCODING *)(node)) #define RNODE_ENCODING(node) ((struct RNode_ENCODING *)(node))
/* FL : 0..4: T_TYPES, 5: KEEP_WB, 6: PROMOTED, 7: FINALIZE, 8: UNUSED, 9: UNUSED, 10: EXIVAR, 11: FREEZE */ /* FL : 0..4: T_TYPES, 5: KEEP_WB, 6: PROMOTED, 7: FINALIZE, 8: UNUSED, 9: UNUSED, 10: EXIVAR, 11: FREEZE */
/* NODE_FL: 0..4: T_TYPES, 5: KEEP_WB, 6: PROMOTED, 7: NODE_FL_NEWLINE, /* NODE_FL: 0..4: UNUSED, 5: UNUSED, 6: UNUSED, 7: NODE_FL_NEWLINE,
* 8..14: nd_type, * 8..14: nd_type,
* 15..: nd_line * 15..: nd_line
*/ */