Time Nick Message 00:14 paramat you wouldn't want all that in console/terminal anyway, too many lines 00:20 Fixer i found a solution -> tail 00:22 Fixer please take a look https://transfer.sh/A7GQQ/out.mkv (look when spikes in mainloop_other uppear and watch log) 00:24 Fixer it is 2min file 00:25 Fixer and it is not like you are on dialup 07:15 rdococ we 07:19 sofar we indeed 07:45 rdococ we 12:58 red-002 yeah the console seems to work in a werid way 12:59 red-002 it seems to ignore log_level 13:04 celeron55 it has been broken forever and it's very annoying 13:05 celeron55 somebody broke it in some log rework commit 13:13 rubenwardy t'was #2628 13:13 ShadowBot https://github.com/minetest/minetest/issues/2628 -- Clean up logging by ShadowNinja 15:32 red-002 updated #5292 15:32 ShadowBot https://github.com/minetest/minetest/issues/5292 -- Fix crash that can be caused by the shutdown command. by red-001 16:04 tenplus1 hi folks 16:05 tenplus1 does anyone know of an actual way to use voxelmanip to create an area with light nodes that doesnt have lighting glitches all over ??? 16:07 glut32 I have a question : what's the coding style for a member in c++? is it m_currentTextRect or m_current_text_rect or m_CurrentTextRect? 16:10 glut32 (there are nothing about this in http://dev.minetest.net/Code_style_guidelines) 16:11 sfan5 second one 16:11 sfan5 and m_ ONLY if existing code in that class uses it 16:17 nrzkt m_ = member 16:36 glut32 for a protected member must i use "m_" prefix? 21:57 octacian paramat: why exactly do you think that the ability to clear another player's inventory is unneeded? 21:57 octacian Personally, I think it is very useful especially for map creators. 21:57 Calinou think of administration purposes, yeah 21:57 Calinou Minetest is not very good at that right now, compared to a modded Minecraft 21:57 octacian Yep. And I'm sure there's other uses too. 21:58 octacian Plus, it's only like 10 extra lines of code. And it'd make the platform slightly better for map creators and server admins. 21:58 paramat i know uses can be thought up, but i feel they're very specialist and will be rarely used 21:59 Calinou specialist things is what turns open source into a make-or-break, to most people 21:59 Calinou if you can only do the basic things using a software, people won't use it much 22:00 Calinou Minetest is already here to be configurable, it's more of a KDE than a GNOME 3 already ;) 22:00 octacian Yeah. 22:00 nerzhul Calinou, gnome 3.0 maybe 22:00 Calinou we want to promote the opposite of "one size/usecase fits all" 22:00 nerzhul but 3.22 is nice 22:01 paramat can a mod do this? 22:01 octacian Overall though, I think the ability to clear the inventories of other people is very important. 22:01 Calinou paramat: yes, but a mod shouldn't be required for this 22:01 octacian paramat: yes, a mod can. But this is an integral feature IMO (and in the opinions of others), and to my understanding the point is to make basic stuff like that simple. 22:01 Calinou installing mods that contain 15 lines of code just lengthtens the /mods list :P 22:02 octacian lol I've even been using my own modules system in my mods as of late to avoid using modpacks and therefore shorten the /mods list 22:02 Calinou also, letting the "clear inventory" command clear the inventory of other players does not really complexify the syntax of the command 22:02 octacian No. The only difference is /clearinv and /clearinv 22:04 paramat well if you describe some extra uses in the thread maybe you'll convince me 22:04 paramat i can't block it, if it gets the necessary approval it will go ahead 22:04 Calinou paramat: clearing the inventory of abusers/cheaters/griefers/... 22:04 Calinou (when banning them isn't justified) 22:05 paramat just ban them 22:05 octacian That's kinda mean.. xD 22:05 paramat this is specialist enough to be a mod 22:05 octacian (sometimes anyway) 22:05 paramat or some other punishment 22:06 octacian ..and chances are, as part of that punishment, you're gonna want to clear their inventory :P 22:06 octacian Well, probably anyway. 22:07 paramat still not a good, widely useful example 22:08 octacian rubenwardy: If you can, take a look at my clearinv PR, and/or see above discussion. 22:08 sfan5 uh 22:08 sfan5 just because you can't think of an use doesn 22:08 sfan5 't mean there isn't any 22:08 paramat clearing the inventory of a bad player does not prevent their bad behaviour 22:08 sfan5 generally for these kind of commands it just makes sense to allow it to apply to other people 22:09 sfan5 if we have /kill to suicide why not have /kill ; if we have /clearinv why not have /clearinv too? 22:09 sfan5 it's no additional burden to maintain 22:09 octacian Exactly lol 22:09 sfan5 you can't argue this way if someone wants to have support for e.g. some random rare image format (e.g. tga) 22:10 sfan5 since that is actually work to do 22:10 sfan5 but this is trivial 22:10 paramat what i'm resisting is adding every idea people can think up, otherwise the code becomes monstrous, features need to be justified 22:10 sfan5 there is no reason NOT to do it 22:10 paramat that's no reason to do it 22:10 sfan5 umm 22:10 sfan5 this feature is three lines of code 22:10 sfan5 roughtly 22:10 sfan5 -t 22:10 paramat and a poor reason to do it 22:11 paramat well, nevermind it will probably go ahead 22:11 octacian True-ish. The real reason is because it doesn't make sense to not have the ability to target another player. 22:12 twoelk this is a feature that helps server admins and minetest could use some more built in tools in that field 22:12 octacian Anyways, I'm not trying to start an argument myself, so I'll leave this be xD 22:29 paramat i'll probably become neutral once some admin comment who know it will be very useful 23:09 paramat will merge game#1574 later 23:09 ShadowBot https://github.com/minetest/minetest_game/issues/1574 -- Torches: Reduce light source level from 13 to 12 by paramat 23:12 paramat hm i wonder if we can merge game#1552 also? it's had plenty of time for other devs to look at it 23:12 ShadowBot https://github.com/minetest/minetest_game/issues/1552 -- Leafdecay: node timer based implementation, API. by sofar 23:48 paramat please can anyone review #4967 ? i think this is top priority. once i test it i will +1 23:48 ShadowBot https://github.com/minetest/minetest/issues/4967 -- New bulk node light update by juhdanad 23:50 kaeza can somebody review #5292 and merge it please? 23:50 ShadowBot https://github.com/minetest/minetest/issues/5292 -- Fix crash that can be caused by the shutdown command. by red-001 23:53 kaeza see http://irc.minetest.net/minetest-dev/2017-02-21#i_4818167 23:59 paramat hm 'trivial' i may be able to merge it