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 */ |
