Minetest logo

IRC log for #minetest-dev, 2016-03-26

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

All times shown according to UTC.

Time Nick Message
00:01 STHGOM joined #minetest-dev
00:26 rubenwardy joined #minetest-dev
00:42 Lunatrius` joined #minetest-dev
01:22 domtron joined #minetest-dev
01:40 yang2003 joined #minetest-dev
02:07 domtron joined #minetest-dev
02:11 yang2003 joined #minetest-dev
02:24 Wuzzy2 joined #minetest-dev
02:43 luizrpgluiz joined #minetest-dev
02:45 domtron joined #minetest-dev
02:48 luizrpgluiz hello, I enjoyed the new daily build of minetest here in kubuntu 16.04 beta 2, the game's performance was much better now with disabled shaders
03:05 red-001 joined #minetest-dev
04:42 maxxrodz joined #minetest-dev
04:42 maxxrodz hi
04:50 maxxrodz people are getting ignored
04:50 maxxrodz is this minetest
04:50 maxxrodz ok
04:50 maxxrodz is this minetest
04:50 maxxrodz hi
04:53 sofar this channel is for development of the core. other topics, please talk in #minetest, thanks.
06:19 Hunterz joined #minetest-dev
06:36 reisfe joined #minetest-dev
06:54 red-001 joined #minetest-dev
07:19 nrzkt joined #minetest-dev
08:03 Krock joined #minetest-dev
08:16 red-001 joined #minetest-dev
08:39 cd2 joined #minetest-dev
08:54 Obani joined #minetest-dev
09:18 Megaf joined #minetest-dev
10:00 loggingbot_ joined #minetest-dev
10:00 Topic for #minetest-dev is now Minetest core development and maintenance. Freeze at 2016-03-23 (2 weeks). Last release: 0.4.13, Aug 20 2015. Chit-chat goes to #minetest. Consider this instead of /msg celeron55. http://irc.minetest.ru/minetest-dev/ http://dev.minetest.net/
10:09 Calinou joined #minetest-dev
11:16 Obani joined #minetest-dev
11:26 proller joined #minetest-dev
11:46 Calinou joined #minetest-dev
12:03 Fixer joined #minetest-dev
12:15 red-001 joined #minetest-dev
12:24 Obani joined #minetest-dev
13:02 STHGOM joined #minetest-dev
13:23 kaadmy joined #minetest-dev
13:46 ud1 joined #minetest-dev
13:49 fireglow left #minetest-dev
15:11 Megaf joined #minetest-dev
15:45 hmmmm joined #minetest-dev
15:52 iangp joined #minetest-dev
16:08 rubenwardy joined #minetest-dev
16:12 Player_2 joined #minetest-dev
16:13 Dragonop joined #minetest-dev
16:40 Calinou joined #minetest-dev
16:53 turtleman joined #minetest-dev
16:57 PilzAdam joined #minetest-dev
17:13 rom15044 joined #minetest-dev
17:31 red-001 joined #minetest-dev
17:34 rubenwardy joined #minetest-dev
17:45 red-001 joined #minetest-dev
17:52 red-001 joined #minetest-dev
17:58 rubenwardy joined #minetest-dev
18:04 red-001 joined #minetest-dev
18:10 red-001 joined #minetest-dev
18:18 pozzoni joined #minetest-dev
18:29 srifqi joined #minetest-dev
18:34 red-001 joined #minetest-dev
18:40 srifqi joined #minetest-dev
18:45 DFeniks joined #minetest-dev
18:53 red-001 joined #minetest-dev
19:05 red-001 joined #minetest-dev
19:12 rubenwardy joined #minetest-dev
19:35 srifqi ~tell paramat I've updated 2561, please see it.
19:35 ShadowBot srifqi: O.K.
19:41 VargaD joined #minetest-dev
19:49 Darcidride joined #minetest-dev
19:57 red-001 joined #minetest-dev
19:58 Calinou joined #minetest-dev
19:59 red-001 #3919
19:59 ShadowBot https://github.com/minetest/minetest/issues/3919 -- Add base64 encoding and decoding to the lua api. by red-001
20:03 ShadowNinja I found some issues with the clang static analyzer.  Some of them look like false positives, but it found a few good mem leaks.
20:23 Calinou meme leaks
20:23 VanessaE USE UP _ALL_ THE RAM!
20:26 Hunterz joined #minetest-dev
20:35 Fixer VanessaE, develope all the things
22:26 ShadowNinja Clang likes the (new GUIMenu())->drop() pattern just about as much as I do.
22:28 ShadowNinja Apparently set_lists didn't work at all, and safeLoadFile had some leaks.
22:29 ShadowNinja I also found some NULL dereferences and an issue with the SRP API.
22:29 ShadowNinja It handles the case where malloc returns NULL, but then ignores it and operates on uninitialized data.
22:30 ShadowNinja IMO a memory allocation failure should just throw std::bad_alloc and crash, since there isn't much you can do about it.
22:31 ShadowNinja But for not I just made it propagate the error to the caller as a generic SRP error.
22:44 ShadowNinja generateBiomes seems to be copies&pasted across all mapgens but v6.
22:45 Taoki joined #minetest-dev
22:57 ShadowNinja Likewise with checkobject and Lua class-like userdata.
23:03 est31 joined #minetest-dev
23:09 yang2003 joined #minetest-dev
23:12 red-001 joined #minetest-dev
23:28 est31 joined #minetest-dev
23:28 est31 the ->drop() is utter shit
23:28 est31 because it isn't called when unwinding the stack
23:29 est31 so e.g. if you allocate, and unwind before the drop, it gives big trouble
23:30 est31 ShadowNinja, about SRP, the bad alloc throwing should happen at the border, and on the minetest side of the story.
23:30 PilzAdam est31, https://github.com/minetest/minetest/commit/25da0594eb09bb0e72816aaf8280573a668af00c#commitcomment-16856039 thanks
23:30 est31 PilzAdam, np
23:31 est31 ShadowNinja, unless there is an actual bug in SRP itself.
23:31 ShadowNinja est31: In that case the object grabs itself and then frees itself when done though, which I think is a pretty ugly design.
23:32 ShadowNinja est31: And as for SRP, minetest will only get "SRP authentication failed", nothing more specific.
23:32 est31 can you show the code
23:33 ShadowNinja est31: hash_num in calculate_M drops the error.
23:34 est31 hmmm
23:34 est31 err
23:34 est31 I meant hrmmm
23:35 est31 hmmmm, sorry didnt want to ping you
23:35 ShadowNinja Compiler warning/bug fixing patch so far: http://ix.io/uZy/diff
23:35 ShadowNinja est31: You didn't -- until you added that extra 'm' :-P
23:35 est31 lol
23:35 est31 well shrug
23:36 est31 ShadowNinja, about the patch, I don't like it. SRP is meant as a C library, the c++ code is only included for the hash alg i think
23:37 est31 throwing bad_alloc is a bad habit of c++ libraries, C libraries just return NULL pointers or otherwise indicate an error
23:37 ShadowNinja est31: What would you prefer?
23:38 ShadowNinja est31: calculate_M is called by srp_user_process_challenge, which doesn't even really have a way to report errors.
23:38 est31 ShadowNinja, make it return an int and check where the function is called
23:39 ShadowNinja s/int/bool/  I did that.  It prevents the read of uninitialized data, but it otherwise just ignores the error.
23:40 est31 ShadowNinja, C has no bool type
23:40 est31 at least last time I checked
23:40 ShadowNinja est31: stdbool.h / _Bool
23:40 ShadowNinja c99
23:41 est31 ah well
23:42 ShadowNinja est31: Line 66/67 of the patch, I'm not sure which time_of_day should be used.  I used the current value, but that may be a bug.  Do you know which is correct?
23:45 est31 I have no idea why you are even changing it
23:45 est31 reviewing this is pointless without having the output from the clang program
23:49 est31 its too large to be reviewed via irc, it best gets in line as a normal PR
23:49 est31 sorry
23:49 ShadowNinja est31: time_of_day is initialized to the struct's value, then immediately reset to the client's value.
23:49 est31 (also partly I want to do other things, non minetest things now)
23:50 ShadowNinja O.K.

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