Time  Nick       Message
09:13 Andrey     can #10363 be merged now? It has already been ready and reviewed long ago
09:13 ShadowBot  https://github.com/minetest/minetest/issues/10363 -- [MainMenu] Add clear button and icon for search one. by Andrey2470T
09:14 Andrey     the PR is pretty trivial
10:04 nerzhul_   hey
11:26 Krock      will merge #10476 in 10 minutes
11:26 ShadowBot  https://github.com/minetest/minetest/issues/10476 -- Add missing type check to InvRef:set_lists() by Zughy
11:35 Krock      merging
12:30 jas_       instead of loading games from world directories (which is fine), I propose creating an archive format, call it .MTZ for example, and allow to load any available resources from it.  put the resource.mtz in minetest/packs and it detects the world, the game, and the settings
14:10 pmp-p      jas_: hi, would that help for incremental downloads and integrity checks ? what about pack/unpack tools for every platform ?
14:29 jas_       traditionally a standardized format is used
14:29 jas_       for example, all archive managers worth their bits can read quake .pak (or quake3 .pk3) filez
14:30 rubenwardy if you were to do that, you'd make it a .zip file but with a different extension
14:30 rubenwardy like how .jar files are actually .zip files
14:30 jas_       that is correct
14:31 jas_       to my mind i think of tgz tho, where it's the combination of archive and compression
14:32 pmp-p      that precise combination has a an horrible access cost
14:32 pmp-p      zip is fine
14:33 rubenwardy zip is the archive format that Minetest supports
14:33 rubenwardy I think this is probably overengineering
14:34 sfan5      s/minetest/irrlicht/
14:35 sfan5      irrlicht can read stuff from a zip file but MT doesn't use the irrlicht filesystem functions
14:35 rubenwardy I thought Minetest supported zips itself
14:36 rubenwardy zips are supported in order to install things from ContentDB
14:36 rubenwardy it doesn't matter too much whether the support is by Minetest or Irrlicht
14:37 sfan5      dunno how cdb extracts those
14:38 sfan5      but media loading goes through normal filesystems calls, not Irrlichts filesystem stuff (which would support zips)
20:14 Zughy      hey there. Is there a reason variables like this are called "co" and not "sao"? https://github.com/minetest/minetest/blob/81c66d6efb9fb0ab8a03b40e2bc22aa49eff9a04/src/script/lua_api/l_object.cpp#L747
20:18 rubenwardy \o/
20:19 rubenwardy I don't know, probably no reason
20:20 Zughy      Is it alright if I refactor them on a separate PR? It's confusing
20:20 Zughy      (yes, I was working on that class)
20:20 Krock      copypasta-related. was clientobject or so, perhaps
20:26 rubenwardy there wouldn't have been a client-side API at that point
20:30 Zughy      so uhm, is it a yes or a no? Because I guess it'd ask for a few rebases, but at the same time it'd make the code more coherent
20:32 sfan5      just change the name to whatever you think fits
20:59 pgimeno    there are quite a lot of similar functions, are they all going to be changed? that will make the PR hard to read
21:11 Zughy      uhm, one class at a time?