Use tables instead of lame functions #2

New Issue

rudzik8 · 2022-04-24T08:15:25+02:00

rudzik8 commented

2022-04-24 08:15:25 +02:00

like, they basically repeat themselfs (wood types, colors, dye subnames, etc)
combining lame functions just will make them infinitely long and hard to work with

like, they basically repeat themselfs (wood types, colors, dye subnames, etc) combining lame functions just will make them infinitely long and hard to work with

rudzik8 added this to the Version 1.2 milestone 2022-04-24 08:17:11 +02:00

rudzik8 modified the milestone from Version 1.2 to Version 1.3

2022-04-24 08:50:03 +02:00

rudzik8 referenced this issue from a commit

2022-04-30 16:18:14 +02:00

Use tables for registering colored nodes (fix of #2, p1)

rudzik8 commented

2022-04-30 16:22:34 +02:00

fixed for colored nodes in commit 4f43fcf650

dunno if will be fixed for wooden ones too (because wooden stuff registration is already not such a mess and it works mostly the same in mods like mcl_doors (which is part of MineClone))

fixed for colored nodes in commit [4f43fcf650](https://git.minetest.land/rudzik8/mcl_decor/commit/79b53ce5a63431d8e1911edaf3f49cf5520c1d8c) dunno if will be fixed for wooden ones too (because wooden stuff registration is already not such a mess and it works mostly the same in mods like `mcl_doors` (which is part of MineClone))

rudzik8 referenced this issue from a commit

2022-05-01 09:06:28 +02:00

Use tables for registering wooden nodes (fix of #2, p2)

rudzik8 commented

2022-05-01 09:11:04 +02:00

fixed for wooden ones too... see 2989f4c616

BUT code became just bigger and non-usable by other mods (like, litteraly no more API)
so, no merge. tables for colored nodes only. branch will be still here though

fixed for wooden ones too... see [2989f4c616](https://git.minetest.land/rudzik8/mcl_decor/commit/2989f4c616cb839c09e8835ea4500cb52729626d) BUT code became just bigger and non-usable by other mods (like, litteraly no more API) so, no merge. tables for colored nodes only. branch will be still here though

rudzik8 closed this issue

2022-05-01 09:11:07 +02:00

This repo is archived. You cannot comment on issues.