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

rb_io_modestr_fmode: do not goto into a branch

Browse Source 
I'm not necessarily against every goto in general, but jumping into a
branch is definitely a bad idea.  Better refactor.
This commit is contained in:
卜部昌平 2020-06-15 15:26:31 +09:00
parent 0e4ee71546
commit 9e92292e30
Notes: git 2020-06-29 11:07:02 +09:00 
Merged: https://github.com/ruby/ruby/pull/3247
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
io.c
View File

@ -5541,8 +5541,7 @@ rb_io_modestr_fmode(const char *modestr)
fmode |= FMODE_WRITABLE | FMODE_APPEND | FMODE_CREATE;
break;
default:
error:
rb_raise(rb_eArgError, "invalid access mode %s", modestr);
goto error;
}
while (*m) {
@ -5576,6 +5575,9 @@ rb_io_modestr_fmode(const char *modestr)
goto error;
return fmode;
error:
rb_raise(rb_eArgError, "invalid access mode %s", modestr);
}
int