ref: 70b0d8b902fd86542c6233570d8f7c724a45c1fb
parent: e4a6860f6ec750782d9e50c8b29883a5cf92eda2
author: Tor Andersson <[email protected]>
date: Mon Feb 24 19:56:33 EST 2014
Change C functions to not have a return value. They should always push a return value on the JS stack.
--- a/js.h
+++ b/js.h
@@ -26,7 +26,7 @@
typedef struct js_State js_State;
-typedef int (*js_CFunction)(js_State *J, unsigned int argc);
+typedef void (*js_CFunction)(js_State *J, unsigned int argc);
/* Basic functions */
js_State *js_newstate(void);
--- a/jsarray.c
+++ b/jsarray.c
@@ -45,7 +45,7 @@
js_delproperty(J, idx, buf);
}
-static int jsB_new_Array(js_State *J, unsigned int argc)
+static void jsB_new_Array(js_State *J, unsigned int argc)
{
unsigned int i;
@@ -65,11 +65,9 @@
js_setindex(J, -2, i - 1);
}
}
-
- return 1;
}
-static int Ap_concat(js_State *J, unsigned int argc)
+static void Ap_concat(js_State *J, unsigned int argc)
{
unsigned int n, len, i;
@@ -95,11 +93,9 @@
++n;
}
}
-
- return 1;
}
-static int Ap_join(js_State *J, unsigned int argc)
+static void Ap_join(js_State *J, unsigned int argc)
{
char * volatile out = NULL;
const char *sep;
@@ -119,7 +115,7 @@
if (len == 0) {
js_pushliteral(J, "");
- return 1;
+ return;
}
if (js_try(J)) {
@@ -152,10 +148,9 @@
js_pushstring(J, out);
js_endtry(J);
free(out);
- return 1;
}
-static int Ap_pop(js_State *J, unsigned int argc)
+static void Ap_pop(js_State *J, unsigned int argc)
{
unsigned int n;
@@ -169,11 +164,9 @@
js_setlength(J, 0, 0);
js_pushundefined(J);
}
-
- return 1;
}
-static int Ap_push(js_State *J, unsigned int argc)
+static void Ap_push(js_State *J, unsigned int argc)
{
unsigned int i, n;
@@ -187,10 +180,9 @@
js_setlength(J, 0, n);
js_pushnumber(J, n);
- return 1;
}
-static int Ap_reverse(js_State *J, unsigned int argc)
+static void Ap_reverse(js_State *J, unsigned int argc)
{
unsigned int len, middle, lower;
@@ -216,10 +208,9 @@
}
js_copy(J, 0);
- return 1;
}
-static int Ap_shift(js_State *J, unsigned int argc)
+static void Ap_shift(js_State *J, unsigned int argc)
{
unsigned int k, len;
@@ -228,7 +219,7 @@
if (len == 0) {
js_setlength(J, 0, 0);
js_pushundefined(J);
- return 1;
+ return;
}
js_getindex(J, 0, 0);
@@ -242,11 +233,9 @@
js_delindex(J, 0, len - 1);
js_setlength(J, 0, len - 1);
-
- return 1;
}
-static int Ap_slice(js_State *J, unsigned int argc)
+static void Ap_slice(js_State *J, unsigned int argc)
{
unsigned int len, s, e, n;
double sv, ev;
@@ -266,8 +255,6 @@
for (n = 0; s < e; ++s, ++n)
if (js_hasindex(J, 0, s))
js_setindex(J, -2, n);
-
- return 1;
}
static int compare(js_State *J, unsigned int x, unsigned int y, int *hasx, int *hasy, int hasfn)
@@ -306,7 +293,7 @@
return 0;
}
-static int Ap_sort(js_State *J, unsigned int argc)
+static void Ap_sort(js_State *J, unsigned int argc)
{
unsigned int len, i, k;
int hasx, hasy, hasfn;
@@ -333,10 +320,9 @@
}
js_copy(J, 0);
- return 1;
}
-static int Ap_splice(js_State *J, unsigned int argc)
+static void Ap_splice(js_State *J, unsigned int argc)
{
unsigned int len, start, del, add, k;
double f;
@@ -384,11 +370,9 @@
}
js_setlength(J, 0, len - del + add);
-
- return 1;
}
-static int Ap_unshift(js_State *J, unsigned int argc)
+static void Ap_unshift(js_State *J, unsigned int argc)
{
unsigned int i, k, len;
@@ -411,16 +395,15 @@
js_setlength(J, 0, len + argc);
js_pushnumber(J, len + argc);
- return 1;
}
-static int Ap_toString(js_State *J, unsigned int argc)
+static void Ap_toString(js_State *J, unsigned int argc)
{
js_pop(J, argc);
return Ap_join(J, 0);
}
-static int Ap_indexOf(js_State *J, unsigned int argc)
+static void Ap_indexOf(js_State *J, unsigned int argc)
{
int k, len, from;
@@ -434,7 +417,7 @@
if (js_hasindex(J, 0, k)) {
if (js_strictequal(J)) {
js_pushnumber(J, k);
- return 1;
+ return;
}
js_pop(J, 1);
}
@@ -441,10 +424,9 @@
}
js_pushnumber(J, -1);
- return 1;
}
-static int Ap_lastIndexOf(js_State *J, unsigned int argc)
+static void Ap_lastIndexOf(js_State *J, unsigned int argc)
{
int k, len, from;
@@ -458,7 +440,7 @@
if (js_hasindex(J, 0, k)) {
if (js_strictequal(J)) {
js_pushnumber(J, k);
- return 1;
+ return;
}
js_pop(J, 1);
}
@@ -465,10 +447,9 @@
}
js_pushnumber(J, -1);
- return 1;
}
-static int Ap_every(js_State *J, unsigned int argc)
+static void Ap_every(js_State *J, unsigned int argc)
{
int k, len;
@@ -488,16 +469,15 @@
js_copy(J, 0);
js_call(J, 3);
if (!js_toboolean(J, -1))
- return 1;
+ return;
js_pop(J, 2);
}
}
js_pushboolean(J, 1);
- return 1;
}
-static int Ap_some(js_State *J, unsigned int argc)
+static void Ap_some(js_State *J, unsigned int argc)
{
int k, len;
@@ -517,16 +497,15 @@
js_copy(J, 0);
js_call(J, 3);
if (js_toboolean(J, -1))
- return 1;
+ return;
js_pop(J, 2);
}
}
js_pushboolean(J, 0);
- return 1;
}
-static int Ap_forEach(js_State *J, unsigned int argc)
+static void Ap_forEach(js_State *J, unsigned int argc)
{
int k, len;
@@ -549,10 +528,10 @@
}
}
- return 0;
+ js_pushundefined(J);
}
-static int Ap_map(js_State *J, unsigned int argc)
+static void Ap_map(js_State *J, unsigned int argc)
{
int k, len;
@@ -577,11 +556,9 @@
js_pop(J, 1);
}
}
-
- return 1;
}
-static int Ap_filter(js_State *J, unsigned int argc)
+static void Ap_filter(js_State *J, unsigned int argc)
{
int k, to, len;
@@ -611,11 +588,9 @@
}
}
}
-
- return 1;
}
-static int Ap_reduce(js_State *J, unsigned int argc)
+static void Ap_reduce(js_State *J, unsigned int argc)
{
int k, len;
@@ -652,10 +627,10 @@
++k;
}
- return 1; /* return accumulator */
+ /* return accumulator */
}
-static int Ap_reduceRight(js_State *J, unsigned int argc)
+static void Ap_reduceRight(js_State *J, unsigned int argc)
{
int k, len;
@@ -692,18 +667,17 @@
--k;
}
- return 1; /* return accumulator */
+ /* return accumulator */
}
-static int A_isArray(js_State *J, unsigned int argc)
+static void A_isArray(js_State *J, unsigned int argc)
{
if (js_isobject(J, 1)) {
js_Object *T = js_toobject(J, 1);
js_pushboolean(J, T->type == JS_CARRAY);
- return 1;
+ } else {
+ js_pushboolean(J, 0);
}
- js_pushboolean(J, 0);
- return 1;
}
void jsB_initarray(js_State *J)
--- a/jsboolean.c
+++ b/jsboolean.c
@@ -2,32 +2,28 @@
#include "jsvalue.h"
#include "jsbuiltin.h"
-static int jsB_new_Boolean(js_State *J, unsigned int argc)
+static void jsB_new_Boolean(js_State *J, unsigned int argc)
{
js_newboolean(J, js_toboolean(J, 1));
- return 1;
}
-static int jsB_Boolean(js_State *J, unsigned int argc)
+static void jsB_Boolean(js_State *J, unsigned int argc)
{
js_pushboolean(J, js_toboolean(J, 1));
- return 1;
}
-static int Bp_toString(js_State *J, unsigned int argc)
+static void Bp_toString(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (self->type != JS_CBOOLEAN) js_typeerror(J, "not a boolean");
js_pushliteral(J, self->u.boolean ? "true" : "false");
- return 1;
}
-static int Bp_valueOf(js_State *J, unsigned int argc)
+static void Bp_valueOf(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (self->type != JS_CBOOLEAN) js_typeerror(J, "not a boolean");
js_pushboolean(J, self->u.boolean);
- return 1;
}
void jsB_initboolean(js_State *J)
--- a/jsbuiltin.c
+++ b/jsbuiltin.c
@@ -28,50 +28,45 @@
js_defproperty(J, -2, name, JS_DONTENUM);
}
-static int jsB_eval(js_State *J, unsigned int argc)
+static void jsB_eval(js_State *J, unsigned int argc)
{
if (!js_isstring(J, -1)) {
js_copy(J, 1);
- return 1;
+ return;
}
js_loadstring(J, "(eval)", js_tostring(J, -1));
js_pushglobal(J);
js_call(J, 0);
- return 1;
}
-static int jsB_parseInt(js_State *J, unsigned int argc)
+static void jsB_parseInt(js_State *J, unsigned int argc)
{
const char *s = js_tostring(J, 1);
double radix = js_isdefined(J, 2) ? js_tonumber(J, 2) : 10;
while (jsY_iswhite(*s) || jsY_isnewline(*s)) ++s;
js_pushnumber(J, strtol(s, NULL, radix == 0 ? 10 : radix));
- return 1;
}
-static int jsB_parseFloat(js_State *J, unsigned int argc)
+static void jsB_parseFloat(js_State *J, unsigned int argc)
{
const char *s = js_tostring(J, 1);
while (jsY_iswhite(*s) || jsY_isnewline(*s)) ++s;
js_pushnumber(J, strtod(s, NULL));
- return 1;
}
-static int jsB_isNaN(js_State *J, unsigned int argc)
+static void jsB_isNaN(js_State *J, unsigned int argc)
{
double n = js_tonumber(J, 1);
js_pushboolean(J, isnan(n));
- return 1;
}
-static int jsB_isFinite(js_State *J, unsigned int argc)
+static void jsB_isFinite(js_State *J, unsigned int argc)
{
double n = js_tonumber(J, 1);
js_pushboolean(J, isfinite(n));
- return 1;
}
-static int Encode(js_State *J, const char *str, const char *unescaped)
+static void Encode(js_State *J, const char *str, const char *unescaped)
{
js_Buffer *sb = NULL;
@@ -96,10 +91,9 @@
js_pushstring(J, sb ? sb->s : "");
js_endtry(J);
free(sb);
- return 1;
}
-static int Decode(js_State *J, const char *str, const char *reserved)
+static void Decode(js_State *J, const char *str, const char *reserved)
{
js_Buffer *sb = NULL;
int a, b;
@@ -134,7 +128,6 @@
js_pushstring(J, sb ? sb->s : "");
js_endtry(J);
free(sb);
- return 1;
}
#define URIRESERVED ";/?:@&=+$,"
@@ -143,24 +136,24 @@
#define URIMARK "-_.!~*`()"
#define URIUNESCAPED URIALPHA URIDIGIT URIMARK
-static int jsB_decodeURI(js_State *J, unsigned int argc)
+static void jsB_decodeURI(js_State *J, unsigned int argc)
{
- return Decode(J, js_tostring(J, 1), URIRESERVED "#");
+ Decode(J, js_tostring(J, 1), URIRESERVED "#");
}
-static int jsB_decodeURIComponent(js_State *J, unsigned int argc)
+static void jsB_decodeURIComponent(js_State *J, unsigned int argc)
{
- return Decode(J, js_tostring(J, 1), "");
+ Decode(J, js_tostring(J, 1), "");
}
-static int jsB_encodeURI(js_State *J, unsigned int argc)
+static void jsB_encodeURI(js_State *J, unsigned int argc)
{
- return Encode(J, js_tostring(J, 1), URIUNESCAPED URIRESERVED "#");
+ Encode(J, js_tostring(J, 1), URIUNESCAPED URIRESERVED "#");
}
-static int jsB_encodeURIComponent(js_State *J, unsigned int argc)
+static void jsB_encodeURIComponent(js_State *J, unsigned int argc)
{
- return Encode(J, js_tostring(J, 1), URIUNESCAPED);
+ Encode(J, js_tostring(J, 1), URIUNESCAPED);
}
void jsB_init(js_State *J)
--- a/jsdate.c
+++ b/jsdate.c
@@ -253,7 +253,7 @@
return self->u.number;
}
-static int js_setdate(js_State *J, int idx, double t)
+static void js_setdate(js_State *J, int idx, double t)
{
js_Object *self = js_toobject(J, idx);
if (self->type != JS_CDATE)
@@ -260,17 +260,15 @@
js_typeerror(J, "not a date");
self->u.number = TimeClip(t);
js_pushnumber(J, self->u.number);
- return 1;
}
-static int D_parse(js_State *J, unsigned int argc)
+static void D_parse(js_State *J, unsigned int argc)
{
double t = parseDate(js_tostring(J, 1));
js_pushnumber(J, t);
- return 1;
}
-static int D_UTC(js_State *J, unsigned int argc)
+static void D_UTC(js_State *J, unsigned int argc)
{
double y, m, d, H, M, S, ms, t;
y = js_tonumber(J, 1);
@@ -284,22 +282,19 @@
t = MakeDate(MakeDay(y, m, d), MakeTime(H, M, S, ms));
t = TimeClip(t);
js_pushnumber(J, t);
- return 1;
}
-static int D_now(js_State *J, unsigned int argc)
+static void D_now(js_State *J, unsigned int argc)
{
js_pushnumber(J, Now());
- return 1;
}
-static int jsB_Date(js_State *J, unsigned int argc)
+static void jsB_Date(js_State *J, unsigned int argc)
{
js_pushstring(J, fmtlocal(FMT_DATETIME, Now()));
- return 1;
}
-static int jsB_new_Date(js_State *J, unsigned int argc)
+static void jsB_new_Date(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Value v;
@@ -331,176 +326,152 @@
obj->u.number = t;
js_pushobject(J, obj);
- return 1;
}
-static int Dp_valueOf(js_State *J, unsigned int argc)
+static void Dp_valueOf(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, t);
- return 1;
}
-static int Dp_toString(js_State *J, unsigned int argc)
+static void Dp_toString(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushstring(J, fmtlocal(FMT_DATETIME, t));
- return 1;
}
-static int Dp_toDateString(js_State *J, unsigned int argc)
+static void Dp_toDateString(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushstring(J, fmtlocal(FMT_DATE, t));
- return 1;
}
-static int Dp_toTimeString(js_State *J, unsigned int argc)
+static void Dp_toTimeString(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushstring(J, fmtlocal(FMT_TIME, t));
- return 1;
}
-static int Dp_toUTCString(js_State *J, unsigned int argc)
+static void Dp_toUTCString(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushstring(J, fmtutc(FMT_DATETIME, t));
- return 1;
}
-static int Dp_toISOString(js_State *J, unsigned int argc)
+static void Dp_toISOString(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushstring(J, fmtutc(FMT_DATETIME_ISO, t));
- return 1;
}
-static int Dp_getFullYear(js_State *J, unsigned int argc)
+static void Dp_getFullYear(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, YearFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getMonth(js_State *J, unsigned int argc)
+static void Dp_getMonth(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, MonthFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getDate(js_State *J, unsigned int argc)
+static void Dp_getDate(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, DateFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getDay(js_State *J, unsigned int argc)
+static void Dp_getDay(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, WeekDay(LocalTime(t)));
- return 1;
}
-static int Dp_getHours(js_State *J, unsigned int argc)
+static void Dp_getHours(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, HourFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getMinutes(js_State *J, unsigned int argc)
+static void Dp_getMinutes(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, MinFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getSeconds(js_State *J, unsigned int argc)
+static void Dp_getSeconds(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, SecFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getMilliseconds(js_State *J, unsigned int argc)
+static void Dp_getMilliseconds(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, msFromTime(LocalTime(t)));
- return 1;
}
-static int Dp_getUTCFullYear(js_State *J, unsigned int argc)
+static void Dp_getUTCFullYear(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, YearFromTime(t));
- return 1;
}
-static int Dp_getUTCMonth(js_State *J, unsigned int argc)
+static void Dp_getUTCMonth(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, MonthFromTime(t));
- return 1;
}
-static int Dp_getUTCDate(js_State *J, unsigned int argc)
+static void Dp_getUTCDate(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, DateFromTime(t));
- return 1;
}
-static int Dp_getUTCDay(js_State *J, unsigned int argc)
+static void Dp_getUTCDay(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, WeekDay(t));
- return 1;
}
-static int Dp_getUTCHours(js_State *J, unsigned int argc)
+static void Dp_getUTCHours(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, HourFromTime(t));
- return 1;
}
-static int Dp_getUTCMinutes(js_State *J, unsigned int argc)
+static void Dp_getUTCMinutes(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, MinFromTime(t));
- return 1;
}
-static int Dp_getUTCSeconds(js_State *J, unsigned int argc)
+static void Dp_getUTCSeconds(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, SecFromTime(t));
- return 1;
}
-static int Dp_getUTCMilliseconds(js_State *J, unsigned int argc)
+static void Dp_getUTCMilliseconds(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, msFromTime(t));
- return 1;
}
-static int Dp_getTimezoneOffset(js_State *J, unsigned int argc)
+static void Dp_getTimezoneOffset(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
js_pushnumber(J, (t - LocalTime(t)) / msPerMinute);
- return 1;
}
-static int Dp_setTime(js_State *J, unsigned int argc)
+static void Dp_setTime(js_State *J, unsigned int argc)
{
- return js_setdate(J, 0, js_tonumber(J, 1));
+ js_setdate(J, 0, js_tonumber(J, 1));
}
-static int Dp_setMilliseconds(js_State *J, unsigned int argc)
+static void Dp_setMilliseconds(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double h = HourFromTime(t);
@@ -507,10 +478,10 @@
double m = MinFromTime(t);
double s = SecFromTime(t);
double ms = js_tonumber(J, 1);
- return js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
+ js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
}
-static int Dp_setSeconds(js_State *J, unsigned int argc)
+static void Dp_setSeconds(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double h = HourFromTime(t);
@@ -517,10 +488,10 @@
double m = MinFromTime(t);
double s = js_tonumber(J, 1);
double ms = argc > 1 ? js_tonumber(J, 2) : msFromTime(t);
- return js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
+ js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
}
-static int Dp_setMinutes(js_State *J, unsigned int argc)
+static void Dp_setMinutes(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double h = HourFromTime(t);
@@ -527,10 +498,10 @@
double m = js_tonumber(J, 1);
double s = argc > 1 ? js_tonumber(J, 2) : SecFromTime(t);
double ms = argc > 2 ? js_tonumber(J, 3) : msFromTime(t);
- return js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
+ js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
}
-static int Dp_setHours(js_State *J, unsigned int argc)
+static void Dp_setHours(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double h = js_tonumber(J, 1);
@@ -537,37 +508,37 @@
double m = argc > 1 ? js_tonumber(J, 2) : HourFromTime(t);
double s = argc > 2 ? js_tonumber(J, 3) : SecFromTime(t);
double ms = argc > 3 ? js_tonumber(J, 4) : msFromTime(t);
- return js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
+ js_setdate(J, 0, UTC(MakeDate(Day(t), MakeTime(h, m, s, ms))));
}
-static int Dp_setDate(js_State *J, unsigned int argc)
+static void Dp_setDate(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double y = YearFromTime(t);
double m = MonthFromTime(t);
double d = js_tonumber(J, 1);
- return js_setdate(J, 0, UTC(MakeDate(MakeDay(y, m, d), TimeWithinDay(t))));
+ js_setdate(J, 0, UTC(MakeDate(MakeDay(y, m, d), TimeWithinDay(t))));
}
-static int Dp_setMonth(js_State *J, unsigned int argc)
+static void Dp_setMonth(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double y = YearFromTime(t);
double m = js_tonumber(J, 1);
double d = argc > 1 ? js_tonumber(J, 3) : DateFromTime(t);
- return js_setdate(J, 0, UTC(MakeDate(MakeDay(y, m, d), TimeWithinDay(t))));
+ js_setdate(J, 0, UTC(MakeDate(MakeDay(y, m, d), TimeWithinDay(t))));
}
-static int Dp_setFullYear(js_State *J, unsigned int argc)
+static void Dp_setFullYear(js_State *J, unsigned int argc)
{
double t = LocalTime(js_todate(J, 0));
double y = js_tonumber(J, 1);
double m = argc > 1 ? js_tonumber(J, 2) : MonthFromTime(t);
double d = argc > 2 ? js_tonumber(J, 3) : DateFromTime(t);
- return js_setdate(J, 0, UTC(MakeDate(MakeDay(y, m, d), TimeWithinDay(t))));
+ js_setdate(J, 0, UTC(MakeDate(MakeDay(y, m, d), TimeWithinDay(t))));
}
-static int Dp_setUTCMilliseconds(js_State *J, unsigned int argc)
+static void Dp_setUTCMilliseconds(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double h = HourFromTime(t);
@@ -574,10 +545,10 @@
double m = MinFromTime(t);
double s = SecFromTime(t);
double ms = js_tonumber(J, 1);
- return js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
+ js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
}
-static int Dp_setUTCSeconds(js_State *J, unsigned int argc)
+static void Dp_setUTCSeconds(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double h = HourFromTime(t);
@@ -584,10 +555,10 @@
double m = MinFromTime(t);
double s = js_tonumber(J, 1);
double ms = argc > 1 ? js_tonumber(J, 2) : msFromTime(t);
- return js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
+ js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
}
-static int Dp_setUTCMinutes(js_State *J, unsigned int argc)
+static void Dp_setUTCMinutes(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double h = HourFromTime(t);
@@ -594,10 +565,10 @@
double m = js_tonumber(J, 1);
double s = argc > 1 ? js_tonumber(J, 2) : SecFromTime(t);
double ms = argc > 2 ? js_tonumber(J, 3) : msFromTime(t);
- return js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
+ js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
}
-static int Dp_setUTCHours(js_State *J, unsigned int argc)
+static void Dp_setUTCHours(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double h = js_tonumber(J, 1);
@@ -604,43 +575,43 @@
double m = argc > 1 ? js_tonumber(J, 2) : HourFromTime(t);
double s = argc > 2 ? js_tonumber(J, 3) : SecFromTime(t);
double ms = argc > 3 ? js_tonumber(J, 4) : msFromTime(t);
- return js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
+ js_setdate(J, 0, MakeDate(Day(t), MakeTime(h, m, s, ms)));
}
-static int Dp_setUTCDate(js_State *J, unsigned int argc)
+static void Dp_setUTCDate(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double y = YearFromTime(t);
double m = MonthFromTime(t);
double d = js_tonumber(J, 1);
- return js_setdate(J, 0, MakeDate(MakeDay(y, m, d), TimeWithinDay(t)));
+ js_setdate(J, 0, MakeDate(MakeDay(y, m, d), TimeWithinDay(t)));
}
-static int Dp_setUTCMonth(js_State *J, unsigned int argc)
+static void Dp_setUTCMonth(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double y = YearFromTime(t);
double m = js_tonumber(J, 1);
double d = argc > 1 ? js_tonumber(J, 3) : DateFromTime(t);
- return js_setdate(J, 0, MakeDate(MakeDay(y, m, d), TimeWithinDay(t)));
+ js_setdate(J, 0, MakeDate(MakeDay(y, m, d), TimeWithinDay(t)));
}
-static int Dp_setUTCFullYear(js_State *J, unsigned int argc)
+static void Dp_setUTCFullYear(js_State *J, unsigned int argc)
{
double t = js_todate(J, 0);
double y = js_tonumber(J, 1);
double m = argc > 1 ? js_tonumber(J, 2) : MonthFromTime(t);
double d = argc > 2 ? js_tonumber(J, 3) : DateFromTime(t);
- return js_setdate(J, 0, MakeDate(MakeDay(y, m, d), TimeWithinDay(t)));
+ js_setdate(J, 0, MakeDate(MakeDay(y, m, d), TimeWithinDay(t)));
}
-static int Dp_toJSON(js_State *J, unsigned int argc)
+static void Dp_toJSON(js_State *J, unsigned int argc)
{
js_Object *obj = js_toobject(J, 0);
js_Value tv = js_toprimitive(J, 0, JS_HNUMBER);
if (tv.type == JS_TNULL && !isfinite(tv.u.number)) {
js_pushnull(J);
- return 1;
+ return;
}
js_pushobject(J, obj);
js_getproperty(J, -1, "toISOString");
@@ -648,7 +619,6 @@
js_typeerror(J, "Date.prototype.toJSON: this.toISOString not a function");
js_pushobject(J, obj);
js_call(J, 0);
- return 1;
}
void jsB_initdate(js_State *J)
--- a/jserror.c
+++ b/jserror.c
@@ -5,7 +5,7 @@
#define QQ(X) #X
#define Q(X) QQ(X)
-static int Ep_toString(js_State *J, unsigned int argc)
+static void Ep_toString(js_State *J, unsigned int argc)
{
const char *name = "Error";
const char *message = "";
@@ -34,7 +34,6 @@
js_pushliteral(J, message);
js_concat(J);
}
- return 1;
}
static int jsB_ErrorX(js_State *J, unsigned int argc, js_Object *prototype)
@@ -55,8 +54,8 @@
}
#define DERROR(name, Name) \
- static int jsB_##Name(js_State *J, unsigned int argc) { \
- return jsB_ErrorX(J, argc, J->Name##_prototype); \
+ static void jsB_##Name(js_State *J, unsigned int argc) { \
+ jsB_ErrorX(J, argc, J->Name##_prototype); \
} \
void js_new##name(js_State *J, const char *s) { \
js_newerrorx(J, s, J->Name##_prototype); \
--- a/jsfunction.c
+++ b/jsfunction.c
@@ -4,7 +4,7 @@
#include "jsvalue.h"
#include "jsbuiltin.h"
-static int jsB_Function(js_State *J, unsigned int argc)
+static void jsB_Function(js_State *J, unsigned int argc)
{
const char *source;
js_Buffer *sb;
@@ -40,16 +40,14 @@
jsP_freeparse(J);
js_newfunction(J, fun, J->GE);
- return 1;
}
-static int jsB_Function_prototype(js_State *J, unsigned int argc)
+static void jsB_Function_prototype(js_State *J, unsigned int argc)
{
js_pushundefined(J);
- return 1;
}
-static int Fp_toString(js_State *J, unsigned int argc)
+static void Fp_toString(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
char *s;
@@ -78,11 +76,9 @@
} else {
js_pushliteral(J, "function () { ... }");
}
-
- return 1;
}
-static int Fp_apply(js_State *J, unsigned int argc)
+static void Fp_apply(js_State *J, unsigned int argc)
{
int i, n;
char name[20];
@@ -103,10 +99,9 @@
}
js_call(J, n);
- return 1;
}
-static int Fp_call(js_State *J, unsigned int argc)
+static void Fp_call(js_State *J, unsigned int argc)
{
unsigned int i;
@@ -119,7 +114,6 @@
js_copy(J, i);
js_call(J, argc - 1);
- return 1;
}
void jsB_initfunction(js_State *J)
--- a/jsi.h
+++ b/jsi.h
@@ -53,7 +53,7 @@
void js_dup1rot3(js_State *J);
void js_dup1rot4(js_State *J);
-int js_RegExp_prototype_exec(js_State *J, js_Regexp *re, const char *text);
+void js_RegExp_prototype_exec(js_State *J, js_Regexp *re, const char *text);
/* Exception handling */
--- a/jsmath.c
+++ b/jsmath.c
@@ -2,103 +2,87 @@
#include "jsvalue.h"
#include "jsbuiltin.h"
-static int Math_abs(js_State *J, unsigned int argc)
+static void Math_abs(js_State *J, unsigned int argc)
{
js_pushnumber(J, abs(js_tonumber(J, 1)));
- return 1;
}
-static int Math_acos(js_State *J, unsigned int argc)
+static void Math_acos(js_State *J, unsigned int argc)
{
js_pushnumber(J, acos(js_tonumber(J, 1)));
- return 1;
}
-static int Math_asin(js_State *J, unsigned int argc)
+static void Math_asin(js_State *J, unsigned int argc)
{
js_pushnumber(J, asin(js_tonumber(J, 1)));
- return 1;
}
-static int Math_atan(js_State *J, unsigned int argc)
+static void Math_atan(js_State *J, unsigned int argc)
{
js_pushnumber(J, atan(js_tonumber(J, 1)));
- return 1;
}
-static int Math_atan2(js_State *J, unsigned int argc)
+static void Math_atan2(js_State *J, unsigned int argc)
{
js_pushnumber(J, atan2(js_tonumber(J, 1), js_tonumber(J, 2)));
- return 1;
}
-static int Math_ceil(js_State *J, unsigned int argc)
+static void Math_ceil(js_State *J, unsigned int argc)
{
js_pushnumber(J, ceil(js_tonumber(J, 1)));
- return 1;
}
-static int Math_cos(js_State *J, unsigned int argc)
+static void Math_cos(js_State *J, unsigned int argc)
{
js_pushnumber(J, cos(js_tonumber(J, 1)));
- return 1;
}
-static int Math_exp(js_State *J, unsigned int argc)
+static void Math_exp(js_State *J, unsigned int argc)
{
js_pushnumber(J, exp(js_tonumber(J, 1)));
- return 1;
}
-static int Math_floor(js_State *J, unsigned int argc)
+static void Math_floor(js_State *J, unsigned int argc)
{
js_pushnumber(J, floor(js_tonumber(J, 1)));
- return 1;
}
-static int Math_log(js_State *J, unsigned int argc)
+static void Math_log(js_State *J, unsigned int argc)
{
js_pushnumber(J, log(js_tonumber(J, 1)));
- return 1;
}
-static int Math_pow(js_State *J, unsigned int argc)
+static void Math_pow(js_State *J, unsigned int argc)
{
js_pushnumber(J, pow(js_tonumber(J, 1), js_tonumber(J, 2)));
- return 1;
}
-static int Math_random(js_State *J, unsigned int argc)
+static void Math_random(js_State *J, unsigned int argc)
{
js_pushnumber(J, (double)rand() / (RAND_MAX - 1));
- return 1;
}
-static int Math_round(js_State *J, unsigned int argc)
+static void Math_round(js_State *J, unsigned int argc)
{
js_pushnumber(J, round(js_tonumber(J, 1)));
- return 1;
}
-static int Math_sin(js_State *J, unsigned int argc)
+static void Math_sin(js_State *J, unsigned int argc)
{
js_pushnumber(J, sin(js_tonumber(J, 1)));
- return 1;
}
-static int Math_sqrt(js_State *J, unsigned int argc)
+static void Math_sqrt(js_State *J, unsigned int argc)
{
js_pushnumber(J, sqrt(js_tonumber(J, 1)));
- return 1;
}
-static int Math_tan(js_State *J, unsigned int argc)
+static void Math_tan(js_State *J, unsigned int argc)
{
js_pushnumber(J, tan(js_tonumber(J, 1)));
- return 1;
}
-static int Math_max(js_State *J, unsigned int argc)
+static void Math_max(js_State *J, unsigned int argc)
{
double n = js_tonumber(J, 1);
unsigned int i;
@@ -107,10 +91,9 @@
n = n > m ? n : m;
}
js_pushnumber(J, n);
- return 1;
}
-static int Math_min(js_State *J, unsigned int argc)
+static void Math_min(js_State *J, unsigned int argc)
{
double n = js_tonumber(J, 1);
unsigned int i;
@@ -119,7 +102,6 @@
n = n < m ? n : m;
}
js_pushnumber(J, n);
- return 1;
}
void jsB_initmath(js_State *J)
--- a/jsnumber.c
+++ b/jsnumber.c
@@ -2,35 +2,31 @@
#include "jsvalue.h"
#include "jsbuiltin.h"
-static int jsB_new_Number(js_State *J, unsigned int argc)
+static void jsB_new_Number(js_State *J, unsigned int argc)
{
js_newnumber(J, js_isdefined(J, 1) ? js_tonumber(J, 1) : 0);
- return 1;
}
-static int jsB_Number(js_State *J, unsigned int argc)
+static void jsB_Number(js_State *J, unsigned int argc)
{
js_pushnumber(J, js_isdefined(J, 1) ? js_tonumber(J, 1) : 0);
- return 1;
}
-static int Np_valueOf(js_State *J, unsigned int argc)
+static void Np_valueOf(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (self->type != JS_CNUMBER) js_typeerror(J, "not a number");
js_pushnumber(J, self->u.number);
- return 1;
}
-static int Np_toString(js_State *J, unsigned int argc)
+static void Np_toString(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (self->type != JS_CNUMBER) js_typeerror(J, "not a number");
js_pushliteral(J, jsV_numbertostring(J, self->u.number));
- return 1;
}
-static int Np_toFixed(js_State *J, unsigned int argc)
+static void Np_toFixed(js_State *J, unsigned int argc)
{
char buf[40];
js_Object *self = js_toobject(J, 0);
@@ -38,10 +34,9 @@
if (self->type != JS_CNUMBER) js_typeerror(J, "not a number");
sprintf(buf, "%.*f", width, self->u.number);
js_pushstring(J, buf);
- return 1;
}
-static int Np_toExponential(js_State *J, unsigned int argc)
+static void Np_toExponential(js_State *J, unsigned int argc)
{
char buf[40];
js_Object *self = js_toobject(J, 0);
@@ -49,10 +44,9 @@
if (self->type != JS_CNUMBER) js_typeerror(J, "not a number");
sprintf(buf, "%.*e", width, self->u.number);
js_pushstring(J, buf);
- return 1;
}
-static int Np_toPrecision(js_State *J, unsigned int argc)
+static void Np_toPrecision(js_State *J, unsigned int argc)
{
char buf[40];
js_Object *self = js_toobject(J, 0);
@@ -60,7 +54,6 @@
if (self->type != JS_CNUMBER) js_typeerror(J, "not a number");
sprintf(buf, "%.*g", width, self->u.number);
js_pushstring(J, buf);
- return 1;
}
void jsB_initnumber(js_State *J)
--- a/jsobject.c
+++ b/jsobject.c
@@ -2,25 +2,23 @@
#include "jsvalue.h"
#include "jsbuiltin.h"
-static int jsB_new_Object(js_State *J, unsigned int argc)
+static void jsB_new_Object(js_State *J, unsigned int argc)
{
if (argc == 0 || js_isundefined(J, 1) || js_isnull(J, 1))
js_newobject(J);
else
js_pushobject(J, js_toobject(J, 1));
- return 1;
}
-static int jsB_Object(js_State *J, unsigned int argc)
+static void jsB_Object(js_State *J, unsigned int argc)
{
if (argc == 0 || js_isundefined(J, 1) || js_isnull(J, 1))
js_newobject(J);
else
js_pushobject(J, js_toobject(J, 1));
- return 1;
}
-static int Op_toString(js_State *J, unsigned int argc)
+static void Op_toString(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
switch (self->type) {
@@ -46,25 +44,22 @@
js_concat(J);
break;
}
- return 1;
}
-static int Op_valueOf(js_State *J, unsigned int argc)
+static void Op_valueOf(js_State *J, unsigned int argc)
{
js_copy(J, 0);
- return 1;
}
-static int Op_hasOwnProperty(js_State *J, unsigned int argc)
+static void Op_hasOwnProperty(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
const char *name = js_tostring(J, 1);
js_Property *ref = jsV_getownproperty(J, self, name);
js_pushboolean(J, ref != NULL);
- return 1;
}
-static int Op_isPrototypeOf(js_State *J, unsigned int argc)
+static void Op_isPrototypeOf(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (js_isobject(J, 1)) {
@@ -73,24 +68,22 @@
V = V->prototype;
if (V == self) {
js_pushboolean(J, 1);
- return 1;
+ return;
}
} while (V);
}
js_pushboolean(J, 0);
- return 1;
}
-static int Op_propertyIsEnumerable(js_State *J, unsigned int argc)
+static void Op_propertyIsEnumerable(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
const char *name = js_tostring(J, 1);
js_Property *ref = jsV_getownproperty(J, self, name);
js_pushboolean(J, ref && !(ref->atts & JS_DONTENUM));
- return 1;
}
-static int O_getPrototypeOf(js_State *J, unsigned int argc)
+static void O_getPrototypeOf(js_State *J, unsigned int argc)
{
js_Object *obj;
if (!js_isobject(J, 1))
@@ -100,10 +93,9 @@
js_pushobject(J, obj->prototype);
else
js_pushnull(J);
- return 1;
}
-static int O_getOwnPropertyDescriptor(js_State *J, unsigned int argc)
+static void O_getOwnPropertyDescriptor(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -137,10 +129,9 @@
js_pushboolean(J, !(ref->atts & JS_DONTCONF));
js_setproperty(J, -2, "configurable");
}
- return 1;
}
-static int O_getOwnPropertyNames(js_State *J, unsigned int argc)
+static void O_getOwnPropertyNames(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -185,8 +176,6 @@
js_pushliteral(J, "lastIndex");
js_setindex(J, -2, i++);
}
-
- return 1;
}
static void ToPropertyDescriptor(js_State *J, js_Object *obj, const char *name, js_Object *desc)
@@ -242,16 +231,15 @@
js_pop(J, 2);
}
-static int O_defineProperty(js_State *J, unsigned int argc)
+static void O_defineProperty(js_State *J, unsigned int argc)
{
if (!js_isobject(J, 1)) js_typeerror(J, "not an object");
if (!js_isobject(J, 3)) js_typeerror(J, "not an object");
ToPropertyDescriptor(J, js_toobject(J, 1), js_tostring(J, 2), js_toobject(J, 3));
js_copy(J, 1);
- return 1;
}
-static int O_defineProperties(js_State *J, unsigned int argc)
+static void O_defineProperties(js_State *J, unsigned int argc)
{
js_Object *props;
js_Property *ref;
@@ -269,10 +257,9 @@
}
js_copy(J, 1);
- return 1;
}
-static int O_create(js_State *J, unsigned int argc)
+static void O_create(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Object *proto;
@@ -299,11 +286,9 @@
}
}
}
-
- return 1;
}
-static int O_keys(js_State *J, unsigned int argc)
+static void O_keys(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -330,28 +315,24 @@
js_setindex(J, -2, i++);
}
}
-
- return 1;
}
-static int O_preventExtensions(js_State *J, unsigned int argc)
+static void O_preventExtensions(js_State *J, unsigned int argc)
{
if (!js_isobject(J, 1))
js_typeerror(J, "not an object");
js_toobject(J, 1)->extensible = 0;
js_copy(J, 1);
- return 1;
}
-static int O_isExtensible(js_State *J, unsigned int argc)
+static void O_isExtensible(js_State *J, unsigned int argc)
{
if (!js_isobject(J, 1))
js_typeerror(J, "not an object");
js_pushboolean(J, js_toobject(J, 1)->extensible);
- return 1;
}
-static int O_seal(js_State *J, unsigned int argc)
+static void O_seal(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -366,10 +347,9 @@
ref->atts |= JS_DONTCONF;
js_copy(J, 1);
- return 1;
}
-static int O_isSealed(js_State *J, unsigned int argc)
+static void O_isSealed(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -380,21 +360,20 @@
obj = js_toobject(J, 1);
if (obj->extensible) {
js_pushboolean(J, 0);
- return 1;
+ return;
}
for (ref = obj->head; ref; ref = ref->next) {
if (!(ref->atts & JS_DONTCONF)) {
js_pushboolean(J, 0);
- return 1;
+ return;
}
}
js_pushboolean(J, 1);
- return 1;
}
-static int O_freeze(js_State *J, unsigned int argc)
+static void O_freeze(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -409,10 +388,9 @@
ref->atts |= JS_READONLY | JS_DONTCONF;
js_copy(J, 1);
- return 1;
}
-static int O_isFrozen(js_State *J, unsigned int argc)
+static void O_isFrozen(js_State *J, unsigned int argc)
{
js_Object *obj;
js_Property *ref;
@@ -423,18 +401,17 @@
obj = js_toobject(J, 1);
if (obj->extensible) {
js_pushboolean(J, 0);
- return 1;
+ return;
}
for (ref = obj->head; ref; ref = ref->next) {
if (!(ref->atts & (JS_READONLY | JS_DONTCONF))) {
js_pushboolean(J, 0);
- return 1;
+ return;
}
}
js_pushboolean(J, 1);
- return 1;
}
void jsB_initobject(js_State *J)
--- a/json.c
+++ b/json.c
@@ -92,7 +92,7 @@
}
}
-static int JSON_parse(js_State *J, unsigned int argc)
+static void JSON_parse(js_State *J, unsigned int argc)
{
const char *source = js_tostring(J, 1);
jsY_initlex(J, "JSON", source);
@@ -99,7 +99,6 @@
jsonnext(J);
jsonvalue(J);
// TODO: reviver Walk()
- return 1;
}
static void fmtnum(js_Buffer **sb, double n)
@@ -237,7 +236,7 @@
return 1;
}
-static int JSON_stringify(js_State *J, unsigned int argc)
+static void JSON_stringify(js_State *J, unsigned int argc)
{
js_Buffer *sb = NULL;
if (argc > 0) {
@@ -251,10 +250,10 @@
js_pushstring(J, sb ? sb->s : "");
js_endtry(J);
free(sb);
- return 1;
}
+ } else {
+ js_pushundefined(J);
}
- return 0;
}
void jsB_initjson(js_State *J)
--- a/jsregexp.c
+++ b/jsregexp.c
@@ -29,7 +29,7 @@
js_pushobject(J, obj);
}
-int js_RegExp_prototype_exec(js_State *J, js_Regexp *re, const char *text)
+void js_RegExp_prototype_exec(js_State *J, js_Regexp *re, const char *text)
{
Resub m[10];
unsigned int i;
@@ -40,7 +40,7 @@
if (re->last > strlen(text)) {
re->last = 0;
js_pushnull(J);
- return 1;
+ return;
}
if (re->last > 0) {
text += re->last;
@@ -56,7 +56,7 @@
}
if (re->flags & JS_REGEXP_G)
re->last = re->last + (m[0].ep - text);
- return 1;
+ return;
}
if (re->flags & JS_REGEXP_G)
@@ -63,10 +63,9 @@
re->last = 0;
js_pushnull(J);
- return 1;
}
-static int Rp_test(js_State *J, unsigned int argc)
+static void Rp_test(js_State *J, unsigned int argc)
{
js_Regexp *re;
const char *text;
@@ -81,7 +80,7 @@
if (re->last > strlen(text)) {
re->last = 0;
js_pushboolean(J, 0);
- return 1;
+ return;
}
if (re->last > 0) {
text += re->last;
@@ -93,7 +92,7 @@
if (re->flags & JS_REGEXP_G)
re->last = re->last + (m[0].ep - text);
js_pushboolean(J, 1);
- return 1;
+ return;
}
if (re->flags & JS_REGEXP_G)
@@ -100,10 +99,9 @@
re->last = 0;
js_pushboolean(J, 0);
- return 1;
}
-static int jsB_new_RegExp(js_State *J, unsigned int argc)
+static void jsB_new_RegExp(js_State *J, unsigned int argc)
{
js_Regexp *old;
const char *pattern;
@@ -142,17 +140,16 @@
}
js_newregexp(J, pattern, flags);
- return 1;
}
-static int jsB_RegExp(js_State *J, unsigned int argc)
+static void jsB_RegExp(js_State *J, unsigned int argc)
{
if (js_isregexp(J, 1) && argc == 1)
- return 1;
+ return;
return jsB_new_RegExp(J, argc);
}
-static int Rp_toString(js_State *J, unsigned int argc)
+static void Rp_toString(js_State *J, unsigned int argc)
{
js_Regexp *re;
char *out;
@@ -175,10 +172,9 @@
js_pushstring(J, out);
js_endtry(J);
free(out);
- return 1;
}
-static int Rp_exec(js_State *J, unsigned int argc)
+static void Rp_exec(js_State *J, unsigned int argc)
{
return js_RegExp_prototype_exec(J, js_toregexp(J, 0), js_tostring(J, 1));
}
--- a/jsrun.c
+++ b/jsrun.c
@@ -840,22 +840,18 @@
js_pushvalue(J, v);
}
-static void jsR_callcfunction(js_State *J, unsigned int n, int min, js_CFunction F)
+static void jsR_callcfunction(js_State *J, unsigned int n, unsigned int min, js_CFunction F)
{
- int rv, i;
+ unsigned int i;
+ js_Value v;
for (i = n; i < min; ++i)
js_pushundefined(J);
- rv = F(J, n);
- if (rv) {
- js_Value v = js_tovalue(J, -1);
- TOP = --BOT; /* clear stack */
- js_pushvalue(J, v);
- } else {
- TOP = --BOT; /* clear stack */
- js_pushundefined(J);
- }
+ F(J, n);
+ v = js_tovalue(J, -1);
+ TOP = --BOT; /* clear stack */
+ js_pushvalue(J, v);
}
void js_call(js_State *J, unsigned int n)
--- a/jsstring.c
+++ b/jsstring.c
@@ -50,35 +50,31 @@
return i;
}
-static int jsB_new_String(js_State *J, unsigned int argc)
+static void jsB_new_String(js_State *J, unsigned int argc)
{
js_newstring(J, js_isdefined(J, 1) ? js_tostring(J, 1) : "");
- return 1;
}
-static int jsB_String(js_State *J, unsigned int argc)
+static void jsB_String(js_State *J, unsigned int argc)
{
js_pushliteral(J, js_isdefined(J, 1) ? js_tostring(J, 1) : "");
- return 1;
}
-static int Sp_toString(js_State *J, unsigned int argc)
+static void Sp_toString(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (self->type != JS_CSTRING) js_typeerror(J, "not a string");
js_pushliteral(J, self->u.s.string);
- return 1;
}
-static int Sp_valueOf(js_State *J, unsigned int argc)
+static void Sp_valueOf(js_State *J, unsigned int argc)
{
js_Object *self = js_toobject(J, 0);
if (self->type != JS_CSTRING) js_typeerror(J, "not a string");
js_pushliteral(J, self->u.s.string);
- return 1;
}
-static int Sp_charAt(js_State *J, unsigned int argc)
+static void Sp_charAt(js_State *J, unsigned int argc)
{
char buf[UTFmax + 1];
const char *s = js_tostring(J, 0);
@@ -90,10 +86,9 @@
} else {
js_pushliteral(J, "");
}
- return 1;
}
-static int Sp_charCodeAt(js_State *J, unsigned int argc)
+static void Sp_charCodeAt(js_State *J, unsigned int argc)
{
const char *s = js_tostring(J, 0);
int pos = js_tointeger(J, 1);
@@ -102,10 +97,9 @@
js_pushnumber(J, rune);
else
js_pushnumber(J, NAN);
- return 1;
}
-static int Sp_concat(js_State *J, unsigned int argc)
+static void Sp_concat(js_State *J, unsigned int argc)
{
char * volatile out;
const char *s;
@@ -112,7 +106,7 @@
unsigned int i, n;
if (argc == 0)
- return 1;
+ return;
s = js_tostring(J, 0);
n = strlen(s);
@@ -134,10 +128,9 @@
js_pushstring(J, out);
js_endtry(J);
free(out);
- return 1;
}
-static int Sp_indexOf(js_State *J, unsigned int argc)
+static void Sp_indexOf(js_State *J, unsigned int argc)
{
const char *haystack = js_tostring(J, 0);
const char *needle = js_tostring(J, 1);
@@ -148,16 +141,15 @@
while (*haystack) {
if (k >= pos && !strncmp(haystack, needle, len)) {
js_pushnumber(J, k);
- return 1;
+ return;
}
haystack += chartorune(&rune, haystack);
++k;
}
js_pushnumber(J, -1);
- return 1;
}
-static int Sp_lastIndexOf(js_State *J, unsigned int argc)
+static void Sp_lastIndexOf(js_State *J, unsigned int argc)
{
const char *haystack = js_tostring(J, 0);
const char *needle = js_tostring(J, 1);
@@ -172,17 +164,16 @@
++k;
}
js_pushnumber(J, last);
- return 1;
}
-static int Sp_localeCompare(js_State *J, unsigned int argc)
+static void Sp_localeCompare(js_State *J, unsigned int argc)
{
const char *a = js_tostring(J, 0);
const char *b = js_tostring(J, 1);
- return strcmp(a, b);
+ js_pushnumber(J, strcmp(a, b));
}
-static int Sp_slice(js_State *J, unsigned int argc)
+static void Sp_slice(js_State *J, unsigned int argc)
{
const char *str = js_tostring(J, 0);
const char *ss, *ee;
@@ -205,10 +196,9 @@
}
js_pushlstring(J, ss, ee - ss);
- return 1;
}
-static int Sp_substring(js_State *J, unsigned int argc)
+static void Sp_substring(js_State *J, unsigned int argc)
{
const char *str = js_tostring(J, 0);
const char *ss, *ee;
@@ -228,10 +218,9 @@
}
js_pushlstring(J, ss, ee - ss);
- return 1;
}
-static int Sp_toLowerCase(js_State *J, unsigned int argc)
+static void Sp_toLowerCase(js_State *J, unsigned int argc)
{
const char *src = js_tostring(J, 0);
char *dst = malloc(UTFmax * strlen(src) + 1);
@@ -250,10 +239,9 @@
js_pushstring(J, dst);
js_endtry(J);
free(dst);
- return 1;
}
-static int Sp_toUpperCase(js_State *J, unsigned int argc)
+static void Sp_toUpperCase(js_State *J, unsigned int argc)
{
const char *src = js_tostring(J, 0);
char *dst = malloc(UTFmax * strlen(src) + 1);
@@ -272,7 +260,6 @@
js_pushstring(J, dst);
js_endtry(J);
free(dst);
- return 1;
}
static int istrim(int c)
@@ -281,7 +268,7 @@
c == 0xA || c == 0xD || c == 0x2028 || c == 0x2029;
}
-static int Sp_trim(js_State *J, unsigned int argc)
+static void Sp_trim(js_State *J, unsigned int argc)
{
const char *s, *e;
s = js_tostring(J, 0);
@@ -291,10 +278,9 @@
while (e > s && istrim(e[-1]))
--e;
js_pushlstring(J, s, e - s);
- return 1;
}
-static int S_fromCharCode(js_State *J, unsigned int argc)
+static void S_fromCharCode(js_State *J, unsigned int argc)
{
unsigned int i;
Rune c;
@@ -316,10 +302,9 @@
js_endtry(J);
free(s);
- return 1;
}
-static int Sp_match(js_State *J, unsigned int argc)
+static void Sp_match(js_State *J, unsigned int argc)
{
js_Regexp *re;
Resub m[10];
@@ -361,11 +346,9 @@
if (c - b == 0)
++a;
}
-
- return 1;
}
-static int Sp_search(js_State *J, unsigned int argc)
+static void Sp_search(js_State *J, unsigned int argc)
{
js_Regexp *re;
Resub m[10];
@@ -386,11 +369,9 @@
js_pushnumber(J, js_utfptrtoidx(text, m[0].sp));
else
js_pushnumber(J, -1);
-
- return 1;
}
-static int Sp_replace_regexp(js_State *J, unsigned int argc)
+static void Sp_replace_regexp(js_State *J, unsigned int argc)
{
js_Regexp *re;
Resub m[10];
@@ -403,7 +384,7 @@
if (js_regexec(re->prog, source, nelem(m), m, 0)) {
js_copy(J, 0);
- return 1;
+ return;
}
re->last = 0;
@@ -481,10 +462,9 @@
js_pushstring(J, sb ? sb->s : "");
js_endtry(J);
free(sb);
- return 1;
}
-static int Sp_replace_string(js_State *J, unsigned int argc)
+static void Sp_replace_string(js_State *J, unsigned int argc)
{
const char *source, *needle, *s, *r;
js_Buffer *sb = NULL;
@@ -496,7 +476,7 @@
s = strstr(source, needle);
if (!s) {
js_copy(J, 0);
- return 1;
+ return;
}
n = strlen(needle);
@@ -541,10 +521,9 @@
js_pushstring(J, sb ? sb->s : "");
js_endtry(J);
free(sb);
- return 1;
}
-static int Sp_replace(js_State *J, unsigned int argc)
+static void Sp_replace(js_State *J, unsigned int argc)
{
if (js_isregexp(J, 1))
return Sp_replace_regexp(J, argc);
@@ -551,7 +530,7 @@
return Sp_replace_string(J, argc);
}
-static int Sp_split_regexp(js_State *J, unsigned int argc)
+static void Sp_split_regexp(js_State *J, unsigned int argc)
{
js_Regexp *re;
Resub m[10];
@@ -571,11 +550,11 @@
/* splitting the empty string */
if (e == 0) {
if (js_regexec(re->prog, text, nelem(m), m, 0)) {
- if (len == limit) return 1;
+ if (len == limit) return;
js_pushliteral(J, "");
js_setindex(J, -2, 0);
}
- return 1;
+ return;
}
p = a = text;
@@ -592,12 +571,12 @@
continue;
}
- if (len == limit) return 1;
+ if (len == limit) return;
js_pushlstring(J, p, b - p);
js_setindex(J, -2, len++);
for (k = 1; k < nelem(m) && m[k].sp; ++k) {
- if (len == limit) return 1;
+ if (len == limit) return;
js_pushlstring(J, m[k].sp, m[k].ep - m[k].sp);
js_setindex(J, -2, len++);
}
@@ -605,14 +584,12 @@
a = p = c;
}
- if (len == limit) return 1;
+ if (len == limit) return;
js_pushstring(J, p);
js_setindex(J, -2, len);
-
- return 1;
}
-static int Sp_split_string(js_State *J, unsigned int argc)
+static void Sp_split_string(js_State *J, unsigned int argc)
{
const char *str = js_tostring(J, 0);
const char *sep = js_tostring(J, 1);
@@ -632,7 +609,7 @@
js_setindex(J, -2, i);
str += n;
}
- return 1;
+ return;
}
for (i = 0; str && i < limit; ++i) {
@@ -647,21 +624,19 @@
str = NULL;
}
}
-
- return 1;
}
-static int Sp_split(js_State *J, unsigned int argc)
+static void Sp_split(js_State *J, unsigned int argc)
{
if (js_isundefined(J, 1)) {
js_newarray(J);
js_copy(J, 0);
js_setindex(J, -2, 0);
- return 1;
+ } else if (js_isregexp(J, 1)) {
+ Sp_split_regexp(J, argc);
+ } else {
+ Sp_split_string(J, argc);
}
- if (js_isregexp(J, 1))
- return Sp_split_regexp(J, argc);
- return Sp_split_string(J, argc);
}
void jsB_initstring(js_State *J)
--- a/main.c
+++ b/main.c
@@ -4,22 +4,21 @@
#define PS1 "> "
-static int jsB_gc(js_State *J, unsigned int argc)
+static void jsB_gc(js_State *J, unsigned int argc)
{
int report = js_toboolean(J, 1);
js_gc(J, report);
- return 0;
+ js_pushundefined(J);
}
-static int jsB_load(js_State *J, unsigned int argc)
+static void jsB_load(js_State *J, unsigned int argc)
{
const char *filename = js_tostring(J, 1);
int rv = js_dofile(J, filename);
js_pushboolean(J, !rv);
- return 1;
}
-static int jsB_print(js_State *J, unsigned int argc)
+static void jsB_print(js_State *J, unsigned int argc)
{
unsigned int i;
for (i = 1; i <= argc; ++i) {
@@ -28,7 +27,7 @@
fputs(s, stdout);
}
putchar('\n');
- return 0;
+ js_pushundefined(J);
}
int