Minetest logo

IRC log for #minetest-dev, 2017-02-23

| Channels | #minetest-dev index | Today | | Google Search | Plaintext

All times shown according to UTC.

Time Nick Message
00:00 red-001 joined #minetest-dev
00:14 paramat you wouldn't want all that in console/terminal anyway, too many lines
00:16 betterthanyou711 joined #minetest-dev
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
00:30 proller joined #minetest-dev
00:34 ircSparky joined #minetest-dev
00:34 ircSparky joined #minetest-dev
00:59 ptv joined #minetest-dev
01:05 XtremeHacker joined #minetest-dev
01:09 shivajiva joined #minetest-dev
01:19 paramat joined #minetest-dev
01:46 fuchstraumer joined #minetest-dev
02:40 ircSparky joined #minetest-dev
02:40 ircSparky joined #minetest-dev
02:45 diemartin joined #minetest-dev
02:57 ircSparky joined #minetest-dev
03:02 ircSparky joined #minetest-dev
03:02 ircSparky joined #minetest-dev
03:11 Miner_48er joined #minetest-dev
03:23 ircSparky_ joined #minetest-dev
04:20 kaeza joined #minetest-dev
04:20 Hunterz joined #minetest-dev
05:43 kaeza joined #minetest-dev
05:50 lumidify joined #minetest-dev
06:06 Hunterz joined #minetest-dev
06:08 nerzhul joined #minetest-dev
07:15 rdococ we
07:19 sofar we indeed
07:37 proller joined #minetest-dev
07:45 rdococ we
07:52 ssieb joined #minetest-dev
08:44 nrzkt joined #minetest-dev
10:26 red-002 joined #minetest-dev
10:26 red-002 joined #minetest-dev
11:28 Fixer joined #minetest-dev
11:35 Warr1024 joined #minetest-dev
12:12 YuGiOhJCJ joined #minetest-dev
12:43 rubenwardy joined #minetest-dev
12:50 rubenwardy joined #minetest-dev
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
13:13 DI3HARD139 joined #minetest-dev
13:13 DI3HARD139 joined #minetest-dev
13:14 DI3HARD139 joined #minetest-dev
13:24 fling joined #minetest-dev
13:24 lisac joined #minetest-dev
13:47 ircSparky joined #minetest-dev
14:07 DI3HARD139 joined #minetest-dev
14:07 proller joined #minetest-dev
14:15 xunto joined #minetest-dev
14:26 Darcidride joined #minetest-dev
14:28 octacian joined #minetest-dev
14:32 Fixer joined #minetest-dev
15:09 Hunterz joined #minetest-dev
15:13 Player_2 joined #minetest-dev
15:24 blaze joined #minetest-dev
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
15:55 XtremeHacker joined #minetest-dev
16:01 betterthanyou710 joined #minetest-dev
16:04 glut32 joined #minetest-dev
16:04 tenplus1 joined #minetest-dev
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:06 betterthanyou711 joined #minetest-dev
16:07 betterthanyou711 joined #minetest-dev
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:15 tenplus1 left #minetest-dev
16:16 lumidify joined #minetest-dev
16:17 nrzkt m_ = member
16:36 glut32 for a protected member must i use "m_" prefix?
16:41 Hunterz joined #minetest-dev
17:05 rubenwardy joined #minetest-dev
17:40 ptv joined #minetest-dev
17:43 ircSparky joined #minetest-dev
17:43 ircSparky joined #minetest-dev
17:45 rubenwardy joined #minetest-dev
17:51 ircSparky joined #minetest-dev
17:53 book` joined #minetest-dev
18:16 blaze joined #minetest-dev
18:23 numZero joined #minetest-dev
18:24 Krock joined #minetest-dev
18:24 Krock joined #minetest-dev
18:29 twoelk joined #minetest-dev
18:41 nerzhul joined #minetest-dev
18:43 betterthanyou710 joined #minetest-dev
18:55 troller joined #minetest-dev
18:56 rubenwardy joined #minetest-dev
19:00 ircSparky joined #minetest-dev
19:03 ssieb joined #minetest-dev
19:37 ircSparky joined #minetest-dev
19:53 fwhcat joined #minetest-dev
19:55 Warr1024 joined #minetest-dev
20:14 friti joined #minetest-dev
20:20 ircSparky joined #minetest-dev
21:45 paramat joined #minetest-dev
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 <player name>
22:03 rubenwardy joined #minetest-dev
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 <playername>; if we have /clearinv why not have /clearinv <playername> 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
22:42 kaeza joined #minetest-dev
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:28 xunto joined #minetest-dev
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/minet​est-dev/2017-02-21#i_4818167
23:59 paramat hm 'trivial' i may be able to merge it

| Channels | #minetest-dev index | Today | | Google Search | Plaintext