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

Use chomp: option when chomp mode

Browse Source 
Get rid of depending on using $/ internally in String#chomp!, and
chomp the separator at once.
This commit is contained in:
Nobuyoshi Nakada 2022-01-06 23:27:16 +09:00
parent 83b987054a
commit 6baa78bb78
Notes: git 2022-01-07 09:24:07 +09:00 
Merged: https://github.com/ruby/ruby/pull/5403
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
parse.y
View File

@ -13127,9 +13127,9 @@ parser_append_options(struct parser_params *p, NODE *node)
node = block_append(p, split, node); node = block_append(p, split, node);
} }
if (p->do_chomp) { if (p->do_chomp) {
NODE *chomp = NEW_CALL(NEW_GVAR(idLASTLINE, LOC), NODE *chomp = NEW_LIT(ID2SYM(rb_intern("chomp")), LOC);
rb_intern("chomp!"), 0, LOC); chomp = list_append(p, NEW_LIST(chomp, LOC), NEW_TRUE(LOC));
node = block_append(p, chomp, node); irs = list_append(p, irs, NEW_HASH(chomp, LOC));
} }
node = NEW_WHILE(NEW_FCALL(idGets, irs, LOC), node, 1, LOC); node = NEW_WHILE(NEW_FCALL(idGets, irs, LOC), node, 1, LOC);