ScummVM IRC Log - 11 Jun 2021

[Back to Index]

[00:32] --> gsi_ joined #scummvm.
[00:35] <-- gsi left irc: Ping timeout: 252 seconds
[00:39] --> reset joined #scummvm.
[01:24] <Scummette> [scummvm] dreammaster pushed 1 new commits to master: https://git.io/JZzeo
[01:24] <Scummette> scummvm/master 49e8ae9 dreammaster: AGS: Added detection entries
[01:35] --> _sev_ joined #scummvm.
[01:35] #scummvm: mode change '+o _sev_' by ChanServ!ChanServ@services.libera.chat
[01:38] <-- _sev left irc: Ping timeout: 268 seconds
[02:05] <-- Lightkey left irc: Ping timeout: 268 seconds
[02:17] --> Lightkey joined #scummvm.
[02:37] <grogbot> <patches> the real question is if there's a templeos port
[02:46] <-- ced117 left irc: Ping timeout: 244 seconds
[02:47] --> ced117 joined #scummvm.
[03:19] LePhilousophe[m] <-- (~lephilous@scummvm/LePhilousophe) left irc: Ping timeout: 244 seconds
[05:20] <Scummette> [scummvm] digitall closed pull request #3059: STARTREK: Fix format-security warnings (master...fix-format-security-warnings) https://git.io/JZR5D
[05:20] <Scummette> [scummvm] digitall pushed 1 new commits to master: https://git.io/JZgyY
[05:20] <Scummette> scummvm/master e12c0f3 xavery: STARTREK: Fix format-security warnings
[05:51] <eriktorbjorn> Anyone around who knows the Starship Titanic engine?
[06:17] <Scummette> [scummvm] eriktorbjorn pushed 1 new commits to master: https://git.io/JZ2LB
[06:17] <Scummette> scummvm/master b099687 eriktorbjorn: TITANIC: Make Esc at the start-up dialog quit the game
[07:14] <eriktorbjorn> I guess not. Oh well, I've filed a bug report about my issue: https://bugs.scummvm.org/ticket/12616
[07:21] LePhilousophe[m] --> (~lephilous@scummvm/LePhilousophe) joined #scummvm.
[07:21] #scummvm: mode change '+o LePhilousophe[m]' by ChanServ!ChanServ@services.libera.chat
[07:29] <Scummette> [scummvm] ysj1173886760 opened pull request #3061: DIRECTOR: offset fix (master...offset-fix) https://git.io/JZ2PA
[07:42] --> ny00123 joined #scummvm.
[08:56] <-- SylvainTV left irc: Read error: Connection reset by peer
[08:57] --> SylvainTV joined #scummvm.
[08:57] <-- Dominus left irc: Ping timeout: 245 seconds
[08:58] --> grogbot1 joined #scummvm.
[09:00] <-- ScummBot left irc: Ping timeout: 244 seconds
[09:00] <-- grogbot left irc: Read error: Connection reset by peer
[09:01] <-- Unseen2 left irc: Ping timeout: 272 seconds
[09:01] --> Unseen2 joined #scummvm.
[09:01] --> ScummBot joined #scummvm.
[09:01] Nick change: grogbot1 -> grogbot
[09:11] <eriktorbjorn> Hmm... We seem to have a deadlock in the AGOS MIDI code on shutdown. (Of course, it doesn't happen every time, but often enough. At least with the MT-32 emulator.)
[09:16] <eriktorbjorn> The order of things seems to be that AGOS's MidiPlayer gets destructed while emumidi.h is running readBuffer(), but before it calls _timerProc(). Then MidiPlayer locks the mutex, leaving _timerProc() waiting for the same mutex, I think. And that makes _driver->close() never finish somehow.
[09:17] <-- SylvainTV left irc: Read error: Connection reset by peer
[09:21] --> Dominus joined #scummvm.
[09:44] <eriktorbjorn> I'm not sure how to fix it, but I've filed it as https://bugs.scummvm.org/ticket/12617
[09:53] --> girafe joined #scummvm.
[09:55] <-- girafe2 left irc: Ping timeout: 264 seconds
[10:19] <grogbot> <ordinarytranquil> Hey all 👋
[10:27] <Scummette> [scummvm] alxpnv pushed 2 new commits to master: https://git.io/JZVen
[10:27] <Scummette> scummvm/master 75c1b82 alxpnv: ASYLUM: rename two ActorStatus values and related opcodes
[10:27] <Scummette> scummvm/master 71f3d82 alxpnv: ASYLUM: rename Scene::pointIntersectsRect() to pointBelowLine()
[10:29] <Scummette> [scummvm] sev- pushed 2 new commits to master: https://git.io/JZVfe
[10:29] <Scummette> scummvm/master 44fbf42 sev-: SCUMM: Added support for Japanese Mac Dig
[10:29] <Scummette> scummvm/master 96c5f72 sev-: NEWS: Mention Jap Mac Dig support
[11:40] <-- TMM_ left irc: Quit: https://quassel-irc.org - Chat comfortably. Anywhere.
[11:40] --> TMM_ joined #scummvm.
[13:23] <Scummette> [scummvm] mduggan pushed 1 new commits to master: https://git.io/JZwBI
[13:23] <Scummette> scummvm/master a8e83a8 mduggan: ULTIMA8: Call full code for enterFastArea from intrinsic
[13:46] <Scummette> [scummvm] SupSuper pushed 2 new commits to master: https://git.io/JZwr0
[13:46] <Scummette> scummvm/master 2f59326 SupSuper: NEWS: Added Nightlong
[13:46] <Scummette> scummvm/master 03fae0e SupSuper: TRECISION: Enable Nightlong testing
[14:40] <Scummette> [scummvm] ScummVM-Translations pushed 1 new commits to master: https://git.io/JZref
[14:40] <Scummette> scummvm/master dda5f4e tag2015: I18N: Update translation (Italian)
[15:01] <grogbot> <_athrxx> @eriktorbjorn It's the usual mutex thing, your ticket description sums it up well enough.
[15:02] <grogbot> <_athrxx> I think I worked around it for the PC-98 sound drivers. I guess, I should have fixed it for the rest of the engine, too.
[15:03] <grogbot> <_athrxx> Should be sufficient to move the mutex lock in the destructor down before clearConstructs(), although it could probably go completely. But I am just guessing atm. I'll have to test it...
[15:10] <eriktorbjorn> _athrxx: Thanks. I'm a bit lost
[15:11] <eriktorbjorn> _athrxx: But at least this time I could add my debugging messages without the problem going away. That's not always the case. :-)
[15:13] <grogbot> <_athrxx> Yes, minor changes of the timing may already "solve" the issue 😂
[15:14] <grogbot> <_athrxx> Same with single-stepping through the code. The issue might get more apparent or vanish...
[15:16] <grogbot> <_athrxx> But if we move down the mutex lock (or remove it completely) I have to make some changes to PC-98 Midi, too. That one currently relies on the AGOS mutex being locked.
[15:17] <grogbot> <_athrxx> But that's no trouble, just have to change the destruction order a bit there, too
[15:40] <-- ny00123 left irc: Quit: Leaving
[15:51] <Scummette> [scummvm] athrxx pushed 1 new commits to master: https://git.io/JZrVt
[15:51] <Scummette> scummvm/master 4a6dbc7 athrxx: AGOS: attempt to fix bug no. 12617 ("AGOS: Deadlock when shutting down game")
[15:51] <Scummette> [scummvm] ScummVM-Translations pushed 1 new commits to master: https://git.io/JZrV3
[15:51] <Scummette> scummvm/master ac276c3 : I18N: Update translations templates
[15:54] <grogbot> <_athrxx> @eriktorbjorn I have pushed the changes that I mentioned. But I tested only for a bit. And without tsan.
[15:54] <grogbot> <_athrxx> So no guarantee that this is already the perfect solution.
[16:28] Harekiet (harekiet@kittens.harekiet.com) got netsplit.
[16:39] Harekiet (harekiet@kittens.harekiet.com) got lost in the net-split.
[16:48] --> Harekiet joined #scummvm.
[17:31] <eriktorbjorn> _athrxx: Well, I wasn't able to reproduce the deadlock in 20 attempts, so that's promising. Thanks!
[17:31] <eriktorbjorn> _athrxx: I have no experience with tsan, though.
[17:53] <Scummette> [scummvm] lotharsm pushed 1 new commits to master: https://git.io/JZoBr
[17:53] <Scummette> scummvm/master 28b835f lotharsm: NEWS: Update German NEWS file
[17:53] <Scummette> [scummvm] ScummVM-Translations pushed 1 new commits to master: https://git.io/JZoBM
[17:53] <Scummette> scummvm/master 434452d : I18N: Update translation files
[18:20] <grogbot> <_athrxx> @eriktorbjorn: 20 attempts is sure twice as much as I did 😅 . And these things are very dependent on race conditions. So always good to have more testing. But I think/hope that fix is simple enough, anyway.
[18:58] --> Deledrius_ joined #scummvm.
[19:02] <-- Deledrius left irc: Ping timeout: 264 seconds
[19:20] --> DominusExult joined #scummvm.
[19:22] <-- Dominus left irc: Ping timeout: 264 seconds
[19:22] Nick change: DominusExult -> Dominus
[20:08] <grogbot> <chkuendig> hey, i'm working again on the webassembly port and I was wondering if somebody has input on a minimal initial featureset which would help the project (and provide enough utility to get merged)
[20:11] <grogbot> <rootfather> hi @chkuendig, I just checked out your latest demo version - I love what you did so far, it's a really amazing project
[20:17] <Scummette> [scummvm] bluegr pushed 2 new commits to master: https://git.io/JZKlK
[20:17] <Scummette> scummvm/master a8882c9 bluegr: TRECISION: Fix Joshua's action after combining inventory items
[20:17] <Scummette> scummvm/master 24d791b bluegr: TRECISION: Remove superfluous inventory state variables
[20:17] <Scummette> [scummvm] ScummVM-Translations pushed 1 new commits to master: https://git.io/JZKlD
[20:17] <Scummette> scummvm/master ae99147 : I18N: Update translations templates
[20:25] <grogbot> <Mataniko> right off the bat i'd be nice if you had some basic config set up
[20:25] <grogbot> <Mataniko> so it starts up in opengl mode, aspect ratio correction, etc
[20:26] <grogbot> <Mataniko> data passing to config via url params
[20:26] <grogbot> <Mataniko> that would be useful for embedding
[20:28] <grogbot> <Mataniko> (i'd use that on the site to start a specific game and bypassing launcher)
[20:29] <grogbot> <chkuendig> i'm trying to build up a basic config setup
[20:29] <grogbot> <chkuendig> btw, where is the default established in code? i currently inject it directly into the FS (i.e. grab it out from the browser and then manually inject it in the JS part of the page)
[20:30] <grogbot> <chkuendig> thanks 🙂
[20:30] <grogbot> <chkuendig> the data passing via url params is a great idea! i guess also launch parameters?
[20:32] <grogbot> <Mataniko> yeah. via JS works too
[20:34] <grogbot> <SupSuper> i think the defaults are all in commandline.cpp
[20:34] <grogbot> <SupSuper> dunno if backends have their own
[20:37] <grogbot> <chkuendig> in that case i already have it. that's how all the games are loaded when you go to http://localhost:8080/scummvm.html
[20:50] <-- TMM_ left irc: Quit: https://quassel-irc.org - Chat comfortably. Anywhere.
[20:50] --> TMM_ joined #scummvm.
[21:03] <Scummette> [scummvm] bluegr pushed 3 new commits to master: https://git.io/JZKiu
[21:03] <Scummette> scummvm/master ff97b24 bluegr: TRECISION: Remove the now redundant maskMouse variable
[21:03] <Scummette> scummvm/master 271ee6a bluegr: TRECISION: Change _inventorySpeed into a static const
[21:03] <Scummette> scummvm/master 0effa2f bluegr: TRECISION: Make _sortTableReplay private
[21:03] <Scummette> [scummvm] ScummVM-Translations pushed 1 new commits to master: https://git.io/JZKiV
[21:03] <Scummette> scummvm/master 8936c4b : I18N: Update translation files
[21:11] --> Deledrius__ joined #scummvm.
[21:12] Nick change: Deledrius__ -> Deledrius
[21:14] <-- Deledrius_ left irc: Ping timeout: 264 seconds
[22:13] <Scummette> [scummvm] SupSuper pushed 1 new commits to master: https://git.io/JZ6Ua
[22:13] <Scummette> scummvm/master 2caa965 SupSuper: TRECISION: Fix subtitle crash
[22:17] <Scummette> [scummvm] bluegr pushed 1 new commits to master: https://git.io/JZ6TP
[22:17] <Scummette> scummvm/master 2b30872 bluegr: TRECISION: Rename the filedump dumpfile, add a dumpanim command
[22:39] <-- speachy left irc: Quit: WeeChat 3.0.1
[22:43] <-- reset left irc: Quit: reset
[23:26] <Scummette> [scummvm] bluegr pushed 1 new commits to master: https://git.io/JZ6z3
[23:26] <Scummette> scummvm/master 9ec9b28 bluegr: TRECISION: Fix alt room initialization
[00:00] --- Sat Jun 12 2021