ScummVM IRC Log - 18 Feb 2017

[Back to Index]

[00:02] <wjp> another one of the cases where you can't walk close enough to where the scripts expect you to go?
[00:12] <m_kiewitz> it seems thee is possibly one special place, where the scripts will try to move ego backwards and it gets stuck somehow
[00:12] <m_kiewitz> should be trivial to fix, the same happened downstairs in that game
[00:18] <m_kiewitz> i bet it's stepsize related
[00:39] <-- girafe left irc: Read error: Connection reset by peer
[00:55] <-- Farmboy0 left irc: Remote host closed the connection
[01:01] --> Vampire0 joined #scummvm.
[01:01] <-- Vampire0_ left irc: Ping timeout: 240 seconds
[01:10] <-- m_kiewitz left irc: Quit: technology isn't intrinsically good or evil. It's how it's used. Like the Death Ray.
[01:23] <rsn8887> Is the goal of ScummVM to make the game scripts better than the original?
[01:25] <snover> we try to fix bugs when possible.
[01:42] <-- Henke37 left irc: Quit: ERR_SHUTDOWN
[01:49] <rsn8887> I see.
[01:51] <rsn8887> Is there any reason why the apple builds always take way longer in the queue on Travis CL?
[01:54] <snover> probably fewer machines in that cluster
[02:18] --> GitHub30 joined #scummvm.
[02:18] <GitHub30> [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vD7LZ
[02:18] <GitHub30> scummvm/master c8c96e9 Paul Gilbert: TITANIC: Fix playing phonograph when cylinder holder is open
[02:18] GitHub30 (GitHub30@192.30.252.41) left #scummvm.
[02:26] <-- WooShell left irc: Quit: If you understand or if you don't, if you believe or if you doubt - There's a universal justice, and the eyes of truth are always watching you.
[02:33] <-- Dominus left irc: Ping timeout: 240 seconds
[02:35] --> Dominus joined #scummvm.
[02:35] <-- Dominus left irc: Changing host
[02:35] --> Dominus joined #scummvm.
[02:41] <-- TMM left irc: Ping timeout: 240 seconds
[02:46] --> TMM joined #scummvm.
[02:46] #scummvm: mode change '+o TMM' by ChanServ!ChanServ@services.
[02:50] --> Vampire0_ joined #scummvm.
[02:54] <-- Vampire0 left irc: Ping timeout: 255 seconds
[03:32] --> GitHub188 joined #scummvm.
[03:32] <GitHub188> [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vD7qS
[03:32] <GitHub188> scummvm/master e72a56e Paul Gilbert: TITANIC: Renamings for CArm class
[03:32] GitHub188 (GitHub188@192.30.252.42) left #scummvm.
[03:33] --> omer_mor_ joined #scummvm.
[03:37] <-- omer_mor left irc: Ping timeout: 255 seconds
[03:57] --> GitHub162 joined #scummvm.
[03:57] <GitHub162> [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vD7mM
[03:57] <GitHub162> scummvm/master 8dc3b4a Paul Gilbert: TITANIC: Renamings for CMaitreD
[03:57] GitHub162 (GitHub162@192.30.252.40) left #scummvm.
[04:05] --> GitHub167 joined #scummvm.
[04:05] <GitHub167> [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vD7Yk
[04:05] <GitHub167> scummvm/master 086e89b Paul Gilbert: TITANIC: Refactored CMaitreD static to instance field
[04:05] GitHub167 (GitHub167@192.30.252.34) left #scummvm.
[04:13] <-- dreammaster left irc:
[04:16] <-- snover left irc: Ping timeout: 260 seconds
[04:22] --> snover joined #scummvm.
[04:22] #scummvm: mode change '+o snover' by ChanServ!ChanServ@services.

[05:02] <-- diehumblex left irc: Quit: Connection closed for inactivity
[06:39] <-- LittleToonCat left irc: Remote host closed the connection
[06:40] --> omer_mor joined #scummvm.
[06:42] <-- omer_mor_ left irc: Ping timeout: 240 seconds
[07:06] --> ajax16384 joined #scummvm.
[07:06] #scummvm: mode change '+o ajax16384' by ChanServ!ChanServ@services.
[07:15] <-- nutron left irc: Remote host closed the connection
[07:25] <-- ajax16384 left irc: Read error: Connection reset by peer
[07:25] --> diehumblex joined #scummvm.
[07:50] --> nutron joined #scummvm.
[07:57] --> m_kiewitz joined #scummvm.
[07:57] <-- m_kiewitz left irc: Changing host
[07:57] --> m_kiewitz joined #scummvm.
[07:57] #scummvm: mode change '+o m_kiewitz' by ChanServ!ChanServ@services.
[08:04] <-- Lightkey left irc: Ping timeout: 240 seconds
[08:06] <-- TMM left irc: Read error: Connection reset by peer
[08:08] --> TMM joined #scummvm.
[08:08] #scummvm: mode change '+o TMM' by ChanServ!ChanServ@services.
[08:09] <-- TMM left irc: Max SendQ exceeded
[08:10] --> TMM joined #scummvm.
[08:10] #scummvm: mode change '+o TMM' by ChanServ!ChanServ@services.
[08:18] --> Lightkey joined #scummvm.
[08:19] --> jamm joined #scummvm.
[08:30] --> waltervn joined #scummvm.
[08:30] #scummvm: mode change '+o waltervn' by ChanServ!ChanServ@services.
[08:30] <waltervn> morning
[08:46] --> Begasus joined #scummvm.
[08:56] --> Begas_VM joined #scummvm.
[09:05] <m_kiewitz> oh noes another game bug
[09:05] <m_kiewitz> https://www.twitch.tv/videos/122787128
[09:05] <m_kiewitz> at around 1:54:40
[09:06] <m_kiewitz> he talked to some character while laura was moving automatically
[09:06] <m_kiewitz> then that other character wanted to leave the screen and got stuck on Laura lol
[09:06] <m_kiewitz> quite obviously also a script bug :(
[09:12] --> t0by joined #scummvm.
[09:12] <-- t0by left irc: Changing host
[09:12] --> t0by joined #scummvm.
[09:12] #scummvm: mode change '+v t0by' by ChanServ!ChanServ@services.
[09:15] <-- demonimin_ left irc: Remote host closed the connection
[09:19] --> demonimin joined #scummvm.
[09:26] --> Henke37 joined #scummvm.
[09:52] --> lubomyr joined #scummvm.
[10:23] <-- lubomyr left irc: Quit: KVIrc 4.2.0 Equilibrium http://www.kvirc.net/
[10:34] --> WooShell joined #scummvm.
[10:36] <wjp> oh, that sounds like a potentially tricky one
[10:38] <m_kiewitz> well i guess Laura walks to some specific spot and you are then supposed to enter commands
[10:38] <m_kiewitz> so we would have to disable the text field while shes walking. But yeah, in case there is no target script that is called when she reaches destination, then it will get tricky
[10:39] <m_kiewitz> and in any case, some extra space needs to be found for that additional code
[10:45] <WooShell> good meowning =^.^=
[10:51] --> GitHub159 joined #scummvm.
[10:51] <GitHub159> [scummvm] waltervn pushed 2 new commits to master: https://git.io/vD74f
[10:51] <GitHub159> scummvm/master 9ed6333 Walter van Niftrik: ADL: Fix restoring of global vars in v4+
[10:51] <GitHub159> scummvm/master 549ac7c Walter van Niftrik: ADL: Implement hires6 save/restore
[10:51] GitHub159 (GitHub159@192.30.252.42) left #scummvm.
[10:51] <-- Harekiet left irc: Ping timeout: 258 seconds
[10:52] <waltervn> I managed to get to the end of hires5, with a bug in my code that destroyed almost half of the game variables on restore. Curious.
[10:53] <waltervn> I should say, it destroyed all the global (non-region) variables.
[10:58] <wjp> m_kiewitz: in this one Laura doesn't seem to move in a scripted way
[10:58] <m_kiewitz> ah she isn't?
[10:58] <m_kiewitz> then it's really bad :(
[10:58] <m_kiewitz> to me she seemed scripted
[10:58] <wjp> in his second attempt he walks Laura to a spot at the top of the screen to make sure she's out of the way
[10:59] <m_kiewitz> but yeah maybe the player just walked like that
[10:59] <m_kiewitz> dang
[10:59] <m_kiewitz> then it's going to be a pain to fix
[10:59] <wjp> (1:56:30)
[11:00] <wjp> I wonder if there could be avoidance code in the scripts that's just broken, or if there is no avoidance at all
[11:00] <m_kiewitz> looks to me like no avoidance at all
[11:00] <m_kiewitz> but well i will fix the cupboard one first, that should be trivial
[11:00] <wjp> I'm kind of afraid of that too
[11:00] <m_kiewitz> i had similar ones downstairs in that game
[11:00] <m_kiewitz> were simple to fix
[11:00] <m_kiewitz> the exact same happened
[11:01] <m_kiewitz> you don't own it on GOG?
[11:01] <m_kiewitz> i wonder if they used ScummVM or not
[11:01] <wjp> ah, the armour one
[11:01] <m_kiewitz> in case they did, would it make sense to backport the patches and then release another 1.9 or whatever the current version of ScummVM is
[11:01] <wjp> I didn't buy the GOG one yet
[11:02] <-- waltervn left irc: Read error: Connection reset by peer
[11:03] --> waltervn joined #scummvm.
[11:03] #scummvm: mode change '+o waltervn' by ChanServ!ChanServ@services.
[11:03] <m_kiewitz> it would be weird in case they use ScummVM on twich, but sell it w/ dosbox
[11:04] --> Harekiet joined #scummvm.
[11:04] <wjp> they don't have the "powered by scummvm" bit on the game pages
[11:06] <m_kiewitz> well then they have even more issues in the game lol
[11:06] <m_kiewitz> in case they use dosbox
[11:08] <wjp> it's quite bad luck to hit two of these bugs in a single playthrough
[11:09] <m_kiewitz> yeah
[11:09] <m_kiewitz> that guy would be the perfect play tester :P
[11:10] <wjp> s/bad luck/good luck/ then :-)
[11:12] <-- diehumblex left irc: Quit: Connection closed for inactivity
[11:22] <wjp> m_kiewitz: there is some hope
[11:23] <wjp> I'm not 100% sure I'm looking at the right script, but in "goSee" in script 264, it does try to move ego out of the way if she's in a few specific 'bad' rectangles
[11:23] <wjp> ("goSee" is triggered after 'tell[/lil]/gertie<about')
[11:30] <m_kiewitz> wjp: oh wow
[11:30] <m_kiewitz> just got a reply from GOG
[11:30] <m_kiewitz> "As for the release itself, we're using ScummVM for both titles."
[11:32] <wjp> cool
[11:33] <waltervn> does that mean the original binaries are not present?
[11:35] <wjp> I _think_ ego is standing at 103,155 in that twitch video
[11:35] <wjp> which is not inside any of these rects
[11:35] <wjp> s/any/either/
[11:36] <wjp> waltervn: usually they do add the original binaries too
[11:37] <waltervn> ah, nice
[11:37] <waltervn> I somehow recalled that that wasn't the case
[11:38] <wjp> there could be exceptions, I don't know
[11:41] <m_kiewitz> it's quite weird, I thought Colonel's Bequest was almost bug free
[11:41] <m_kiewitz> qfg1 ega is except for i think 1 or 2 corner case issues
[11:43] <wjp> waltervn: LB1 and LB2 on GOG do have the binaries, yes
[11:45] <m_kiewitz> wjp: I guess you just bought them?
[11:45] <wjp> yeah
[11:45] <m_kiewitz> do they use ScummVM 1.9?
[11:45] <m_kiewitz> just to make sure, I think for Larry they used some very old ScummVM release, which was even outdated back when they released it
[11:46] <m_kiewitz> i had to contact them, so that they finally updated it. I must have fixed at least one game breaking game bug in Larry 5 (script bug) inbetween those releases
[11:46] <wjp> LB2 has the NEWS.txt from 1.9.0 in any case
[11:46] <m_kiewitz> great
[11:47] <wjp> LB1 too
[11:47] <wjp> scummvm.exe is ScummVM 1.9.0.2 (Oct 18 2016 23:58:50)
[11:48] <wjp> (in both LB1 and LB2)
[11:59] <-- Begas_VM left irc: Quit: Vision[0.9.8]: i've been blurred!
[12:04] <_sev> wjp, m_kiewitz: gog.com provided us with redeem codes for both colonel bequest games. Who wants those? I have 1 for each
[12:04] <m_kiewitz> i already own them physically, so if anyone else wants to grab them...
[12:04] <_sev> and they're using ScummVM as per their words
[12:05] <m_kiewitz> snover: knock? ^^^
[12:08] <m_kiewitz> waltervn: ^^^?
[12:11] <wjp> oh, oops, I should've held off on buying them :-)
[12:21] <waltervn> I own both of those games (not on GOG though). Maybe snover doesn't have them?
[12:40] <-- l_bratch left irc: Quit: Leaving
[13:01] <-- TMM left irc: Ping timeout: 255 seconds
[13:02] --> TMM joined #scummvm.
[13:02] #scummvm: mode change '+o TMM' by ChanServ!ChanServ@services.
[13:22] --> D0SFreak joined #scummvm.
[13:22] --> ajax16384 joined #scummvm.
[13:22] #scummvm: mode change '+o ajax16384' by ChanServ!ChanServ@services.
[13:25] <-- RLP__ left irc: Ping timeout: 255 seconds
[14:07] --> GitHub137 joined #scummvm.
[14:07] <GitHub137> [scummvm] wjp pushed 1 new commit to master: https://git.io/vD7gT
[14:07] <GitHub137> scummvm/master 905feaa Willem Jan Palenstijn: SCI: Fix unaligned read
[14:07] GitHub137 (GitHub137@192.30.252.41) left #scummvm.
[14:12] <wjp> any hypotheses about the point of the check in sci's savegame.cpp line 460?
[14:14] <waltervn> prevent infinite loop?
[14:15] <wjp> buf is always strictly greater than _buf there
[14:15] --> Vampire0 joined #scummvm.
[14:15] <wjp> ( assert(blockSize > 0) )
[14:16] <-- Vampire0_ left irc: Ping timeout: 240 seconds
[14:17] <waltervn> sure, but I still stick with my earlier guess :)
[14:18] <wjp> I guess I'll just remove it
[14:18] <wjp> findBlockSCI0 also only has the == 0 break
[14:18] <waltervn> sounds reasonable
[14:19] <wjp> and it's been in there since 2010
[14:21] --> GitHub20 joined #scummvm.
[14:21] <GitHub20> [scummvm] wjp pushed 1 new commit to master: https://git.io/vD7ga
[14:21] <GitHub20> scummvm/master cdfe5a3 Willem Jan Palenstijn: SCI: Remove check that is never triggered...
[14:21] GitHub20 (GitHub20@192.30.252.45) left #scummvm.
[14:22] --> RLP__ joined #scummvm.
[14:25] <-- D0SFreak left irc: Ping timeout: 240 seconds
[14:28] --> Farmboy0 joined #scummvm.
[15:04] <-- |Cable| left irc: Read error: Connection reset by peer
[15:14] <-- RLP__ left irc: Ping timeout: 240 seconds
[15:44] --> macdude22 joined #scummvm.
[15:45] <-- ajax16384 left irc: Quit: Leaving
[16:37] <snover> I have LB1 and LB2 from KQ Collection already so I dont need more copies
[16:48] <wjp> I'll probably have caused some minor merge conflicts with those two commits
[16:51] <snover> thats ok
[16:54] --> ny00123 joined #scummvm.
[16:56] --> Littleboy joined #scummvm.
[16:56] #scummvm: mode change '+o Littleboy' by ChanServ!ChanServ@services.
[17:06] <snover> wjp: let me know how qfg4 goes :)
[17:11] <Harekiet> Could do with a sort saves in reverse or I should make less saves :)
[17:12] --> Vampire0_ joined #scummvm.
[17:14] <-- Vampire0 left irc: Ping timeout: 240 seconds
[17:28] <wjp> qfg4 is going great, I just unlocked the Zoltan Cruiser layout B!
[17:28] <wjp> or maybe I'm mixing up games here
[17:37] <snover> its understandable, theyre very similar
[17:38] --> offstream joined #scummvm.
[17:38] offstream (User@h-78-202.a175.priv.bahnhof.se) left #scummvm.
[17:50] <-- Littleboy left irc: Read error: Connection reset by peer
[17:53] <wjp> qfg3 shows a (scummvm) dialog when opening the import screen to explain how imports work
[17:53] <wjp> it might be nice to add that in qfg4 too
[17:57] --> LittleToonCat joined #scummvm.
[17:58] <wjp> hm, the interface (mouseovers and such) is very laggy
[17:59] <snover> wjp: which branch are you running?
[17:59] <wjp> and the mouse cursor is flickering
[17:59] <wjp> your working branch
[18:00] <snover> hm.
[18:00] <snover> what is your graphics/scaler mode?
[18:00] --> gig3x joined #scummvm.
[18:01] <wjp> opengl, <default>
[18:02] <snover> something is wrong with opengl mode
[18:02] <snover> it should run correctly if you use 2x
[18:02] <snover> or&anything that isnt openl
[18:02] <snover> opengl*
[18:03] --> Littleboy joined #scummvm.
[18:03] #scummvm: mode change '+o Littleboy' by ChanServ!ChanServ@services.
[18:03] <wjp> hm, indeed
[18:04] --> ajax16384 joined #scummvm.
[18:04] #scummvm: mode change '+o ajax16384' by ChanServ!ChanServ@services.
[18:05] <rsn8887> I fixed my pull request. How do i make those nice comments with "quoted code" from my PR?
[18:05] <rsn8887> I am fairly new to github, but I like how the devs can "insert" a snippet of a PR.
[18:06] <wjp> that's just what happens when you click on a line and add a comment on that line
[18:06] <wjp> there's no explicit inserting going on
[18:07] <rsn8887> Ah cool
[18:09] <rsn8887> wjp: I made all the changes to my PR https://github.com/scummvm/scummvm/pull/906
[18:10] <rsn8887> wjp: shall I rebase squash? There's now three commits showing the first one and the new ones where I made the changes after the review
[18:10] <t0by> somaen, hey, just checking: did you receive my mail or did it get eaten by the spam filter?
[18:11] <wjp> rsn8887: I'd probably leave it like this for now, so that we can see what you changed
[18:11] <wjp> rsn8887: and rebase/squash again later if people ask for it
[18:12] <rsn8887> wjp: Awesome, I think it will be a real improvement for RPi and Android users that use gamepads with analog joysticks.
[18:15] <wjp> sounds great
[18:16] <wjp> snover: first real issue: importing characters goes wrong due to FileIOReadString being called with a maxsize that is larger than the destination
[18:16] <wjp> this makes the memcpy to the destination throw a warning and then return
[18:17] <wjp> as a quick test, truncating the length but still doing the memcpy seems to fix the import
[18:19] <snover> sheesh.
[18:19] <snover> kFileIO is kind of a mess.
[18:20] <-- ajax16384 left irc: Read error: Connection reset by peer
[18:28] <wjp> hm, inventory oddities
[18:29] <wjp> this one is a bit weird
[18:30] <wjp> in the inventory window, if you click in the item area but not on any item, it seems to act as if you clicked on the currently selected action
[18:30] <wjp> in dosbox it seems to act as if you clicked on a semi-random item nearby, although I can't quite figure out the logic
[18:31] <wjp> although scummvm also switches to that behaviour at some point, apparently
[18:31] <wjp> hrm
[18:32] <wjp> maybe it's actually happening in dosbox too, but the visual effect is almost impossible to see
[18:32] <wjp> so false alarm probably
[18:33] <wjp> when equipping torch: WARNING: Uninitialized read for parameter 1 from method hero::changeGait (room 770, script 28, localCall ffffffff)! (twice)
[18:36] <-- t0by left irc: Quit: Bye!
[18:40] --> exmensa joined #scummvm.
[18:40] <-- exmensa_ left irc: Read error: Connection reset by peer
[18:40] <snover> there are probably a ton of uninitialised param read warnings, since scummvm was silent about them and just returned 0 until that didnt work for shivers
[18:40] <snover> and now it still returns 0 but warns about all of them
[18:41] <snover> im not really sure what the best thing to do about that is, whether there should be a workarounds table for params too, or what.
[18:43] <wjp> oh, the savegame thumbnails are of the menu instead of the game
[18:44] <wjp> right, crash in a place where the original would also semi-regularly crash
[18:48] <wjp> gog adds a patch from NRS for this it seems
[18:49] <wjp> let's see if that one helps
[18:49] <snover> i saw there was an NRS patch, im not sure what to do with that either
[18:49] <snover> before i stopped trying to run through it, i got to the t.r.a.p. puzzle which is speed slider-dependent in the original release
[18:50] <snover> the NRS patch fixes that, but its a recompiled script
[18:50] <snover> or at least one that has had extensive work done on it to increase the script size for the extra code
[18:51] <snover> unrelated, i managed to get to the end of LSL7, but there is one game stopper and serious problems with rendering and inventory
[18:56] <wjp> odd, with the GOG+NRS version, the mouse cursor doesn't animate
[18:57] <wjp> (it does in dosbox)
[19:04] <wjp> oh, by the way, nice thing about the NRS patch for GOG QfG4: it's in a separate subdirectory
[19:10] <m_kiewitz> snover: we should blacklist that one and fix it ourselves
[19:10] <m_kiewitz> so that we don't get another qfg3 mess
[19:11] <wjp> oh no, they mixed the original patches and NRS together in that subdirectory
[19:11] <m_kiewitz> urgh
[19:11] <wjp> at least they seem disjoint
[19:12] <m_kiewitz> we really have to blacklist every single one of them
[19:12] <m_kiewitz> or maybe even do whitelists instead
[19:12] <wjp> the NRS patches for QG4 fix quite a lot of things
[19:12] <m_kiewitz> can you check the timestamps
[19:12] <m_kiewitz> yes, but i would prefer doing it properly without breaking saved games
[19:12] <wjp> 2012-03-22 for the original patches with gog, 1980-1-1 for the NRS patches
[19:12] <m_kiewitz> qfg3 is a 100% mess thanks to those patches
[19:13] <m_kiewitz> at least that makes it easy to identify
[19:13] <m_kiewitz> when we fix issues directly, users also do not need to use any foreign patches
[19:21] <m_kiewitz> wjp: how many fan patch files are there?
[19:22] <wjp> 11 scr+hep pairs
[19:22] <wjp> and the readme lists 11 issues
[19:26] --> dreammaster joined #scummvm.
[19:26] #scummvm: mode change '+o dreammaster' by ChanServ!ChanServ@services.
[19:31] <m_kiewitz> wjp: ah does it say what exactly the problems were?
[19:31] <m_kiewitz> that would make it somewhat easy to fix them ourselves
[19:31] <m_kiewitz> in case there were timer issues, maybe those aren't triggered in ScummVM anyway
[19:32] <m_kiewitz> i couldn't reproduce the common sq6 timer issue in ScummVM ever. Wanted to fix that, but it seems it doesn#t need fixing in our SCI
[19:33] <m_kiewitz> DrMcCoy: do you own Colonel's Bequest / Laura Bow 2 already?
[19:35] <wjp> m_kiewitz: http://www.sierrahelp.com/Patches-Updates/Patches-Updates-Games/QuestForGloryUpdates.html
[19:35] <-- Harekiet left irc: Read error: Connection reset by peer
[19:36] <wjp> m_kiewitz: the notes for the qfg4cd NRS patch are what's in the readme with the gog version
[19:36] <-- jamm left irc: Remote host closed the connection
[19:36] <m_kiewitz> ah, not GOG exclusive like the qfg3 ones
[19:36] --> jamm joined #scummvm.
[19:36] <wjp> hm? Was that GOG exclusive?
[19:36] --> Harekiet joined #scummvm.
[19:37] <m_kiewitz> i thought so, I'm not sure, I don't own it on GOG, but I think there were recompiled scripts in there
[19:37] <m_kiewitz> ah so that one is timer issues only
[19:37] <wjp> there are, but are those not the NRS ones?
[19:37] <m_kiewitz> i guess those should be somewhat simple to fix and maybe a few do not even happen
[19:37] <wjp> which one is timer issues only?
[19:38] <m_kiewitz> http://www.sierrahelp.com/Files/Patches/QuestForGlory/QfG4CD_NRS-B4.zip
[19:38] <m_kiewitz> this seems to be timer issues only for CD
[19:38] <m_kiewitz> ah and right, if we patch those properly, the patches would also work for floppy version
[19:38] <m_kiewitz> im not sure if there are localized versions too, there were for qfg3
[19:38] <m_kiewitz> that's another problem with those fan made patches
[19:38] <wjp> the list of fixes looks like it has non-timer issues too
[19:39] <m_kiewitz> could be all timer related
[19:39] <m_kiewitz> it says right at the top
[19:39] <m_kiewitz> This non-Sierra update is one of NewRisingSun's script level timer bugs patches.
[19:40] <m_kiewitz> do you know the locations of all of those?
[19:40] <m_kiewitz> i have never played qfg4
[19:40] <m_kiewitz> so i guess i would need your help to supply saved games of bugs that you can reproduce in scummvm
[19:41] <m_kiewitz> oh and i guess we could verify that the user applied the sierra patches for qfg4 floppy
[19:41] <m_kiewitz> and add some warning message otherwise
[19:54] --> |Cable| joined #scummvm.
[19:56] Nick change: rootfather|afk -> rootfather
[19:58] <m_kiewitz> wjp: can you please check the atari st / amiga version of colonel's bequest if sierra changed/fixed those rects for that one room (2nd GOG Twitch issue)?
[19:59] <m_kiewitz> they fixed the armor one in those 2 versions according to my comments
[20:00] <wjp> I don't have non-PC versions
[20:02] <m_kiewitz> ah dang
[20:02] --> Vampire0 joined #scummvm.
[20:03] <-- Vampire0_ left irc: Ping timeout: 240 seconds
[20:10] --> GitHub85 joined #scummvm.
[20:10] <GitHub85> [scummvm] sev- pushed 3 new commits to master: https://git.io/vD7QJ
[20:10] <GitHub85> scummvm/master ef07ed0 Eugene Sandulenko: DIRECTOR: Fix CAS* reading for Windows
[20:10] <GitHub85> scummvm/master 6e6f1b1 Eugene Sandulenko: DIRECTOR: Fixed Shared cast filename setting
[20:10] <GitHub85> scummvm/master 84866d6 Eugene Sandulenko: DIRECTOR: Add detection for Meet Mediaband
[20:10] GitHub85 (GitHub85@192.30.252.40) left #scummvm.
[20:17] <-- Begasus left irc: Quit: Ex-Chat
[20:23] <m_kiewitz> wjp: any idea what this is about?
[20:23] <m_kiewitz> http://www.sierrahelp.com/Files/Patches/QuestForGlory/QfG4(QG4IMP).zip
[20:24] <m_kiewitz> is it because overflows of values?
[20:24] <m_kiewitz> i fixed those for qfg3 character import
[20:25] <wjp> could be; I don't know the specifics
[20:32] <-- Vampire0 left irc: Ping timeout: 240 seconds
[20:33] --> Vampire0 joined #scummvm.
[20:35] <snover> wjp: i noted the control panel screenshot thing too, i am not sure what to do about that
[20:41] <-- jamm left irc: Ping timeout: 268 seconds
[21:05] <-- gig3x left irc:
[21:11] <m_kiewitz> snover: on saved games?
[21:11] <snover> yeah
[21:13] <m_kiewitz> well it seems saving game is called while the menu is still visible
[21:13] <m_kiewitz> i doubt we can do much about that
[21:14] <m_kiewitz> sierra didn't do that for sci16, thank god. well we are the ones adding screenshots
[21:14] <m_kiewitz> we could try to script patch our way out of it, but i guess that's going to be a nightmare
[21:15] <wjp> does it happen in more/most/all sci32 games?
[21:16] <m_kiewitz> i saw it at least in space quest 6
[21:16] <m_kiewitz> didn't happen obviously when using our own saved game menu
[21:16] <m_kiewitz> i think gabriel knight 1 was fine, but I'm not absolutely sure.
[21:21] <snover> qfg4 and shivers and sometimes kq7, othersw seem ok
[21:21] <snover> all others*
[21:23] <m_kiewitz> oh that's not so bad then
[21:23] <m_kiewitz> we can look into patching our way out.
[21:27] <snover> shivers actually sends you to a different room to save
[21:28] <snover> so i doubt that theres any patching that one out
[21:32] <m_kiewitz> yeah that's a no no :(
[21:33] <m_kiewitz> well we could hack something up, but would that be worth it? i doubt
[21:33] <m_kiewitz> we could for example check for room change and then make a screenshot in advance
[21:35] <m_kiewitz> snover: would that be possible in theory? do you go to that save room via a simple button or some menu?
[21:35] <snover> menu
[21:35] <snover> well
[21:35] <snover> you press a button
[21:35] <snover> then you go to the menu room
[21:36] <snover> then you press another button which sends you to the save room
[21:36] <m_kiewitz> ah a menu room
[21:36] <m_kiewitz> well we could make a screenshot when going to that menu room then
[21:36] <m_kiewitz> save it inside memory in case the user chooses to save
[21:37] <m_kiewitz> ah, we could even make it really easy for us and if there is some common call for that menu, we could create another kernel call for that game and use it to make the screenshot
[21:37] <m_kiewitz> patch that in -> done
[21:37] <m_kiewitz> have you already implemented the ScummVM specific kernel call functionality?
[21:37] <m_kiewitz> via 0xEx kernel calls?
[21:38] <snover> yeah, i moved the calls to 0xe0
[21:41] --> GitHub164 joined #scummvm.
[21:41] <GitHub164> [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vD7F4
[21:41] <GitHub164> scummvm/master 7762dae Paul Gilbert: TITANIC: Fix Maitre'D playing music check
[21:41] GitHub164 (GitHub164@192.30.252.40) left #scummvm.
[21:43] <-- ny00123 left irc: Remote host closed the connection
[21:49] <wjp> scummvm specific kernel calls? I seem to have missed that
[21:51] <m_kiewitz> wjp: http://www.enderboi.com/log.php?log=scummvm.log.04Feb2017&format=html
[21:52] <wjp> thanks
[21:53] <snover> i hope that isnt the log from when i had a mental breakdown :)
[22:03] <snover> https://zetafleet.com/i/58a8c62648a92.jpg 👍 the last one to start is now starting
[22:06] <m_kiewitz> \o/
[22:06] <m_kiewitz> i wonder if the Japanese Lighthouse would also at least start
[22:06] --> GitHub73 joined #scummvm.
[22:06] <GitHub73> [scummvm] dreammaster pushed 2 new commits to master: https://git.io/vD7bd
[22:06] <GitHub73> scummvm/master d65c717 Paul Gilbert: TITANIC: Workaround original Maitre'D bug where _musicHandler was null
[22:06] <GitHub73> scummvm/master 0343c0d Paul Gilbert: TITANIC: Fix adding held items to inventory after unlocking arms
[22:06] GitHub73 (GitHub73@192.30.252.45) left #scummvm.
[22:12] <snover> ill have this pushed up in just a minute, got a little debuggy stuff to delete
[22:22] <snover> m_kiewitz: if youd like, pull my working branch and give it a shot
[22:22] <snover> the VMD decoder needs some work to support whatever Sierra did to the VMDs in Lighthouse
[22:23] <snover> right now it seems to think some of the videos are stereo 8-bit raw, but i suspect it might actually be stereo 16-bit dpcm
[22:23] <snover> or maybe stereo 8-bit dpcm, but in any case playing it as raw 8-bit audio does not sound right
[22:24] <m_kiewitz> currently in a debug session for colonel's bequest
[22:25] <m_kiewitz> this silly "widget <> has clipping area" warning really needs to get fixed
[22:25] <m_kiewitz> it's a nightmare to single step through scripts
[22:29] <snover> ill be back in a bit. thanks wjp and m_kiewitz for looking into all sorts of stuff today :)
[23:09] <Lightkey> _sev: if you are going to remind GOG.com of the missing "powered by ScummVM", you could also ask them to write it separately of the Windows requirements for multi-platform titles, so that Linux and macOS users will see it..
[23:09] <Lightkey> einmal mit profis
[23:18] --> D0SFreak joined #scummvm.
[23:19] <criezy> Oh, I never noticed there was a "powered by ScummVM" note for games using ScummVM on GoG.com.
[23:19] <criezy> Neat!
[23:21] <Lightkey> see
[23:22] <criezy> I just checked BASS and it is actually present for all three (Windows, Mac and Linux).
[23:22] <m_kiewitz> criezy: Colonel's Bequest + Laura Bow 2 didn't get it (yet), so I'm not sure if others are missing it as well
[23:23] <Lightkey> well, great, more inconsistency
[23:27] <Lightkey> even better, they already have it separate for The 7th Guest, instead of adding it to each platform requirements
[23:34] --> GitHub65 joined #scummvm.
[23:34] <GitHub65> [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vD7pK
[23:34] <GitHub65> scummvm/master 80ba4b5 Paul Gilbert: TITANIC: Renaming and cleanup for CGondolierSlider
[23:34] GitHub65 (GitHub65@192.30.252.41) left #scummvm.
[23:36] <-- waltervn left irc: Quit: Leaving
[23:42] <Lightkey> I don't have Indiana Jones and the Last Crusade on GOG.com yet but isn't it using ScummVM as well?
[00:00] --- Sun Feb 19 2017