Time Nick Message 11:04 kilbith ERROR[Main]: Some exception: "std::bad_alloc" 11:04 kilbith very helpful 11:08 red-001 gdb 11:10 red-001 well that wouldn't help with a memory leak 11:17 kilbith ok I know why 11:17 kilbith it occurs when you try to divide 0 inside a formspec element 11:18 kilbith like that: "image[1," .. (0/2) .. ";1,1;foo.bar]" 11:23 red-001 eh wtf minetest started to segfault for me on world start 11:25 red-001 oh maybe it's because I updated my system 11:25 red-001 I also got a new warning https://paste.ee/p/Yt3LE 11:35 red-001 yeah nvm that 11:35 red-001 I just had to recompile 11:46 nerzhul red-001, this i irrlicht warningin you can ignore safely 11:47 nerzhul kilbith, interesting how did you did that exactly ? 11:50 kilbith already said 11:50 kilbith make a mod 11:51 nerzhul who did the division ? C++ or Lua ? 11:51 nerzhul seems here you setted a 0 for one of the dimensions, right ? 11:58 red-001 nerzhul, yeah I managed to fix my issue 12:22 Wuzzy What is the scope of minetest.register_alias? Does it only alias items or are entities and other things supported, too? 12:32 kilbith why not testing yourself 13:05 Wuzzy why not adding it to lua_api.txt? :P 13:06 nerzhul Just propose a PR if you want it 13:27 Wuzzy sigh 13:27 Wuzzy I do not document things I do not understand 13:35 Wuzzy What value types are OFFICIALLY allowed for player:get_attribute and player:set_attribute? 13:39 nerzhul guys, there are 4 PR which needs to be merged before release: https://github.com/minetest/minetest/projects/2 3 are mergeable, one hsould be improved 13:51 Wuzzy Add https://github.com/minetest/minetest/pull/5879 to that list. :-) 13:52 Wuzzy okay maybe not really. but anyway 13:52 Wuzzy YESS!!! I just realized singlenode got back in. :-) 13:57 nerzhul thanks for your PR, it's important on trivial things everybody contributes :) 13:57 nerzhul merging #5876 in ~10 mins 13:57 ShadowBot https://github.com/minetest/minetest/issues/5876 -- Document hardware coloring and soft node overlays by juhdanad 13:58 nerzhul oops 13:58 nerzhul i'm stupid 13:58 nerzhul i merged it now instead of closing tab 13:58 nerzhul sorry :) 14:00 nerzhul red-001, Wuzzy do you know it's possible to handle "inf" in /shutdown ? i dont' know what representation it used in Lua 14:18 Wuzzy nerzhul: I still think /shutdown inf is completely senseless 14:18 nerzhul i agree 14:18 Wuzzy also, the chat message you receive as reply looks broken 14:19 red-001 I mean why would you run that through? 14:19 red-001 it behaves as expected 14:19 Wuzzy what is expected? 14:19 red-001 I mean this could be fixed but that would require checking if the string is equal to "inf" 14:20 Wuzzy inf is not a special symbol in lua 14:20 red-001 it is 14:20 Wuzzy $ lua 14:20 Wuzzy a = inf 14:20 Wuzzy print(a) 14:20 Wuzzy > nil 14:20 red-001 try running that in minetest 14:21 Wuzzy ?! wtf 14:21 Wuzzy okay ... 14:21 red-001 local a = "inf" 14:21 Wuzzy /lua print(inf) 14:21 red-001 print(tonumber(a)) 14:21 Wuzzy nothing replied 14:21 Wuzzy /lua print(dump(inf)) 14:21 Wuzzy nil 14:21 Wuzzy inf is just interpreted as a variable name, i guess 14:22 DS-minetest /lua print(tonumber("inf")) ? 14:22 Wuzzy aha 14:22 Wuzzy the chat command runs the variable through tonumber 14:23 Wuzzy LOL... wtf 14:23 Wuzzy still nil?!! 14:23 red-001 huh werid it worked for me 14:23 Wuzzy nvm 14:23 Wuzzy I stupid. hurr durr 14:23 red-001 print(tonumber("inf")) 14:24 Wuzzy yes, the function does exactly that when you entered /shutdown inf 14:25 Wuzzy ah 14:25 Wuzzy Lua has even math.huge 14:26 Wuzzy defined as "The value HUGE_VAL, a value larger than or equal to any other numerical value." 14:26 red-001 I think inf might be equal to math.huge 14:26 red-001 but I'm not sure 14:26 Wuzzy for me, it is 14:26 red-001 inf is kinda random and hard to depend on 14:27 Wuzzy might be implementaiton-dependant? 14:27 Wuzzy HUGE_VAL is a reference to C btw 14:27 Wuzzy Lua has a lot to do with C behind the scenes 14:27 Wuzzy anyway 14:28 Wuzzy the fix should be obvious now. just filter out inf 14:28 Wuzzy ooor at least pretend this value does not exist :D 14:28 Wuzzy because we might miss edge cases 14:29 red-001 I don't see the point 14:29 Wuzzy well explicitly mentioning or supporting inf is pointless, too 14:29 red-001 well I think thats meant to show all valid lua numbers are supported 14:30 red-001 btw you can replcate the bug is causes just by using a really large number 14:30 Wuzzy in that case, we definitely must mention /shotdown nan 14:30 Wuzzy xD 14:30 red-001 like 1e309 14:30 red-001 no NaN isn't a valid number 14:31 red-001 it's the opposite of a valid number 14:31 Wuzzy well minetest munches this nan just fine for some reason 14:32 Wuzzy although this doesnt make sense of course 14:32 Wuzzy i was just exaggerating 14:32 red-001 it does? 14:32 Wuzzy insta-shutdown 14:32 Wuzzy PROOF THAT NAN = 0 14:32 Wuzzy xD 14:33 red-001 wow I think I proved "one" == 0 14:33 red-001 /shutdown one 14:34 Wuzzy at least Lua doesnt coearce "one" to 0... 14:34 Wuzzy the code just falls back to 0 when the param is invalid 14:35 red-001 yes 14:35 Wuzzy anyway 14:35 red-001 I mean this could be worked around but this is just a shutdown command 14:35 red-001 no reason to make it too complex 14:35 Wuzzy My suggestion: Just dont mention shutdown inf as it is just plain confusing and useless 14:36 Wuzzy yeah, i dont suggest to build a full-blown parser 14:36 red-001 well I wouldn't mind replacing the 0..inf with "Vaild number" or something like that 14:41 Wuzzy okay 14:41 Wuzzy post PR soon 14:41 Wuzzy We have discussed this WAY too long by now. :D 14:41 red-001 ^ 15:00 Wuzzy red-001, there: https://github.com/minetest/minetest/pull/5880 15:03 nerzhul Wuzzy, nice, i will merge it as it's trivial after CI suiccess 15:05 red-001 +1 15:16 nerzhul merged 15:38 Wuzzy nerzhul, red-001: should i work on fixing translatable strings or awe we too close to release now? 15:39 Wuzzy might be not the nicest move to translators 15:39 Wuzzy unless there will be no release on June 4th 15:39 nerzhul you can, but it should be finished and validated tomorrow in ~18h maximum 15:39 Wuzzy thats not the issue 15:39 Wuzzy the issue is that this will very possibly stump translators 15:40 Wuzzy hmmm i think i better dont touch this for now. and keep this for after release 15:40 Wuzzy because we already have translators working 15:40 Wuzzy if yousay 18h maxiumum thats was too close :/ 15:41 Fixer i finished my translation for now (excluding that advanced settings insanity), good enough :} just update translations before release 15:41 nerzhul i will merge translations before release 15:41 Wuzzy yes 15:41 Wuzzy Fixer: Why not the advanced settings? 15:41 Wuzzy which language? 15:41 red-001 ukrainan? 15:42 Wuzzy well 61.3% is hardly finished in my view 15:44 Wuzzy but thanks for your work anyways 16:11 Fixer Wuzzy2: a lot of technical stuff to translate, not in a mood 16:11 Wuzzy2 ok 16:45 paramat Wuzzy2 do you want to fix the 2 issues in #5879 or shall i fix those on merge? i'm thinking of merging in a few mins to avoid conflicts arising 16:45 ShadowBot https://github.com/minetest/minetest/issues/5879 -- Fix lua_api.txt syntax errors and add a few bits missing documentation by Wuzzy2 16:46 Wuzzy2 fix on merge plz as I am now busy in debugging MineClone 2 :) 16:46 paramat ok 16:47 Wuzzy2 thx 16:47 paramat will merge that over the next few mins 17:02 paramat merging 5879 17:04 Wuzzy2 :-) 17:09 paramat complete 19:08 Krock paramat, "Whenever a sneak node was detected, the player will be moved to the topmost position." raising the node height to 2m will not solve this problem. Nothing but an invisible dummy node on top of the fence can solve it 19:11 paramat i see now, thanks 19:15 Krock np, at least I learned a few things about it while digging in this code for hours :3