Minetest logo

IRC log for #minetest-dev, 2021-12-21

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

All times shown according to UTC.

Time Nick Message
02:05 YuGiOhJCJ joined #minetest-dev
03:28 queria joined #minetest-dev
03:33 queria joined #minetest-dev
03:47 olliy joined #minetest-dev
05:00 MTDiscord joined #minetest-dev
05:08 fluxionary joined #minetest-dev
05:26 beanzilla joined #minetest-dev
09:19 calcul0n joined #minetest-dev
10:54 olliy joined #minetest-dev
12:20 Fleckenstein joined #minetest-dev
12:30 kilbith joined #minetest-dev
12:30 kilbith https://www.phoronix.com/scan.php?page=news_item&px=Zstd-1.5.1-Released
12:32 MTDiscord <Sublayer plank> so it's smaller, faster and all this in a maintenance release? very cool
12:32 appguru joined #minetest-dev
12:58 MTDiscord <luatic> What's the matter with the claim that "string.char" is "unsafe" introduced by the PNG writer PR: https://github.com/minetest/minetest/blob/master/doc/lua_api.txt#L4703 ? How is this substantiated?
13:01 MTDiscord <luatic> Is it because no "locale" can be guaranteed? Why doesn't builtin just call os.setlocale then to settle this? (arguably this might break mods which rely on a specific locale, but introducing insecurity concerning the reliability of string.char and string.byte seems unacceptable to me)
13:06 MTDiscord <luatic> The Lua 5.1 implementation of string.char at least doesn't show anything I consider locale-specific (grep for str_char in the minetest dir to have a look)
13:08 MTDiscord <luatic> The LuaJIT codebase is similar, just grep for string_char there. Just a cast to char.
13:54 sfan5 @luatic https://github.com/minetest/minetest/pull/11485#discussion_r676166902
13:54 sfan5 "Why doesn't builtin just call os.setlocale then" <- gettext also uses the set locale, I think this should answer your question
13:55 sfan5 tochigi: trying with gdb/lldb and checking the stacktraces would probably help determine where it's hanging
14:24 hlqkj joined #minetest-dev
15:05 Taoki joined #minetest-dev
15:13 proller joined #minetest-dev
15:47 calcul0n joined #minetest-dev
15:56 Fixer joined #minetest-dev
16:01 MTDiscord <luatic> sfan5: wouldn't it be possible to just set the locale for Lua though?
16:01 sfan5 no
16:01 MTDiscord <luatic> or to override string.char and string.byte to provide platform-independent behavior?
16:02 sfan5 if you say the implementations are safe why would we need that
16:02 MTDiscord <luatic> well, according to hecktest's comment I was wrong
16:29 queria^clone joined #minetest-dev
16:37 appguru joined #minetest-dev
17:00 Taoki joined #minetest-dev
17:52 olliy joined #minetest-dev
17:53 tochigi sfan5: i am too lazy
17:54 sfan5 ¯\_(ツ)_/¯
18:51 appguru joined #minetest-dev
19:03 v-rob joined #minetest-dev
19:19 v-rob joined #minetest-dev
19:53 kilbith joined #minetest-dev
19:54 proller joined #minetest-dev
20:31 appguru joined #minetest-dev
21:04 v-rob joined #minetest-dev
21:51 GreenXenith joined #minetest-dev

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