Find nodes in area (under air): Raise volume limit and document it
authorparamat <paramat@users.noreply.github.com>
Wed, 21 Feb 2018 14:34:06 +0000 (14:34 +0000)
committerSmallJoker <mk939@ymail.com>
Sun, 3 Jun 2018 15:32:00 +0000 (17:32 +0200)
doc/lua_api.txt
src/script/lua_api/l_env.cpp

index 73572ac7836449d22517efb70e6f308886d5ee9e..04ae336545c674f1ea01622bc6b02e256aee7cb8 100644 (file)
@@ -2578,9 +2578,11 @@ and `minetest.auth_reload` call the authetification handler.
     * `nodenames`: e.g. `{"ignore", "group:tree"}` or `"default:dirt"`
     * First return value: Table with all node positions
     * Second return value: Table with the count of each node with the node name as index
+    * Area volume is limited to 4,096,000 nodes
 * `minetest.find_nodes_in_area_under_air(pos1, pos2, nodenames)`: returns a list of positions
     * `nodenames`: e.g. `{"ignore", "group:tree"}` or `"default:dirt"`
     * Return value: Table with all node positions with a node air above
+    * Area volume is limited to 4,096,000 nodes
 * `minetest.get_perlin(noiseparams)`
 * `minetest.get_perlin(seeddiff, octaves, persistence, scale)`
     * Return world-specific perlin noise (`int(worldseed)+seeddiff`)
index 46745facd26ac38540b11ca8163d763a796ee7f9..4a7885da7639acdc347e6abd9641a86cfee7ad38 100644 (file)
@@ -655,14 +655,10 @@ int ModApiEnvMod::l_find_nodes_in_area(lua_State *L)
        sortBoxVerticies(minp, maxp);
 
        v3s16 cube = maxp - minp + 1;
-
-       /* Limit for too large areas, assume default values
-        * and give tolerances of 1 node on each side
-        * (chunksize * MAP_BLOCKSIZE + 2)^3 = 551368
-       */
-       if ((u64)cube.X * (u64)cube.Y * (u64)cube.Z > 551368) {
+       // Volume limit equal to 8 default mapchunks, (80 * 2) ^ 3 = 4,096,000
+       if ((u64)cube.X * (u64)cube.Y * (u64)cube.Z > 4096000) {
                luaL_error(L, "find_nodes_in_area(): area volume"
-                               " exceeds allowed value of 551368");
+                               " exceeds allowed value of 4096000");
                return 0;
        }
 
@@ -723,14 +719,10 @@ int ModApiEnvMod::l_find_nodes_in_area_under_air(lua_State *L)
        sortBoxVerticies(minp, maxp);
 
        v3s16 cube = maxp - minp + 1;
-
-       /* Limit for too large areas, assume default values
-        * and give tolerances of 1 node on each side
-        * (chunksize * MAP_BLOCKSIZE + 2)^3 = 551368
-       */
-       if ((u64)cube.X * (u64)cube.Y * (u64)cube.Z > 551368) {
+       // Volume limit equal to 8 default mapchunks, (80 * 2) ^ 3 = 4,096,000
+       if ((u64)cube.X * (u64)cube.Y * (u64)cube.Z > 4096000) {
                luaL_error(L, "find_nodes_in_area_under_air(): area volume"
-                               " exceeds allowed value of 551368");
+                               " exceeds allowed value of 4096000");
                return 0;
        }