Use lua_createtable instead of lua_newtable

Allows preallocating the needed size for its elements since they're known in advance
edo9300 · Dec 8, 2023 · cdb14fd · cdb14fd
1 parent e794a31
commit cdb14fd
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/libcard.cpp b/libcard.cpp
@@ -1931,7 +1931,7 @@ LUA_FUNCTION(GetCounter) {
 }
 LUA_FUNCTION(GetAllCounters) {
 	check_param_count(L, 1);
-	lua_newtable(L);
+	lua_createtable(L, self->counters.size(), 0);
 	for(const auto& counter : self->counters) {
 		lua_pushinteger(L, counter.first);
 		lua_pushinteger(L, counter.second[0] + counter.second[1]);

diff --git a/libduel.cpp b/libduel.cpp
@@ -2360,7 +2360,7 @@ LUA_STATIC_FUNCTION(SelectCardsFromCodes) {
 			luaL_checkstack(L, static_cast<int>(ret_codes.list.size() + (3 * ret_index) /* account for table creation */), nullptr);
 			for(const auto& obj : ret_codes.list) {
 				if(ret_index) {
-					lua_newtable(L);
+					lua_createtable(L, 2, 0);
 					lua_pushinteger(L, 1);
 				}
 				lua_pushinteger(L, obj.first);

diff --git a/libgroup.cpp b/libgroup.cpp
@@ -580,7 +580,7 @@ LUA_FUNCTION(GetClass) {
 	for(auto& pcard : self->container) {
 		er.insert(pduel->lua->get_operation_value(pcard, findex, extraargs));
 	}
-	lua_newtable(L);
+	lua_createtable(L, er.size(), 0);
 	int i = 1;
 	for(auto& val : er) {
 		lua_pushinteger(L, i++);