[00:19] Nick change: Stormkeeper -> Storm-AFK [00:33] TMM (~Hein-Piet@fsf/member/pdpc.professional.tmm) left irc: Remote host closed the connection [00:44] GitHub121 (GitHub121@gateway/service/github.com/x-dnooftxksjevlsqf) joined #scummvm. [00:44] [scummvm] dafioram opened pull request #1159: MOHAWK: MYST: Fix Observatory displayed hour from 0:XX to 12:XX (master...fix10486) https://git.io/vperd [00:44] GitHub121 (GitHub121@gateway/service/github.com/x-dnooftxksjevlsqf) left #scummvm. [00:44] moralrecordings (~scott@ec2-52-64-49-234.ap-southeast-2.compute.amazonaws.com) left irc: Ping timeout: 256 seconds [00:44] moralrecordings (~scott@ec2-52-64-49-234.ap-southeast-2.compute.amazonaws.com) joined #scummvm. [00:47] LubomirR (~LubomirR_@bband-dyn239.178-40-29.t-com.sk) left irc: Quit: Leaving [01:02] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 260 seconds [01:03] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [01:03] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [01:08] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 268 seconds [01:09] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [01:09] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [01:16] Yuv422 (~Yuv422@60-240-103-165.tpgi.com.au) joined #scummvm. [01:18] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 264 seconds [01:20] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [01:20] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [01:24] Yuv422 (~Yuv422@60-240-103-165.tpgi.com.au) left irc: Quit: Yuv422 [01:32] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) left irc: Ping timeout: 265 seconds [01:41] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 264 seconds [01:42] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [01:42] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [01:49] noobineer (~noobineer@184-215-35-214.pools.spcsdns.net) joined #scummvm. [01:51] Dominus (~dominus@unaffiliated/dominus) left irc: Ping timeout: 260 seconds [01:52] DominusExult (~dominus@unaffiliated/dominus) joined #scummvm. [01:52] Nick change: DominusExult -> Dominus [01:55] noobineer (~noobineer@184-215-35-214.pools.spcsdns.net) left irc: Ping timeout: 264 seconds [02:22] macdude22_baseme (~macdude22@2604:2d80:4035:8ce0:1d51:b589:50f0:1cce) left irc: Quit: Textual IRC Client: www.textualapp.com [02:58] linux_dr (uid130126@gateway/web/irccloud.com/x-rtkqxmfpddkyhiay) left irc: Quit: Connection closed for inactivity [03:01] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) joined #scummvm. [03:28] dreammaster (~dreammast@c-73-149-116-247.hsd1.ma.comcast.net) left irc: [03:37] Yuv422 (~Yuv422@60-240-103-165.tpgi.com.au) joined #scummvm. [03:51] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) left irc: Ping timeout: 260 seconds [04:06] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) joined #scummvm. [04:12] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) left irc: Max SendQ exceeded [04:13] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) joined #scummvm. [04:20] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) left irc: Ping timeout: 240 seconds [04:20] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) joined #scummvm. [04:34] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) left irc: Read error: Connection reset by peer [04:34] jamm (~jam@unaffiliated/jamm) left irc: Read error: Connection reset by peer [05:01] SylvainTV (~Sylvain@ALille-458-1-12-69.w86-228.abo.wanadoo.fr) left irc: Ping timeout: 276 seconds [05:06] _sev_ (~sev@scummvm/undead/sev) left irc: Ping timeout: 264 seconds [05:24] waltervn (~waltervn@541B2DBA.cm-5-4a.dynamic.ziggo.nl) joined #scummvm. [05:24] #scummvm: mode change '+o waltervn' by ChanServ!ChanServ@services. [05:24] morning [05:27] waltervn: good morning! [05:52] GitHub165 (GitHub165@gateway/service/github.com/x-kdrxwrnodqrbddhe) joined #scummvm. [05:52] [scummvm] bgK pushed 1 new commit to master: https://git.io/vpeME [05:52] scummvm/master b1f927f Bastien Bouclet: MOHAWK: MYST: Make changeBackgroundSound wait for the effect to complete... [05:52] GitHub165 (GitHub165@gateway/service/github.com/x-kdrxwrnodqrbddhe) left #scummvm. [05:53] macdude22 (~macdude22@2604:2d80:4035:8ce0:51bd:85c4:4175:9b0e) joined #scummvm. [05:55] GitHub88 (GitHub88@gateway/service/github.com/x-wzcpmuweculhzvlx) joined #scummvm. [05:55] [scummvm] bgK pushed 1 new commit to master: https://git.io/vpeMo [05:55] scummvm/master 24956f0 Bastien Bouclet: MOHAWK: MYST: Fix observatory slider sounds repeating too quickly... [05:55] GitHub88 (GitHub88@gateway/service/github.com/x-wzcpmuweculhzvlx) left #scummvm. [06:59] rootfather (~rootfathe@unaffiliated/rootfather) joined #scummvm. [06:59] #scummvm: mode change '+o rootfather' by ChanServ!ChanServ@services. [07:02] _sev_ (~sev@d158167.upc-d.chello.nl) joined #scummvm. [07:02] _sev_ (~sev@d158167.upc-d.chello.nl) left irc: Changing host [07:02] _sev_ (~sev@scummvm/undead/sev) joined #scummvm. [07:02] #scummvm: mode change '+o _sev_' by ChanServ!ChanServ@services. [07:18] morning folks [07:19] PSA: The GoG.com version of Myst and Riven are updated to ScummVM \o/ [07:25] you were not around when bgK told us five days ago [07:26] oops, sorry [07:42] I wonder if Cyan already compiled the Gold Master for their upcoming Myst Collection or if we would have time for an additional testing period for Myst and Riven [07:46] rootfather: I'm planning to contact them to see if they need anything and maybe send them updated builds with all the Myst / Riven / Exile fixes that have been done since the release of 2.0.0 [07:48] bgK: that's great [08:00] Drenn (~Drenn@toroon0713w-lp140-04-69-158-16-62.dsl.bell.ca) left irc: Ping timeout: 240 seconds [08:04] stroggoff (~Praetoria@ppp-94-67-175-131.home.otenet.gr) joined #scummvm. [08:27] _sev_ (~sev@scummvm/undead/sev) left irc: Ping timeout: 240 seconds [08:44] I'm confused, has Colin retired?! [08:49] what? [08:49] why do you think this? [08:51] LittleToonCat (~littlecat@142.162.196.136) left irc: Remote host closed the connection [08:52] Lightkey (~Darklock@p200300F693D1378122CF30FFFE083718.dip0.t-ipconnect.de) left irc: Ping timeout: 255 seconds [08:58] I hope this is not related to him not getting the proper access to update the buildbot server as mentioned here: https://github.com/csnover/scummvm-buildbot/issues/28#issuecomment-379426629 [08:59] that's sad... [09:02] indeed [09:05] Lightkey (~Darklock@p200300F693D1375622CF30FFFE083718.dip0.t-ipconnect.de) joined #scummvm. [09:13] DominusExult (~dominus@unaffiliated/dominus) joined #scummvm. [09:15] Dominus (~dominus@unaffiliated/dominus) left irc: Ping timeout: 260 seconds [09:15] Nick change: DominusExult -> Dominus [09:29] ajax16384 (~User@109.60.130.33) joined #scummvm. [09:29] #scummvm: mode change '+o ajax16384' by ChanServ!ChanServ@services. [09:34] t0by (~t0by@unaffiliated/t0by) joined #scummvm. [09:34] #scummvm: mode change '+o t0by' by ChanServ!ChanServ@services. [09:34] criezy, I think I'll tackle the unknown game dialog :) [09:34] I'm just afraid it doesn't meet your expectations [09:35] but on the other hand - what could go wrong besides a "lol nope" :D [10:08] GitHub195 (GitHub195@gateway/service/github.com/x-zilztbbaysajmopg) joined #scummvm. [10:08] [scummvm] criezy pushed 1 new commit to master: https://git.io/vpeQM [10:08] scummvm/master 57ae3a6 Lothar Serra Mari: I18N: Update translation (German)... [10:08] GitHub195 (GitHub195@gateway/service/github.com/x-zilztbbaysajmopg) left #scummvm. [10:12] rootfather: https://bugs.scummvm.org/ticket/10196#comment:15 [10:15] bgK: oh, I totally missed that one [10:24] bonki: snover did amazing work on the new buildbot it's a real shame he was not able to deploy it. [10:30] LubomirR (~LubomirR_@bband-dyn239.178-40-29.t-com.sk) joined #scummvm. [10:32] true... [10:43] SylvainTV (~Sylvain@ALille-458-1-12-69.w86-228.abo.wanadoo.fr) joined #scummvm. [10:43] #scummvm: mode change '+o SylvainTV' by ChanServ!ChanServ@services. [10:52] rootfather: great. Let me know if you need help at some point. [10:52] first question since I don't know the convention [10:52] bonki, bgK: I missed that comment from snover as well. That's sad :-( [10:53] girafe (~girafe@LFbn-LYO-1-449-251.w2-7.abo.wanadoo.fr) joined #scummvm. [10:53] is it preferred to write the new dialog functions in a new gui/unknownGameDialog.cpp/.h or should the dialog functions be included in advancedDetector.cpp? [10:54] Do not add it to advancedDetector.cpp. I was wondering however if it might be better to use a new file or to add it to engines/dialogs.cpp. [10:55] I think you can use new files. That's what I would tend to do anyway. [11:09] rootfather, criezy: regarding the unknown game dialog, this branch contains code that moves the unknown game message out of the advanced detector, so it is hopefully easier to implement a GUI dialog: https://github.com/bgK/scummvm/commits/detection-refactor-unknown [11:09] it is untested, might be completely broken, but feel free to use it if you find it useful [11:10] (and I won't mind if you ignore it) [11:33] GitHub156 (GitHub156@gateway/service/github.com/x-rdmxmhsjspnnpean) joined #scummvm. [11:33] [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vpedt [11:33] scummvm/master 95741aa Paul Gilbert: XEEN: Yet further fixes to clip scene drawing to scene area [11:33] GitHub156 (GitHub156@gateway/service/github.com/x-rdmxmhsjspnnpean) left #scummvm. [11:37] LubomirR_ (~LubomirR_@bband-dyn186.178-40-116.t-com.sk) joined #scummvm. [11:40] LubomirR (~LubomirR_@bband-dyn239.178-40-29.t-com.sk) left irc: Ping timeout: 240 seconds [11:45] dreammaster (~dreammast@c-73-149-116-247.hsd1.ma.comcast.net) joined #scummvm. [11:45] #scummvm: mode change '+o dreammaster' by ChanServ!ChanServ@services. [11:46] GitHub18 (GitHub18@gateway/service/github.com/x-ymddgfiairuszygq) joined #scummvm. [11:46] [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vpedK [11:46] scummvm/master a38e16b Paul Gilbert: XEEN: Fix opening of Ali Baba's chest [11:46] GitHub18 (GitHub18@gateway/service/github.com/x-ymddgfiairuszygq) left #scummvm. [11:47] rootfather (~rootfathe@unaffiliated/rootfather) left irc: Ping timeout: 268 seconds [11:58] Yuv422 (~Yuv422@60-240-103-165.tpgi.com.au) left irc: Quit: Yuv422 [11:59] LubomirR_ (~LubomirR_@bband-dyn186.178-40-116.t-com.sk) left irc: Quit: Leaving [12:00] LubomirR (~LubomirR_@bband-dyn186.178-40-116.t-com.sk) joined #scummvm. [12:00] rootfather (~rootfathe@p57874261.dip0.t-ipconnect.de) joined #scummvm. [12:01] rootfather (~rootfathe@p57874261.dip0.t-ipconnect.de) left irc: Changing host [12:01] rootfather (~rootfathe@unaffiliated/rootfather) joined #scummvm. [12:01] #scummvm: mode change '+o rootfather' by ChanServ!ChanServ@services. [12:02] _sev_ (~sev@scummvm/undead/sev) joined #scummvm. [12:02] #scummvm: mode change '+o _sev_' by ChanServ!ChanServ@services. [12:06] criezy, is it possible for gettext translated strings to use the original string instead of the translated one? [12:07] e.g. I'd like to print the translated unknown game message in the dialog, but store the original english one in the logfile and console [12:26] That's very unfortunate, re: csnover :'( [12:29] rootfather: I guess you could set the locale to "C" before calling _() to return the untranslated string and restore the locale afterwards [12:29] indeed [12:40] rootfather: Yes, what I had in mind was display the translating string in the dialog, but have the untranslated string in the log and console. The copy to clipboard would also copy the untranslated string. [12:41] And to do so you just have to not translate it. [12:41] By that I mean use "some" text for the log and console and _("some text") for the dialog. [12:42] The is a small difficulty here as the full message is actually built from several bits. So you might have to build two version of the message, the translated one and untranslated one, at the same time. [12:51] d'oh. yes, just don't translate it :D [12:59] ccawley2011 (8bdec910@gateway/web/freenode/ip.139.222.201.16) joined #scummvm. [13:07] GitHub153 (GitHub153@gateway/service/github.com/x-nzwqghorogufynwo) joined #scummvm. [13:07] [scummvm] rsn8887 pushed 1 new commit to master: https://git.io/vpeNZ [13:07] scummvm/master a7bc089 Cameron Cawley: IMAGE: INDEO: Use the system pixel format for non-8bpp screen modes [13:07] GitHub153 (GitHub153@gateway/service/github.com/x-nzwqghorogufynwo) left #scummvm. [13:25] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 260 seconds [13:29] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [13:29] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [13:33] t0by (~t0by@unaffiliated/t0by) left irc: Quit: Leaving [13:50] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 256 seconds [13:52] GitHub150 (GitHub150@gateway/service/github.com/x-eyzwwgnsfniqdzux) joined #scummvm. [13:52] [scummvm] dreammaster pushed 1 new commit to master: https://git.io/vpexL [13:52] scummvm/master caba188 Paul Gilbert: XEEN: Mode renamings, fix loading savegame during script dragging party [13:52] GitHub150 (GitHub150@gateway/service/github.com/x-eyzwwgnsfniqdzux) left #scummvm. [13:55] ccawley2011 (8bdec910@gateway/web/freenode/ip.139.222.201.16) left irc: Ping timeout: 260 seconds [13:56] Coould someone check if the logs or Buildbot is up and running? I can't access either. Even though downforeveryone says they're up [13:57] Oh actually, trying to go to buildbot now is giving me a privacy error about bugs.scummvm.org, and if I ignore it, it's going to the bug tracker instead [13:57] both are up here [13:58] gnarf [13:58] "Could not load widget position for..." [13:58] I'm too stupid :D [14:01] Well, I have to leave for some gym time in a few minutes, so apologies if Buildbot is still running and borks on some errors due to my recent commits. ;) [14:05] dreammaster: buildbot.scummvm.org is HTTP-only, you are probably surfing to https://buildbot.scummvm.org [14:06] I assume we are hosting them on the same server without SNI so there is only one TLS-enabled doc root which seems to be Trac [14:06] so, just navigate to plain HTTP and it should work [14:07] Unfortunately, it's just giving me "There is no internet connection": ERR_PROXY_CONNECTION_FAILED [14:07] Build if you can connect, could you tell me whether there are any of the servers finished compiling without any errors? [14:07] sec [14:08] about 70% done without errors [14:08] for caba18856914daac4339f040b39b7b73795c6310 [14:08] Okay, safe me to head off then. Thanks [14:08] deffo [14:08] enjoy! :) [14:08] dreammaster (~dreammast@c-73-149-116-247.hsd1.ma.comcast.net) left irc: [14:09] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [14:09] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [14:25] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 268 seconds [14:25] DrMcCoy (~drmccoy@odin.strc.at) joined #scummvm. [14:25] DrMcCoy (~drmccoy@odin.strc.at) left irc: Changing host [14:25] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [14:25] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [14:31] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 256 seconds [14:33] GitHub166 (GitHub166@gateway/service/github.com/x-ejvclwcaaczjyjih) joined #scummvm. [14:33] [scummvm] bonki pushed 1 new commit to master: https://git.io/vpepG [14:33] scummvm/master 3747d85 Adrian Frühwirth: JANITORIAL: Fix whitespace [14:33] GitHub166 (GitHub166@gateway/service/github.com/x-ejvclwcaaczjyjih) left #scummvm. [14:39] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [14:39] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [14:44] criezy, do you think it's better to spawn/craete an entire new dialog or rely on the GUI::...alert() function since the new dialog would be rather simple too? [14:46] GitHub83 (GitHub83@gateway/service/github.com/x-hinccsrjxbwphama) joined #scummvm. [14:46] [scummvm] bonki pushed 1 new commit to master: https://git.io/vpepK [14:46] scummvm/master cf96f3f Adrian Frühwirth: JANITORIAL: Indent GFXTransaction code blocks [14:46] GitHub83 (GitHub83@gateway/service/github.com/x-hinccsrjxbwphama) left #scummvm. [14:51] Dark-Star (~quassel@2a01:238:427f:2f00:3060:fc6b:969e:dcf0) left irc: Ping timeout: 265 seconds [14:52] Dark-Star (~quassel@2a01:238:427f:2f00:3060:fc6b:969e:dcf0) joined #scummvm. [14:55] girafe (~girafe@LFbn-LYO-1-449-251.w2-7.abo.wanadoo.fr) left irc: Read error: Connection reset by peer [14:57] girafe (~girafe@LFbn-LYO-1-449-251.w2-7.abo.wanadoo.fr) joined #scummvm. [14:58] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 255 seconds [15:02] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [15:02] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [15:06] rootfather: I was thinking of a new dialog so that we could thrown in a couple of additional functionalities, such as copy the message to the clipboard (of the clipboard feature is available) and open URL to bug tracker (if open URL feature is available). [15:17] ccawley2011 (8bdec910@gateway/web/freenode/ip.139.222.201.16) joined #scummvm. [15:30] _sev_ (~sev@scummvm/undead/sev) left irc: Ping timeout: 265 seconds [15:31] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 264 seconds [15:32] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [15:32] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [15:32] after declaring/creating a dialog, what [15:32] what's the right command to invoke it? [15:32] I get errors like C:\msys64\home\lotharsm\scummvm/engines/advancedDetector.cpp:360: undefined reference to `GUI::UnknownGameDialog::UnknownGameDialog()'... [15:34] I would bet you missed to add your new cpp file to build system [15:35] nope, I entered it in modules.mk [15:35] :( [15:51] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 264 seconds [15:57] ny00123 (~ny00123@141.226.11.177) joined #scummvm. [16:03] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [16:03] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [16:07] That would have been my bet as well. [16:08] rootfather: can you push your current code to your fork so that I can have a look? [16:08] Unless you already found out what the issue is. [16:08] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 256 seconds [16:10] this is solved, but I think I'm not invoking/drawing the dialog properly - the test dialog I made won't open [16:10] pushing asap [16:11] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [16:11] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [16:12] criezy: https://github.com/rootfather/scummvm/commit/03647540e8f051b3b02c3f01c74d2e7a357ebb74 [16:16] basically if drawing and opening the dialog itself is done, I know pretty much how to implement the other stuff (besides the URL opening part) [16:21] First question, but this is not the issue here: why did you reimplement open() and close() in the UnknownGameDialog()? [16:21] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 264 seconds [16:22] this was taken by the fluidsynth dialog [16:22] void FluidSynthSettingsDialog::open() { [16:22] Dialog::open(); [16:23] I thought this is necessary [16:23] If you don't reimplement those in your dialog that derives from the Dialog class, it will just execute the function from Dialog. So this is unnecessary. [16:24] But this is not an error either. Both work. [16:24] Drenn (~Drenn@toroon0713w-lp140-04-69-158-16-62.dsl.bell.ca) joined #scummvm. [16:25] Also does your code actually compile? Or was the code you push in some intermediate state? [16:25] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [16:25] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [16:26] rootfather: You declare the UnknownGameDialog() as taking no argument, but in advanceDetector you use it with 4 arguments: GUI::UnknownGameDialog::Dialog(1, 1, 320, 200); [16:26] That shouldn't even compile. [16:27] rootfather: And one more thing: in advancedDetector.cpp after you create the dialog you need to run it. [16:28] it compiles - when I pass no arguments then it doesn't compile expecting 4 int arguments [16:28] and how do I actually run it? :D [16:28] So something like GUI::UnknownGameDialog dialog(); dialog.runModal(); [16:28] _sev_ (~sev@scummvm/undead/sev) joined #scummvm. [16:28] #scummvm: mode change '+o _sev_' by ChanServ!ChanServ@services. [16:29] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) joined #scummvm. [16:29] rootfather: And those 4 arguments should actually be used when you call Dialog() in unknown-game-dialog.cpp [16:29] Here: https://github.com/rootfather/scummvm/blob/03647540e8f051b3b02c3f01c74d2e7a357ebb74/gui/unknown-game-dialog.cpp#L35 [16:30] And indeed, that probably compiles. I didn't see the ::Dialog part in the 'GUI::UnknownGameDialog::Dialog(1, 1, 320, 200)' line because I didn't expect it (and it shouldn't be thetre ;) ). [16:32] DrMcCoy (~drmccoy@xoreos/drmccoy) left irc: Ping timeout: 268 seconds [16:34] GitHub162 (GitHub162@gateway/service/github.com/x-taiiqfgilwaqtmxc) joined #scummvm. [16:34] [scummvm] bgK pushed 1 new commit to master: https://git.io/vpvfC [16:34] scummvm/master 284f720 David Fioramonti: MOHAWK: MYST: Fix Observatory displayed hour from 0:XX to 12:XX... [16:34] GitHub162 (GitHub162@gateway/service/github.com/x-taiiqfgilwaqtmxc) left #scummvm. [16:34] GitHub198 (GitHub198@gateway/service/github.com/x-mvucnxxhtuupxbjq) joined #scummvm. [16:34] [scummvm] bgK closed pull request #1159: MOHAWK: MYST: Fix Observatory displayed hour from 0:XX to 12:XX (master...fix10486) https://git.io/vperd [16:34] GitHub198 (GitHub198@gateway/service/github.com/x-mvucnxxhtuupxbjq) left #scummvm. [16:34] error: request for member 'runModal' in 'dialog', which is of non-class type 'GUI::Dialog()' [16:34] hrm [16:35] DrMcCoy (~drmccoy@xoreos/drmccoy) joined #scummvm. [16:35] #scummvm: mode change '+o DrMcCoy' by ChanServ!ChanServ@services. [16:35] So you have GUI::UnknownGameDialog dialog; on one line [16:35] And dialog.runModal(); on the next? [16:36] yep [16:37] stupid me [16:37] I used GUI::UnknownGameDialog dialog(); instead [16:37] and now C:\msys64\home\lotharsm\scummvm/engines/advancedDetector.cpp:361: undefined reference to `GUI::UnknownGameDialog::UnknownGameDialog()' is back :D [16:39] This error looks like the unkown-game-dialog.o file is not included in the link. [16:39] I see you added it to gui/module.mk though [16:40] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) left irc: Ping timeout: 260 seconds [16:40] make clean? [16:43] I wonder if it might be related to the order in which the libengine.a and libgui.a libraries are linked. [16:46] rootfather: could you move you unknown-game-dialog files to engines/ instead of gui/? [16:52] lePhilousophe make clean and a ccache clean didn't work [16:52] criezy, I'll try that [16:55] simply moving the files didn't work, trying without ccache now [16:57] You also removed it from gui/module.mk and added it to engines/module.mk? (just making sure you did not forget that) [16:57] yep [17:02] engines/libengines.a(unknown-game-dialog.o):unknown-game-dialog.cpp:(.rdata$_ZTVN3GUI17UnknownGameDialogE[_ZTVN3GUI17UnknownGameDialogE]+0x20): undefined reference to `GUI::UnknownGameDialog::handleCommand(GUI::CommandSender*, unsigned int, unsigned int)' [17:02] engines/libengines.a(unknown-game-dialog.o):unknown-game-dialog.cpp:(.rdata$_ZTVN3GUI17UnknownGameDialogE[_ZTVN3GUI17UnknownGameDialogE]+0xb0): undefined reference to `GUI::UnknownGameDialog::open()' [17:02] engines/libengines.a(unknown-game-dialog.o):unknown-game-dialog.cpp:(.rdata$_ZTVN3GUI17UnknownGameDialogE[_ZTVN3GUI17UnknownGameDialogE]+0xb8): undefined reference to `GUI::UnknownGameDialog::close()' [17:03] dreammaster (~dreammast@c-73-149-116-247.hsd1.ma.comcast.net) joined #scummvm. [17:03] #scummvm: mode change '+o dreammaster' by ChanServ!ChanServ@services. [17:04] ccawley2011 (8bdec910@gateway/web/freenode/ip.139.222.201.16) left irc: Ping timeout: 260 seconds [17:05] rootfather: Ahah! That is because you declared the handleCommand function in UnknownGameDialog, but have not added it to the cpp yet. [17:05] LittleToonCat (~littlecat@142.162.196.136) joined #scummvm. [17:07] thanks, good catch, that was it [17:08] ccawley2011 (8bdec910@gateway/web/freenode/ip.139.222.201.16) joined #scummvm. [17:08] erm... now it compiles and runs, but when it should display the dialog, it quits... [17:08] Could not load widget position for 'UnknownGame.Title' is in the log file [17:09] Right. I suspected you would get things like that. [17:09] Basically you have two options: specify the layout of a dialog and widget position in the theme file, or hardcode them in the c++ code. [17:10] have fun editing that layout file ;) [17:10] Here by specifying a name "UnknownGame.Title" you tell it to look into the theme file, but it is not there. [17:10] files, there are two themes [17:10] Actually for a simple dialog like that I don't think we need to add it to the theme file. [17:10] ah ok, nevermind then. [17:11] then I'll choose the hardcoded way. [17:11] Look for example at gui/updates-dialog.cpp for a dialog that doesn't use the theme. [17:13] It still use some values from the theme, for example for the button height. But otherwise the position and width of widgets is computed in there. [17:16] wohoo, the first totally empty dialog appeared right when it should :D [17:17] Congrats! :-) [17:17] Now you "just" have to add some content! [17:19] next big thing is making the advanceDetector passing the reported data to the new dialog function :D [17:20] ccawley2011 (8bdec910@gateway/web/freenode/ip.139.222.201.16) left irc: Quit: Page closed [17:21] The easiest might be to generate the report messages (both English and translated) in advanceDetector (where it does it now for the translated one) and pass those to the dialog. [17:23] ajax16384 (~User@109.60.130.33) left irc: Read error: Connection reset by peer [17:50] LubomirR (~LubomirR_@bband-dyn186.178-40-116.t-com.sk) left irc: Quit: Leaving [17:57] criezy, can you tell me why https://github.com/rootfather/scummvm/commit/e4891dea63b3009fd6d65a748341d3037ddd5bd8 fails with [17:57] C++ engines/advancedDetector.o [17:58] engines/advancedDetector.cpp: In member function 'void AdvancedMetaEngine::reportUnknown(const Common::FSNode&, const ADFilePropertiesMap&, const ADGameIdList&) const': [17:58] engines/advancedDetector.cpp:360:25: error: no matching function for call to 'GUI::UnknownGameDialog::UnknownGameDialog()' [17:58] GUI::UnknownGameDialog dialog; [17:58] GitHub133 (GitHub133@gateway/service/github.com/x-xdhmxyohytbcocmc) joined #scummvm. [17:58] [scummvm] criezy pushed 35 new commits to master: https://git.io/vpvkn [17:58] scummvm/master 80be48c Joseph-Eugene Winzer: SUPERNOVA: Removes redundant keypress check... [17:58] scummvm/master 7dab7b7 Joseph-Eugene Winzer: SUPERNOVA: Renames wait2() to wait() [17:58] scummvm/master 9689dff Joseph-Eugene Winzer: SUPERNOVA: Removes mouseInput3()... [17:58] GitHub133 (GitHub133@gateway/service/github.com/x-xdhmxyohytbcocmc) left #scummvm. [17:58] GitHub165 (GitHub165@gateway/service/github.com/x-lzopelacgbzarjtm) joined #scummvm. [17:58] [scummvm] criezy pushed 1 new commit to master: https://git.io/vpvkC [17:58] scummvm/master 91a1bbd Thierry Crozat: I18N: Update translations templates [17:58] GitHub165 (GitHub165@gateway/service/github.com/x-lzopelacgbzarjtm) left #scummvm. [17:59] Pass the "THis is the demo report data." string when you create the dialog, not when you call runModal() [17:59] So GUI::UnknownGameDialog dialog("This is the demo report data."); [18:00] dialog.runModal(); [18:02] GitHub144 (GitHub144@gateway/service/github.com/x-qkmgjwxzhwnwfgjn) joined #scummvm. [18:02] [scummvm] criezy closed pull request #1151: [WIP] SUPERNOVA: Refactoring of the engine (master...merge_nova) https://git.io/vxE3E [18:02] GitHub144 (GitHub144@gateway/service/github.com/x-qkmgjwxzhwnwfgjn) left #scummvm. [18:06] rootfather: In case you did not see my reply, because I forgot to ping you, see 6 minutes ago. [18:10] working now :D [18:10] thanks for your help, I really appreciate that [18:24] assuming we have two Common::Strings, is something like the following possible? [18:24] report & reportTranslated += "\n"; [18:24] basically add tne "\n" to both strings without having to do this separately [18:27] otherwise, having the original and translated message generated in advancedDetector might lead to quite some additional code... [18:33] ny00123 (~ny00123@141.226.11.177) left irc: Read error: Connection reset by peer [18:34] ny00123 (~ny00123@141.226.11.177) joined #scummvm. [18:35] Unfortunately it is not possible. So you will indeed have some dupplicate code. [18:37] Note however that everything after the "Matched game IDs:" is not translated and would be the same for both message. So you could build that part in a separate Common::String and then add it to both the translated and English message. [18:46] TMM (~Hein-Piet@92.110.172.3) joined #scummvm. [18:46] TMM (~Hein-Piet@92.110.172.3) left irc: Changing host [18:46] TMM (~Hein-Piet@fsf/member/pdpc.professional.tmm) joined #scummvm. [18:46] #scummvm: mode change '+o TMM' by ChanServ!ChanServ@services. [18:49] ccawley2011 (56a4f462@gateway/web/freenode/ip.86.164.244.98) joined #scummvm. [18:50] yep that's my current approach [18:50] hm, shouldn't OSystem_SDL::setTextInClipboard("Demo"); work? [18:50] I get engines/unknown-game-dialog.cpp:85:44: error: cannot call member function 'virtual bool OSystem_SDL::setTextInClipboard(const Common::String&)' without object [19:00] Yep, you need to call it on the OSystem instance: g_system->setTextInClipboard("Demo"); [19:02] Only functions declared as static can be called without such an instantiated object. [19:08] that makes sense [19:09] enough for today, I think I can finish it tomorrow [19:09] at least the basic functinallity without the URL stuff [19:10] assuming I can figure out how to make the buttons magically appear after having them declared in the .cpp file :/ [19:10] see you then! [19:16] criezy, maybe you can tell me what the "Copy to clipboard" button doesn't appear in the dialog, only the plain text as intended? [19:16] https://github.com/scummvm/scummvm/compare/master...rootfather:unknown-game-dialog [19:24] rootfather: I don't know. You might want to check the values of _h, _w, buttonPos, buttonWidth and buttonHeight to make sure the button is not draw somewhere outside the dialog. [19:35] Joefish (~Joefish@p200300764F01DC00F0EC8163AB8861DD.dip0.t-ipconnect.de) joined #scummvm. [19:35] #scummvm: mode change '+v Joefish' by ChanServ!ChanServ@services. [19:44] the button is also not drawn if I manually set it to 10, 10... [19:45] OK. I am fetching your code right now and I will see if I can figure it out. [19:48] thanks ;) [19:48] see you tomorrow [19:48] Nick change: rootfather -> rootfather|afk [19:55] rootfather|afk: I found out what the issue is. I will add comments to your commit on GitHub... [20:00] linux_dr (uid130126@gateway/web/irccloud.com/x-btabnthkwisiuwyt) joined #scummvm. [20:11] criezy (~criezy@host86-133-88-154.range86-133.btcentralplus.com) left irc: Quit: criezy [20:12] GitHub12 (GitHub12@gateway/service/github.com/x-emfkhqplakjsmmtm) joined #scummvm. [20:12] [scummvm] yinsimei pushed 5 new commits to master: https://git.io/vpvOI [20:12] scummvm/master 756a109 Simei Yin: SLUDGE: Remove unused global variable captureAllKeys [20:12] scummvm/master 7c74e81 Simei Yin: SLUDGE: Objectify PeopleManager [20:12] scummvm/master 53c79fd Simei Yin: SLUDGE: Refactor PersonaAnimation related code in PeopleManager [20:12] GitHub12 (GitHub12@gateway/service/github.com/x-emfkhqplakjsmmtm) left #scummvm. [20:15] noobineer (~noobineer@c-68-55-184-193.hsd1.mi.comcast.net) joined #scummvm. [21:14] stroggoff (~Praetoria@ppp-94-67-175-131.home.otenet.gr) left irc: Quit: Leaving [21:21] girafe (~girafe@LFbn-LYO-1-449-251.w2-7.abo.wanadoo.fr) left irc: Quit: Leaving [21:26] waltervn (~waltervn@541B2DBA.cm-5-4a.dynamic.ziggo.nl) left irc: Quit: Leaving [21:51] stroggoff (~Praetoria@ppp-94-67-175-131.home.otenet.gr) joined #scummvm. [21:56] GitHub155 (GitHub155@gateway/service/github.com/x-vlllsoobvcmglnpq) joined #scummvm. [21:56] [scummvm] sev- pushed 1 new commit to master: https://git.io/vpvZE [21:56] scummvm/master 0e02da6 Eugene Sandulenko: JANITORIAL: Fix code formatting [21:56] GitHub155 (GitHub155@gateway/service/github.com/x-vlllsoobvcmglnpq) left #scummvm. [22:33] _sev_ (~sev@scummvm/undead/sev) left irc: Ping timeout: 268 seconds [22:56] ny00123 (~ny00123@141.226.11.177) left irc: Quit: Leaving [23:55] Joefish (~Joefish@p200300764F01DC00F0EC8163AB8861DD.dip0.t-ipconnect.de) left irc: Ping timeout: 255 seconds [00:00] --- Mon Apr 16 2018