Time  Nick      Message
11:41 [MatrxMT] <grorp> Congratulations to appgurueu, ExeVirus, Bastrabun, sfan5 on the merged spatial index PR! Thank you for making this happen.
11:51 MTDiscord <bastrabun> Bastrabun didn't contribute, but is dead happy that thing is merged. Many thanks to all the coders and greetings from a peak of 5424 entities  🙂
14:37 MTDiscord <luatic> thanks everyone :)
14:37 MTDiscord <luatic> why is LuaHelper a separate thing from the helpers in c_converter? and why is it a class?
14:57 sfan5     you mean it could be a namespace instead?
14:58 sfan5     i think it being its own file is good
14:58 sfan5     c_converter could use a big cleanup anyway
18:43 MTDiscord <luatic> yes, it should be a namespace. so is the plan moving forward that "clean" conversion code is placed in LuaHelper?
18:44 Krock     depends on the developer's personal preference
18:44 Krock     either is IMO fine
18:47 sfan5     i think having LuaHelper split (primitive types) from c_convert (structures) is fine
18:47 sfan5     but yeah it doesn't need to be a class
20:03 sfan5     merging #15977, #15987 in 10m
20:03 ShadowBot https://github.com/luanti-org/luanti/issues/15977 -- Fix operator[] for vector2d and vector3d being potentially UB by sfan5
20:03 ShadowBot https://github.com/luanti-org/luanti/issues/15987 -- [don't squash] GL driver and related feature work by sfan5
20:03 sfan5     #15990 too actually
20:03 ShadowBot https://github.com/luanti-org/luanti/issues/15990 -- Improve some warning messages by appgurueu