diff options
author | Roberto Ierusalimschy <roberto@inf.puc-rio.br> | 2017-12-15 11:07:10 -0200 |
---|---|---|
committer | Roberto Ierusalimschy <roberto@inf.puc-rio.br> | 2017-12-15 11:07:10 -0200 |
commit | b3f924bc695ed6ebe2539d365197696e41863a91 (patch) | |
tree | bf756351d06bdfc469846e533699c9ce58bc3566 /lparser.c | |
parent | e001d5aea693ca1bcf1a4dfd2f900737fc0a8604 (diff) | |
download | lua-b3f924bc695ed6ebe2539d365197696e41863a91.tar.gz lua-b3f924bc695ed6ebe2539d365197696e41863a91.tar.bz2 lua-b3f924bc695ed6ebe2539d365197696e41863a91.zip |
'Proto->numparams' does not include vararg parameter
(one less subtraction when calling functions...)
Diffstat (limited to 'lparser.c')
-rw-r--r-- | lparser.c | 9 |
1 files changed, 4 insertions, 5 deletions
@@ -1,5 +1,5 @@ | |||
1 | /* | 1 | /* |
2 | ** $Id: lparser.c,v 2.170 2017/12/06 18:36:31 roberto Exp roberto $ | 2 | ** $Id: lparser.c,v 2.171 2017/12/14 14:24:02 roberto Exp roberto $ |
3 | ** Lua Parser | 3 | ** Lua Parser |
4 | ** See Copyright Notice in lua.h | 4 | ** See Copyright Notice in lua.h |
5 | */ | 5 | */ |
@@ -765,7 +765,6 @@ static void parlist (LexState *ls) { | |||
765 | FuncState *fs = ls->fs; | 765 | FuncState *fs = ls->fs; |
766 | Proto *f = fs->f; | 766 | Proto *f = fs->f; |
767 | int nparams = 0; | 767 | int nparams = 0; |
768 | f->is_vararg = 0; | ||
769 | if (ls->t.token != ')') { /* is 'parlist' not empty? */ | 768 | if (ls->t.token != ')') { /* is 'parlist' not empty? */ |
770 | do { | 769 | do { |
771 | switch (ls->t.token) { | 770 | switch (ls->t.token) { |
@@ -789,7 +788,7 @@ static void parlist (LexState *ls) { | |||
789 | } while (!f->is_vararg && testnext(ls, ',')); | 788 | } while (!f->is_vararg && testnext(ls, ',')); |
790 | } | 789 | } |
791 | adjustlocalvars(ls, nparams); | 790 | adjustlocalvars(ls, nparams); |
792 | f->numparams = cast_byte(fs->nactvar); | 791 | f->numparams = cast_byte(fs->nactvar) - f->is_vararg; |
793 | luaK_reserveregs(fs, fs->nactvar); /* reserve register for parameters */ | 792 | luaK_reserveregs(fs, fs->nactvar); /* reserve register for parameters */ |
794 | } | 793 | } |
795 | 794 | ||
@@ -975,7 +974,7 @@ static void simpleexp (LexState *ls, expdesc *v) { | |||
975 | } | 974 | } |
976 | case TK_DOTS: { /* vararg */ | 975 | case TK_DOTS: { /* vararg */ |
977 | FuncState *fs = ls->fs; | 976 | FuncState *fs = ls->fs; |
978 | int lastparam = fs->f->numparams - 1; | 977 | int lastparam = fs->f->numparams; |
979 | check_condition(ls, fs->f->is_vararg, | 978 | check_condition(ls, fs->f->is_vararg, |
980 | "cannot use '...' outside a vararg function"); | 979 | "cannot use '...' outside a vararg function"); |
981 | init_exp(v, VVARARG, luaK_codeABC(fs, OP_VARARG, 0, 1, lastparam)); | 980 | init_exp(v, VVARARG, luaK_codeABC(fs, OP_VARARG, 0, 1, lastparam)); |
@@ -1676,7 +1675,7 @@ static void mainfunc (LexState *ls, FuncState *fs) { | |||
1676 | expdesc v; | 1675 | expdesc v; |
1677 | open_func(ls, fs, &bl); | 1676 | open_func(ls, fs, &bl); |
1678 | fs->f->is_vararg = 1; /* main function is always declared vararg */ | 1677 | fs->f->is_vararg = 1; /* main function is always declared vararg */ |
1679 | fs->f->numparams = 1; | 1678 | fs->f->numparams = 0; |
1680 | new_localvarliteral(ls, "_ARG"); | 1679 | new_localvarliteral(ls, "_ARG"); |
1681 | adjustlocalvars(ls, 1); | 1680 | adjustlocalvars(ls, 1); |
1682 | luaK_reserveregs(fs, 1); /* reserve register for vararg */ | 1681 | luaK_reserveregs(fs, 1); /* reserve register for vararg */ |