'collectgarbage' returns old mode when changing mode

This commit is contained in:
Roberto Ierusalimschy 2018-02-05 15:10:52 -02:00
parent 90569630d6
commit 56e50e8bc5
2 changed files with 14 additions and 6 deletions

6
lapi.c
View File

@ -1,5 +1,5 @@
/* /*
** $Id: lapi.c,v 2.281 2018/01/28 15:13:26 roberto Exp roberto $ ** $Id: lapi.c,v 2.282 2018/01/29 16:21:35 roberto Exp roberto $
** Lua API ** Lua API
** See Copyright Notice in lua.h ** See Copyright Notice in lua.h
*/ */
@ -1121,6 +1121,7 @@ LUA_API int lua_gc (lua_State *L, int what, ...) {
break; break;
} }
case LUA_GCGEN: { case LUA_GCGEN: {
int oldmode = g->gckind;
int minormul = va_arg(argp, int); int minormul = va_arg(argp, int);
int majormul = va_arg(argp, int); int majormul = va_arg(argp, int);
if (minormul != 0) if (minormul != 0)
@ -1128,9 +1129,11 @@ LUA_API int lua_gc (lua_State *L, int what, ...) {
if (majormul != 0) if (majormul != 0)
setgcparam(g->genmajormul, majormul); setgcparam(g->genmajormul, majormul);
luaC_changemode(L, KGC_GEN); luaC_changemode(L, KGC_GEN);
res = (oldmode == KGC_GEN) ? LUA_GCGEN : LUA_GCINC;
break; break;
} }
case LUA_GCINC: { case LUA_GCINC: {
int oldmode = g->gckind;
int pause = va_arg(argp, int); int pause = va_arg(argp, int);
int stepmul = va_arg(argp, int); int stepmul = va_arg(argp, int);
int stepsize = va_arg(argp, int); int stepsize = va_arg(argp, int);
@ -1141,6 +1144,7 @@ LUA_API int lua_gc (lua_State *L, int what, ...) {
if (stepsize != 0) if (stepsize != 0)
g->gcstepsize = stepsize; g->gcstepsize = stepsize;
luaC_changemode(L, KGC_INC); luaC_changemode(L, KGC_INC);
res = (oldmode == KGC_GEN) ? LUA_GCGEN : LUA_GCINC;
break; break;
} }
default: res = -1; /* invalid option */ default: res = -1; /* invalid option */

View File

@ -1,5 +1,5 @@
/* /*
** $Id: lbaselib.c,v 1.317 2017/06/27 18:32:49 roberto Exp roberto $ ** $Id: lbaselib.c,v 1.318 2017/11/16 13:19:06 roberto Exp roberto $
** Basic library ** Basic library
** See Copyright Notice in lua.h ** See Copyright Notice in lua.h
*/ */
@ -170,6 +170,12 @@ static int luaB_rawset (lua_State *L) {
} }
static int pushmode (lua_State *L, int oldmode) {
lua_pushstring(L, (oldmode == LUA_GCINC) ? "incremental" : "generational");
return 1;
}
static int luaB_collectgarbage (lua_State *L) { static int luaB_collectgarbage (lua_State *L) {
static const char *const opts[] = {"stop", "restart", "collect", static const char *const opts[] = {"stop", "restart", "collect",
"count", "step", "setpause", "setstepmul", "count", "step", "setpause", "setstepmul",
@ -206,15 +212,13 @@ static int luaB_collectgarbage (lua_State *L) {
case LUA_GCGEN: { case LUA_GCGEN: {
int minormul = (int)luaL_optinteger(L, 2, 0); int minormul = (int)luaL_optinteger(L, 2, 0);
int majormul = (int)luaL_optinteger(L, 3, 0); int majormul = (int)luaL_optinteger(L, 3, 0);
lua_gc(L, o, minormul, majormul); return pushmode(L, lua_gc(L, o, minormul, majormul));
return 0;
} }
case LUA_GCINC: { case LUA_GCINC: {
int pause = (int)luaL_optinteger(L, 2, 0); int pause = (int)luaL_optinteger(L, 2, 0);
int stepmul = (int)luaL_optinteger(L, 3, 0); int stepmul = (int)luaL_optinteger(L, 3, 0);
int stepsize = (int)luaL_optinteger(L, 4, 0); int stepsize = (int)luaL_optinteger(L, 4, 0);
lua_gc(L, o, pause, stepmul, stepsize); return pushmode(L, lua_gc(L, o, pause, stepmul, stepsize));
return 0;
} }
default: { default: {
int res = lua_gc(L, o); int res = lua_gc(L, o);