Time  Nick        Message
00:54 ANAND       ~tell Wuzzy on_grant and on_revoke do exist, but in a very mangled form that grep can't comprehend :P
00:54 ShadowBot   ANAND: O.K.
00:54 ANAND       no wait
00:57 ANAND       ~tell Wuzzy https://github.com/minetest/minetest/blob/master/builtin/game/chat.lua#L189 and L261 for on_grant and on_revoke respectively
00:57 ShadowBot   ANAND: O.K.
01:11 paramat     merging game#2485
01:11 ShadowBot   https://github.com/minetest/minetest_game/issues/2485 -- Update functions.lua by leucome
01:14 paramat     updated proposal for sorting out the TNT mess, see game#2472
01:14 ShadowBot   https://github.com/minetest/minetest_game/issues/2472 -- Add niter ore and charcoal to rationalise TNT (gunpowder) crafting
01:16 paramat     can anyone support game#2453 ? logical, obvious, simple and will clean up player_api mod to help people use it in new games
01:16 ShadowBot   https://github.com/minetest/minetest_game/issues/2453 -- Move hotbar textures/code, and player damage sound, from player_api to default by paramat
10:47 p_gimeno    hm, I don't think Minetest is in the business to tell people how to make gunpowder. I like the current unrealistic, simple (as in not needing components introduced on purpose for it) and cheap approach. As for the TNT name, while it's not trinitrotoluene, it's not too wrong to call it that. https://en.wikipedia.org/wiki/Dynamite#TNT
10:47 p_gimeno    (re game#2472)
10:47 ShadowBot   https://github.com/minetest/minetest_game/issues/2472 -- Add niter ore and charcoal to rationalise TNT (gunpowder) crafting
10:49 p_gimeno    IOW I don't see issue 2472 as an issue, and I don't think there's anything that needs to be changed
11:20 pmpp        but everyobody is expected to know gunpowder formula so using a wrong one is akward
11:26 H4mlet      I do not know the gunpowder's formula, and I don't even think that putting an iron lump into a furnace would give me a steel ingot.
11:26 pmpp        for steel you need iron and carbon at a higher temp than bronze
11:27 pmpp        that's also in basic chemistry or jules vernes books as you prefer
11:27 H4mlet      Correct, thus IF having a wrong tnt crafting recipe is an issue, then also having a wrong steel ingot recipe is an issue.
11:28 pmpp        indeed
11:28 H4mlet      Long story short: MTG is not realistic, you should try instead Real Test.
11:28 pmpp        i tend to favour mt as an educationnal tool
11:29 p_gimeno    I don't think educating on how to make gunpowder is a priority
11:29 H4mlet      https://github.com/VanessaE/realtest
11:30 pmpp        p_gimeno: you already said that
11:31 H4mlet      Molotov mod anyone? [joking]
11:31 pmpp        *that* is not in science class
11:32 H4mlet      :D I hope so!
11:32 pmpp        but can be learned it in the streets later :p
11:37 pmpp        H4mlet: thx for the link making a mod for https://en.wikipedia.org/wiki/The_Mysterious_Island would require at leat 4 differents explosives acurrate craft lists :)
11:41 H4mlet      You're welcome, this is the forum's thread https://forum.minetest.net/viewtopic.php?f=50&t=2671
11:42 H4mlet      Be warned that it supports just mapgen v6.
11:43 H4mlet      (It would be better moving to #Minetest I think)
11:43 H4mlet      afk back in 15
13:45 Calinou     sfan5: https://github.com/minetest/minetest.github.io/pull/190 :)
13:45 Calinou     wait
13:46 Calinou     fixed it
14:25 sfan5       merging #8640 in 10m
14:26 ShadowBot   https://github.com/minetest/minetest/issues/8640 -- Trigger on_place in many situations even if prediction failed by DS-Minetest
14:41 sfan5       pushing https://a.uguu.se/aS1ldCE6Wxpw_6P2UY1w.txt (small fix) in a few minutes
17:52 Krock       are wielded items limited to players?
17:53 Krock       seems to be worthy merging that into ActiveObject so it's entirely generic
17:55 sfan5       entities have no inventory
17:57 Krock       yeah.. although it's requested many times
18:04 sfan5       merging #7557 in 5 minutes
18:04 ShadowBot   https://github.com/minetest/minetest/issues/7557 -- Add support for per-player FOV overrides and multipliers by ClobberXD
18:05 Krock       and may the hand item be placed? maybe.. it's not decreased, though
18:05 Krock       aagh
18:08 DS-minetest the placement should be avoidable with a lua callback
18:08 DS-minetest (on_place or whatever)
18:09 Krock       I'll limit its capabilities to the groupcaps/damage stuff
18:09 Krock       if someone wants to place or activate the hand item they should implement it how they want it
18:10 sfan5       yes, that's how it should be
18:10 DS-minetest btw. another problem with the "hand" is that its tool properties don't work anymore if you for example have dirt in your hand
18:11 DS-minetest then "" is used
18:12 Krock       indeed
18:12 Krock       unless dirt is not capable digging it
18:13 sfan5       ideally, instead of finding out which item is the hand, the code would call getDigParams with each possible item (wielditem, "hand" list, hardcoded hand) and use the best result
18:14 Krock       use std::min on the returned caps
18:57 Krock       #8888 .. mmh. after this notification I noticed that new areas are inserted in ascending order, but persistent IDs might actually cause it to fail
18:57 ShadowBot   https://github.com/minetest/minetest/issues/8888 -- Fix AreaStore's IDs persistence by SmallJoker
18:57 Krock       i.e. a "find next free ID" would be needed somehow
18:57 Krock       probably just a loop more
19:34 sfan5       Krock: getWieldedItem changing its entire behaviour if you omit an optional parameter leaves a bitter taste
19:35 sfan5       if I saw an api with two pointer-returned values where one can be nullptr, I'd expect "providing an argument and not using it" to be equivalent to "not providing an argument"
19:36 sfan5       it's fine for this PR, though
19:36 Krock       well yeah
19:37 Krock       in other locations both are used so it wasn't that weird