Time  Nick       Message
00:08 Fixer      what happened?
00:08 Fixer      just take it easy
00:10 paramat    we are =)
00:14 Fixer      be glad Linus Torvalds is not commenting on minetest
00:22 rubenwardy fwiw, paramat: kilbith and Wuzzy were wrong in that issue
00:22 rubenwardy #include <cmath>  does add stuff to the std namespace
00:22 rubenwardy it just doesn't add min / max - that's in algorithms
00:22 rubenwardy that's an easy mistake to make (I've done it)
00:22 rubenwardy *algorithms rather than maths
00:23 rubenwardy https://gcc.gnu.org/onlinedocs/libstdc%2B%2B/libstdc%2B%2B-html-USERS-3.4/cmath-source.html
00:34 paramat    std::min/max and fmin/fmax have differnt requirements, easy to forget. doesn't matter to k, it's a chance to attack
02:05 paramat    later i'll merge #7070 #7100
02:05 ShadowBot  https://github.com/minetest/minetest/issues/7070 -- Biomes: Add X and Z biome limits / Getv3intfield: Fix logic of return bool by paramat
02:05 ShadowBot  https://github.com/minetest/minetest/issues/7100 -- Gennotify docs: Clarify returned positions for decorations by paramat
02:34 paramat    #7104 updated
02:34 ShadowBot  https://github.com/minetest/minetest/issues/7104 -- Spawn level: Add 'get_spawn_level(x, z)' API by paramat
03:13 paramat    now merging #7070 #7100
03:13 ShadowBot  https://github.com/minetest/minetest/issues/7070 -- Biomes: Add X and Z biome limits / Getv3intfield: Fix logic of return bool by paramat
03:13 ShadowBot  https://github.com/minetest/minetest/issues/7100 -- Gennotify docs: Clarify returned positions for decorations by paramat
03:25 paramat    merged, <100 PRs again
04:33 paramat    merging #7115
04:33 ShadowBot  https://github.com/minetest/minetest/issues/7115 -- Lua_api.txt: Split long lines part 1 by paramat
04:38 paramat    merged
07:33 nerzhul    Hello, when somes get time i pushed another cleanup patchset https://github.com/minetest/minetest/pull/7114
15:34 Megaf      sfan5: thanks for reviewing my PR, I will be reviewing your review now.
15:44 Megaf      rubenwardy: Shara: https://github.com/minetest/minetest.github.io/pull/121#pullrequestreview-102684694
15:44 Megaf      !tell paramat https://github.com/minetest/minetest.github.io/pull/121#pullrequestreview-102684694
15:44 ShadowBot  Megaf: O.K.
15:44 Megaf      !title
15:44 ShadowBot  Channel description improvements 2. by Megaf · Pull Request #121 · minetest/minetest.github.io · GitHub
15:54 Megaf      sfan5: is that better? https://github.com/minetest/minetest.github.io/pull/121/files#diff-5cc5127841b3b9e922b2cb7fdc182cfdR18
15:54 Megaf      "Main channel where everyone is welcome to discuss the game or request support."
16:45 sfan5      Megaf: yes that's better
16:46 Megaf      "Main channel where everyone is welcome to discuss the game or request support." or "Main channel for general game and engine discussion or support"
16:47 Megaf      because
16:47 Megaf      16:15 <Krock> Megaf, in what channels is someone not welcome? (regarding web PR)
16:47 Megaf      16:16 <Megaf> Krock: open to suggestions :)
16:47 Megaf      16:16 <Krock> Main channel for general game and engine discussion or support.
16:47 Megaf      valid point ^
16:50 Krock      Could someone please spend about 15 minutes to review/test #6787 ? Testing code exists (allows side-by-side comparison of PR and non-PR builds)
16:50 ShadowBot  https://github.com/minetest/minetest/issues/6787 -- textarea[], field[]: Unify function, fix wrong fallback text by SmallJoker
16:51 Krock      non-member testing is also welcome ofc
20:03 paramat    #7104 updated as requested, retested
20:03 ShadowBot  https://github.com/minetest/minetest/issues/7104 -- Spawn level: Add 'get_spawn_level(x, z)' API by paramat
21:05 nerzhul    Krock okay for me
21:05 nerzhul    if someone can take time to review https://github.com/minetest/minetest/pull/7114 ?
21:17 paramat    looking at the voxelarea stuff
21:23 nerzhul    yeah each commit is independant
21:25 Krock      nerzhul, only doc issues from my side. the other code looks good
21:26 Krock      nerzhul, thanks for your textarea PR review
21:27 nerzhul    Krock nice, i will fix that doc issue, paramat thanks for the 6th commit
21:30 nerzhul    Krock doc is fixed, if it's okay to you, please add your approval :) if okay i will merge after travis pass
21:31 rubenwardy is there a key co-ordinate to view the next commit when in the PR files changed mode?
21:31 rubenwardy or do I just have to use the drop down
21:32 nerzhul    if i'm correct the only way is to use the dropdown
21:33 rubenwardy I'm not sure on the point of going from 3  s16s to a v3s16 when it's not used
21:33 rubenwardy well, it's cleaner but I wonder if it affects performance
21:34 rubenwardy oh nvm
21:34 rubenwardy you do use it
21:34 nerzhul    :p
21:34 rubenwardy I blame github's indentation XD
21:35 rubenwardy oh no, nvm, it's not 8 spaces like in gists \o/
21:38 rubenwardy does getNodeNoCheck really not accept a v3s16?
21:38 nerzhul    inline MapNode getNodeNoCheck(v3s16 p, bool *valid_position)
21:38 nerzhul    => return getNodeNoCheck(p.X, p.Y, p.Z, valid_position);
21:38 rubenwardy hmmm
21:38 rubenwardy it reads better to use that, and inline makes it no slower
21:39 nerzhul    i can add this yes it's a good idea
21:44 nerzhul    rubenwardy it's pushed, i didn't notice this call heh
21:52 rubenwardy lol
21:52 rubenwardy getFreeServerActiveObjectId() is quite a funny algorithm
21:59 rubenwardy is addActiveObject called both when an entity is spawned, and when the entity's mapblock becomes active?
22:01 Krock      where funny = "agh, the hell is this inefficient"
22:01 nerzhul    sorry i don't remember, but i'm sure if it's not the only to audit some old server code we can find some strange things :d
22:02 rubenwardy everything entity related sucks
22:02 rubenwardy might be worth looking at n0's entity spatial partition PR
22:02 rubenwardy although my concern is that it duplicates code which already exists for AreaStore
22:04 rubenwardy addActiveObject seems to only be called by minetest.add_entity
22:06 rubenwardy #6587
22:06 ShadowBot  https://github.com/minetest/minetest/issues/6587 -- Optimize entity-entity collision by numberZero
22:08 rubenwardy wtf
22:08 rubenwardy void getStaticData(std::string *result) const { *result = ""; }
22:10 rubenwardy ah, CLion's just lying about no derivatives
22:10 rubenwardy something about Minetest's source code kills CLion
22:11 nerzhul    rubenwardy yes clion sometimes tells wrong thing in C++ code in inheritance, but it's enhanced in recent versions
22:11 rubenwardy commits 1-4 ok, now checking 5
22:11 rubenwardy gmmm
22:11 rubenwardy I'm only a version behind
22:13 rubenwardy up to date now
22:15 Calinou    rubenwardy: try an EAP build maybe?
22:15 Calinou    (early access preview)
22:15 rubenwardy turned out it was lying to me
22:23 rubenwardy http://doxy.minetest.net/classServer__coll__graph.svg
22:23 rubenwardy interesting to see
22:24 rubenwardy worth doing the Saturday meetings again?
22:26 paramat    it seemed people can't commit to certain times, which is fine and understandble, MT has to fit around RL
22:27 paramat    then at unexpected times devs would be around :]
22:27 nerzhul    merging #7114 (unsquashed)
22:27 ShadowBot  https://github.com/minetest/minetest/issues/7114 -- Code cleanup: fixes + performance updates by nerzhul
22:27 rubenwardy getting your commit count up? XD
22:27 nerzhul    ty Krock, paramat and ruben for review
22:27 rubenwardy just kidding :D
22:35 Krock      rubenwardy, doing meetings sounds good to process the current open PRs :)
22:35 rubenwardy yeah
22:37 rubenwardy I really need to familiarise myself with Minetest's graphics
22:37 rubenwardy in particular meshgen
22:37 rubenwardy there's lots of important PRs I just don't understand
22:41 rubenwardy oooh, new clion splashscreen
22:59 rubenwardy so, 75 PRs in a few months?
22:59 rubenwardy \o/
22:59 rubenwardy #7099
23:00 ShadowBot  https://github.com/minetest/minetest/issues/7099 -- Add formspec theming using prepended strings by rubenwardy
23:00 paramat    but yes i can so meetings
23:04 paramat    #7104 is an easy one
23:04 ShadowBot  https://github.com/minetest/minetest/issues/7104 -- Spawn level: Add 'get_spawn_level(x, z)' API by paramat
23:07 rubenwardy will merge #6787 in 10
23:07 ShadowBot  https://github.com/minetest/minetest/issues/6787 -- textarea[], field[]: Unify function, fix wrong fallback text by SmallJoker
23:08 rubenwardy paramat, done
23:12 paramat    thanks
23:13 paramat    i'll merge 7104 tonight with my next lua.api long line fix PR
23:14 rubenwardy nice
23:15 rubenwardy stupid run in place
23:15 rubenwardy alias cmt="cmake . -DRUN_IN_PLACE=1 -DBUILD_CLIENT=1 -DBUILD_SERVER=0 && make -$
23:15 rubenwardy * alias cmt="cmake . -DRUN_IN_PLACE=1 -DBUILD_CLIENT=1 -DBUILD_SERVER=0 && make -j3"
23:16 rubenwardy now run in place will never revert itself again
23:25 rubenwardy merged
23:31 paramat    ugh forgot docs again
23:31 rubenwardy lol
23:31 rubenwardy make sure to note about returning nil
23:32 rubenwardy also, when will biome manager be nil?
23:32 rubenwardy if it should never be nil, even during loading, then you should use FATAL_ERROR_IF rather than returning nil to the API
23:34 paramat    not sure, copying hmmmm's code
23:35 paramat    might leave docs until after my next lua_api long line PR
23:50 Megaf      !title https://github.com/minetest/minetest.github.io/pull/121/commits/8dcd5e43d085152b323345fd1cce1fbd91e20ec4
23:50 ShadowBot  Megaf: Channel description improvements 2. by Megaf · Pull Request #121 · minetest/minetest.github.io · GitHub
23:55 Fixer      rubenwardy: 75 half-rotten PRs... sadly