Time Nick Message 08:16 sfan5 merging #14851, #14845 in like 5 mins 08:16 ShadowBot https://github.com/minetest/minetest/issues/14851 -- Minimize data sent in the default user agent by sfan5 08:16 ShadowBot https://github.com/minetest/minetest/issues/14845 -- Warn contributors about not to run updatepo.sh manually by Emojigit 11:40 Krock changelog updated to a7a719261, whereas c3efcb389 is the most recent one mentioned. 11:41 Krock please let me know if there's any feature that I forgot to mention 11:41 Krock even though I went through the git log line by line, it is still possible that I scrolled past one by accident (if surrounded by build system or doc fix commits) 12:22 MTDiscord I reproduced the "'irrevent' is being used without being initialized" issue and left a comment on the issue thread. I found other ways to reproduce it that fail on different lines, and it looks like it has something to do with the fact that the field being assigned to is a bit-field. 12:45 sfan5 is is my opinion that we could stop wasting time on this issue by just zero-initializing the struct 12:50 MTDiscord I don't think there's any reason for the investigation of the cause of the warning to block merging the fix. 12:56 Krock or push a warning ignore instruction, then re-enable 8) 12:59 MTDiscord It'd have to be pushed and popped in multiple places. 12:59 Krock it wasn't meant serious. zero-init is good. 13:19 MTDiscord FWIW I tested the change on MSVC and it does resolve the issue. 13:19 Krock thanks 13:37 MTDiscord I got tired of seeing the whole screen of warnings and wondering whether any of them were related to the issue, so #14872. 13:37 ShadowBot https://github.com/minetest/minetest/issues/14872 -- Clean up MSVC warnings in CIrrDeviceSDL.{h,cc} by JosiahWI