Time Nick Message 14:23 PilzAdam celeron55, why did you add the always_collect field to staticdata of builtin items? is there any usage of it? 14:24 celeron55 i am going to add a mod called mt_itemdrop to upstream that will load if item_drop isn't loaded and collect those that have it; the tnt mod that will be included will throw stuff around having that set 14:24 celeron55 s/it/always_collect/ 14:25 PilzAdam so i can use it in item_drop? 14:25 celeron55 or, maybe if you have something else in mind, i'd not make it mind about item_rop 14:25 celeron55 item_drop* 14:25 celeron55 but i am not sure how to make them behave reasonably with each other 14:26 celeron55 PilzAdam: what would you use it for? the point of item_drop is to collect everything always anyway 14:26 PilzAdam there is a field called "collect" in my mod 14:26 PilzAdam it is true for items that are dropped by digging 14:27 celeron55 it is for stuff that should be collected _even if the regular minetest block collection is used_ 14:27 PilzAdam and false for everything else like dropping it with "Q" 14:27 celeron55 yes and you are supposed to keep it so 14:28 celeron55 as for what i saw, it is just a thing for timing ignoring of collection of items for a bit of time? 14:28 PilzAdam yes 14:28 PilzAdam so you dont pick up items that you just dropped with Q 14:28 celeron55 always_collect is completely different 14:28 PilzAdam ok 14:29 PilzAdam so if always_collect is there is a builtin function that picks up the item? 14:29 PilzAdam +true 14:29 celeron55 actually, i think i'll make a setting that defines whether all items will be collected always 14:30 celeron55 so you don't need to maintain your mod... but there'd need to be a way to force it per game for MiniTest to work 14:30 PilzAdam minetest.setting_setbool() 14:30 celeron55 no that wouldn't make sense because it would carry to other games 14:30 PilzAdam and register_on_shutdown() could set it back 14:30 celeron55 AWFUL 14:31 celeron55 go burn in hell 14:31 * PilzAdam goes 14:31 * PilzAdam crys because of pain 14:32 celeron55 i guess there should be a thing called "game settings" 14:32 celeron55 actually, no 14:32 PilzAdam if you add a setting for always collect every items you should be sure you can still throw away items 14:32 celeron55 actually, wtf am i talking about 14:33 celeron55 minitest is it's own game so it has it's own mods 14:33 celeron55 there is no problem in here whatsoever, except the problem of item_drop failing to co-operate with mt_itemdrop 14:33 PilzAdam Add a setting called MC = true 14:33 celeron55 when being in minetest 14:34 celeron55 the solution to that is just to not use item_drop 14:34 PilzAdam item_drop can set the always_collect field to true and dont care about picking up items 14:35 celeron55 but how will it set it? 14:35 celeron55 when? 14:35 PilzAdam in handle_node_drops() 14:35 celeron55 oh that 14:35 PilzAdam i have a luanetity ref and can set it to true 14:36 PilzAdam easy 14:36 celeron55 well that makes sense; now the only problem is the old item_drop that will screw everything, but i guess people will remember to get rid of it 14:36 celeron55 i'll just mentiont that in the blog 14:36 celeron55 -t 14:37 PilzAdam i can check if mt_itemdrop is installed 14:37 celeron55 but you can't modify the mod that people have installed already 14:37 PilzAdam and only add the old code if not 14:37 PilzAdam i dont think people are that stupid 14:38 celeron55 the only way to handle this would be to have a way to make mods conflict with each other, and have mandatory version fields in mods 14:38 celeron55 but that kind of stuff is not really the design principle in here 14:38 PilzAdam oh no 14:38 PilzAdam dont do this 14:38 PilzAdam please 14:38 celeron55 "if it breaks, it breaks" 8) 14:39 PilzAdam you shouldnt call mt_itemdrop mt_itemdrop but mt_itempickup 14:40 celeron55 maybe 14:40 PilzAdam because it actually isnt item drop 14:40 PilzAdam its item pick up 14:40 celeron55 but it handles with "item drops" 14:41 celeron55 8) 14:41 * PilzAdam facepalms 14:41 PilzAdam and it is in minetest so you should call it mt_mt 14:41 celeron55 drop, in addition to being a verb, is a substantive meaning the dropped state of an item 14:42 PilzAdam so mt_item_drop_killer 14:42 PilzAdam because it "kills" the drop state 14:43 celeron55 mt_collect_items_that_have_always_collect 14:43 PilzAdam + _set_to_true 14:43 celeron55 maybe we should enforce this kind of naming convention 14:43 celeron55 for every 10 lines of code, there needs to be one word 14:44 PilzAdam farming = mod_that_add_plants_to_mt_that_can_grow_on_soil_wich_is_created_with_hoes 14:45 celeron55 /giveme mod_that_add_plants_to_mt_that_can_grow_on_soil_wich_is_created_with_hoes:hoe_wood 14:45 PilzAdam hoe_made_of_wood_to_create_soil 14:45 celeron55 _and_get_seeds 14:45 PilzAdam _only_randomly 14:46 PilzAdam should add this as a alias easteregg 14:46 PilzAdam i think this is offtopic, isnt it? 14:47 celeron55 why does that matter 14:49 PilzAdam you have any idea when you will push it to upstream? 20:43 Anchakor damn, now I want to know finnish http://www.youtube.com/watch?v=r9YyaknVx_o 20:49 celeron55 lol, horrible