planted acacia got spruce #1031

New Issue

kay27 · 2021-01-30T01:58:40+01:00

kay27 commented

2021-01-30 01:58:40 +01:00

![image](/attachments/e0a1f2d1-de8f-4c6a-9f3d-f11286cd43cf)

image.png

796 KiB

🎉 1 😆 1

kay27 added the

bug

gameplay

#P3: elevated

labels 2021-01-30 01:58:40 +01:00

kay27 commented

2021-01-30 02:13:10 +01:00

actually sapling_grow_action() function is mind-blowing for me

i've understood it, but what was the reason for doing it this way:

you define functions which return a function, which freely uses parent parameters :)

okay, i'm not so familiar with Lua, to be sure if this is an suitable way, where is the parental variables scope in fact? can't it mix up the trees?

i would better rewrite it from scratch, but again, if i only know the reason for this code style! it isn't a lambda function, just some not very common thing for me.

actually ```sapling_grow_action()``` function is mind-blowing for me i've understood it, but what was the reason for doing it this way: you define functions which return _a function_, which freely uses parent parameters :) okay, i'm not so familiar with Lua, to be sure if this is an suitable way, where is the parental variables scope in fact? can't it mix up the trees? i would better rewrite it from scratch, but again, if i only know the reason for this code style! it isn't a lambda function, just some not very common thing for me.

kay27 commented

2021-01-30 02:18:15 +01:00

ref. Wuzzy/MineClone2#1028

ref. https://git.minetest.land/Wuzzy/MineClone2/issues/1028

kay27 referenced this issue from a commit

2021-01-30 02:24:00 +01:00

Fix the trees, https://git.minetest.land/Wuzzy/MineClone2/issues/1031

kay27 commented

2021-01-30 02:25:13 +01:00

It's my bad.
Fixed now.
But have no idea what Quile had seen, then

It's my bad. Fixed now. But have no idea what Quile had seen, then

kay27 closed this issue