summaryrefslogtreecommitdiff
path: root/load.c
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2014-12-03 06:13:58 (GMT)
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2014-12-03 06:13:58 (GMT)
commita177c2762e9bbe01e2666026cc70474bdace50b5 (patch)
tree1296ec9e4dabf8378a1e8526913a84eebc52c84c /load.c
parentf235dbeea5ca8aa9e433f3b1b5a8cc70970af304 (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.c22
1 files changed, 15 insertions, 7 deletions
diff --git a/load.c b/load.c
index 833249a..7095ea3 100644
--- a/load.c
+++ b/load.c
@@ -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