diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-12-03 06:13:58 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-12-03 06:13:58 +0000 |
commit | a177c2762e9bbe01e2666026cc70474bdace50b5 (patch) | |
tree | 1296ec9e4dabf8378a1e8526913a84eebc52c84c /load.c | |
parent | f235dbeea5ca8aa9e433f3b1b5a8cc70970af304 (diff) |
load.c: tweak the return value
* load.c (rb_require_internal): tweak the return value, 1 and 0
correspond to true and false in Kernel#require, respectively.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@48697 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'load.c')
-rw-r--r-- | load.c | 22 |
1 files changed, 15 insertions, 7 deletions
@@ -940,10 +940,17 @@ load_ext(VALUE path) return (VALUE)dln_load(RSTRING_PTR(path)); } +/* + * returns + * 0: if already loaded (false) + * 1: successfully loaded (true) + * <0: not found (LoadError) + * >1: exception + */ static int rb_require_internal(VALUE fname, int safe) { - volatile int result = -2; + volatile int result = -1; rb_thread_t *th = GET_THREAD(); volatile VALUE errinfo = th->errinfo; int state; @@ -985,11 +992,11 @@ rb_require_internal(VALUE fname, int safe) } if (found) { if (!path || !(ftptr = load_lock(RSTRING_PTR(path)))) { - result = -1; + result = 0; } else if (!*ftptr) { rb_provide_feature(path); - result = 0; + result = 1; } else { switch (found) { @@ -1004,7 +1011,7 @@ rb_require_internal(VALUE fname, int safe) break; } rb_provide_feature(path); - result = 0; + result = 1; } } } @@ -1013,6 +1020,7 @@ rb_require_internal(VALUE fname, int safe) rb_set_safe_level_force(saved.safe); if (state) { + /* never TAG_RETURN */ return state; } @@ -1040,14 +1048,14 @@ rb_require_safe(VALUE fname, int safe) { int result = rb_require_internal(fname, safe); - if (result > 0) { + if (result > 1) { JUMP_TAG(result); } - if (result < -1) { + if (result < 0) { load_failed(fname); } - return result ? Qfalse : Qtrue; + return result ? Qtrue : Qfalse; } VALUE |