[00:05] delacroix (~delacroix@2a02:810c:640:e00:48bc:bfff:feb7:af31) left irc: Quit: ZNC - http://znc.in [00:05] delacroix (~delacroix@ip5b43a313.dynamic.kabel-deutschland.de) joined #scummvm. [00:06] [scummvm] athrxx pushed 1 new commits to master: https://git.io/JvMfg [00:06] scummvm/master 5991e8c athrxx: AUDIO: fix compiler warning [00:11] Tea (~tea@fsf/member/tea) left irc: Quit: Quit messages are a thing? [00:13] Tea (~tea@mail.warhaggis.com) joined #scummvm. [00:13] Tea (~tea@mail.warhaggis.com) left irc: Changing host [00:13] Tea (~tea@fsf/member/tea) joined #scummvm. [00:17] Tea (~tea@fsf/member/tea) left irc: Client Quit [00:18] Tea (~tea@mail.warhaggis.com) joined #scummvm. [00:18] Tea (~tea@mail.warhaggis.com) left irc: Changing host [00:18] Tea (~tea@fsf/member/tea) joined #scummvm. [00:42] dreammaster (~dreammast@c-73-241-66-252.hsd1.ca.comcast.net) joined #scummvm. [00:42] #scummvm: mode change '+o dreammaster' by ChanServ!ChanServ@services. [00:50] _sev (~sev@scummvm/undead/sev) left irc: Read error: Connection reset by peer [00:50] _sev_ (~sev@scummvm/undead/sev) joined #scummvm. [00:50] #scummvm: mode change '+o _sev_' by ChanServ!ChanServ@services. [02:18] SylvainTV (~Sylvain@lfbn-lil-1-811-58.w92-148.abo.wanadoo.fr) left irc: Read error: Connection reset by peer [03:25] _dreammaster (~dreammast@c-73-241-66-252.hsd1.ca.comcast.net) joined #scummvm. [03:25] #scummvm: mode change '+o _dreammaster' by ChanServ!ChanServ@services. [03:27] dreammaster (~dreammast@c-73-241-66-252.hsd1.ca.comcast.net) left irc: Ping timeout: 250 seconds [03:58] Nick change: _dreammaster -> dreammaster [04:01] DominusExult (~dominus@port-92-196-112-116.dynamic.as20676.net) joined #scummvm. [04:01] DominusExult (~dominus@port-92-196-112-116.dynamic.as20676.net) left irc: Changing host [04:01] DominusExult (~dominus@unaffiliated/dominus) joined #scummvm. [04:04] Dominus (~dominus@unaffiliated/dominus) left irc: Ping timeout: 264 seconds [04:04] Nick change: DominusExult -> Dominus [04:12] P2E_ (~tgz@209.251.45.7) joined #scummvm. [04:14] P2E (~tgz@209.251.45.7) left irc: Ping timeout: 256 seconds [04:56] dreammaster (~dreammast@c-73-241-66-252.hsd1.ca.comcast.net) left irc: [06:09] BeefEats (~BeefEats@unaffiliated/beefeats) left irc: Read error: Connection reset by peer [06:58] systwi_ (~systwi@173-20-216-139.client.mchsi.com) left irc: Read error: Connection reset by peer [06:58] systwi (~systwi@173-20-216-139.client.mchsi.com) joined #scummvm. [07:24] ny00123 (~ny00123@188.120.132.160) joined #scummvm. [07:40] waltervn (~waltervn@scummvm/undead/waltervn) joined #scummvm. [07:40] #scummvm: mode change '+o waltervn' by ChanServ!ChanServ@services. [08:05] Mia (~Mia@unaffiliated/mia) left irc: Read error: Connection reset by peer [08:10] [scummvm] mduggan opened pull request #2153: ULTIMA8: Add short fade in/out around menu items to match original game (master...u8_add_menu_fades) https://git.io/JvMsu [08:36] Mia (~Mia@176.43.46.76) joined #scummvm. [08:36] Mia (~Mia@176.43.46.76) left irc: Changing host [08:36] Mia (~Mia@unaffiliated/mia) joined #scummvm. [08:41] Lightkey (~Darklock@p200300F693C6494522CF30FFFE083718.dip0.t-ipconnect.de) left irc: Ping timeout: 256 seconds [08:53] Lightkey (~Darklock@p200300F693C6499922CF30FFFE083718.dip0.t-ipconnect.de) joined #scummvm. [08:57] @_athrxx , that's my mistake. I'm sorry. Will send later PR to fix that. Thanks for noting that. [09:10] timofonic (~timofonic@unaffiliated/timofonic) left irc: Remote host closed the connection [09:14] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [09:14] timofonic (~timofonic@unaffiliated/timofonic) left irc: Read error: Connection reset by peer [09:14] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [09:14] timofonic (~timofonic@unaffiliated/timofonic) left irc: Read error: Connection reset by peer [09:15] waltervn_ (~waltervn@scummvm/undead/waltervn) joined #scummvm. [09:15] #scummvm: mode change '+o waltervn_' by ChanServ!ChanServ@services. [09:18] waltervn (~waltervn@scummvm/undead/waltervn) left irc: Ping timeout: 246 seconds [09:26] @_athrxx , now I see that you already fixed that. Thanks... [09:29] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [09:35] Nick change: waltervn_ -> waltervn [09:40] ldevulder_ (~ldevulder@176.167.255.121) joined #scummvm. [09:44] ldevulder (~ldevulder@176.164.182.131) left irc: Ping timeout: 250 seconds [10:13] [scummvm] aryanrawlani28 opened pull request #2154: Fix bug #7125 (master...master) https://git.io/JvMcv [10:22] any chance of my pull request being accepted despite codacy complaining about my use of seemingly unused variables? [11:37] antlarr2 (~quassel@212.red-83-32-80.dynamicip.rima-tde.net) left irc: Quit: bye! [11:38] antlarr (~quassel@212.red-83-32-80.dynamicip.rima-tde.net) joined #scummvm. [11:45] vliaskov (~vliaskov@x4db79c9e.dyn.telefonica.de) joined #scummvm. [12:08] SylvainTV (~Sylvain@lfbn-lil-1-811-58.w92-148.abo.wanadoo.fr) joined #scummvm. [12:08] #scummvm: mode change '+o SylvainTV' by ChanServ!ChanServ@services. [12:15] SylvainTV (~Sylvain@lfbn-lil-1-811-58.w92-148.abo.wanadoo.fr) left irc: Ping timeout: 256 seconds [12:16] [scummvm] yuv422 pushed 1 new commits to master: https://git.io/JvMle [12:16] scummvm/master 21778dc yuv422: DRAGONS: Added power meters to thumb wrestling mini game [12:17] yuv422 (~yuv422@119-18-3-11.771203.syd.nbn.aussiebb.net) left irc: Remote host closed the connection [12:18] yuv422 (~yuv422@119-18-3-11.771203.syd.nbn.aussiebb.net) joined #scummvm. [12:18] #scummvm: mode change '+o yuv422' by ChanServ!ChanServ@services. [12:20] <_athrxx> @Henke37 Yes. Satisfying pointless warnings of static check tools isn't required (or even recommended). You should just check these warnings to see whether they are valid or not. Which you apparently have done... [12:39] timofonic (~timofonic@unaffiliated/timofonic) left irc: Remote host closed the connection [12:39] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [12:39] timofonic (~timofonic@unaffiliated/timofonic) left irc: Read error: Connection reset by peer [12:45] mataniko_ (~mataniko@unaffiliated/mataniko) joined #scummvm. [12:45] #scummvm: mode change '+o mataniko_' by ChanServ!ChanServ@services. [12:47] mataniko (~mataniko@unaffiliated/mataniko) left irc: Ping timeout: 256 seconds [12:54] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [12:56] timofonic (~timofonic@unaffiliated/timofonic) left irc: Read error: Connection reset by peer [13:14] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [14:00] we do not pay way too much attention to the static analysers [14:00] i just wish i knew some magic incantation to make it shut up [14:00] especially I am annoyed by hundreds of "tainted variable" of Coverity: yes, I know I'm not checking data range when reading [14:01] and all that nonsense of unsafe strlen() by codacity [14:03] <_athrxx> @Henke37 Well, you could make a PR for codacy 🙂 [14:03] ah, recently Coverity started to complain a lot about our String class [14:04] but unfortunately, my C++ knowledge does not allow me to understand its validity [14:04] i try to keep the number of project i'm involved in to a minimum [14:04] any volunteers with deep C++ knowledge? 😄 [14:05] <_athrxx> what does it say about the String class? (and no, I don't qualify as a C++ expert) [14:06] let me explain [14:07] i dabble too. and i've got time [14:10] okay, I found a small example: line 229: https://github.com/scummvm/scummvm/blob/master/engines/pink/gui.cpp#L229 [14:10] prefix and suffix are String class [14:10] then we do: prefix + suffix [14:11] not seeing anything strange here. [14:12] <_athrxx> and what is the coverity complaint? [14:12] which eventually executes this: https://github.com/scummvm/scummvm/blob/master/common/str.cpp#L297 [14:13] return operator+=(String(str)); [14:13] create a copy of the current string, nothing strange there. [14:13] now, for the construct String(str) we call reference copying constructor [14:14] here: https://github.com/scummvm/scummvm/blob/master/common/str.cpp#L114 [14:14] as a result [14:14] Coverity complains about double free of suffix [14:15] if you have access to Coverity, here is is the link: https://scan6.coverity.com/reports.htm#v45941/p10140/fileInstanceId=98309893&defectInstanceId=29696439&mergedDefectId=1420333 [14:15] and it shows dozens of such places. E.g. [14:15] String a("a"); [14:15] String b; [14:15] b = a; [14:16] in this case it will be copy by reference [14:16] that's operator +, not operator += [14:16] and seemingly (according to Coverity), b will be freed 2 times [14:17] this is the actual code that runs: https://github.com/scummvm/scummvm/blob/master/common/str.cpp#L900 [14:17] sigh [14:18] <_athrxx> okay, this needs some thought. No, I can't access coverity. ("Your credentials are not valid for this resource") [14:18] here is same code issue with +=: https://github.com/scummvm/scummvm/blob/master/engines/director/lingo/lingo-code.cpp#L567 [14:19] Coverity says: [14:19] CID 1412183 (#1 of 1): Double free (USE_AFTER_FREE) 2. double_free: Calling ~String frees pointer d2.u.s->_str which has already been freed. [14:19] because it says: [14:19] 1. freed_arg: operator += frees d2.u.s._str [14:21] here is issue with `+': https://github.com/scummvm/scummvm/blob/master/gui/ThemeEval.cpp#L160 [14:23] whenever constructor String::String(const String &str) is involved, there seems to be a problem [14:23] because instead of reallocating memory, it makes: _str = str._str;, e.g. copies the pointer [14:25] ah, I see it now [14:25] ugh, this is stupid [14:26] guys, no need to look [14:26] please tell us, because i don't see the issue [14:26] ~Stringdoes not free [14:26] it calls decRefCount() [14:26] and that thing counts references [14:27] https://github.com/scummvm/scummvm/blob/master/common/str.cpp#L216 Pretty simple stuff once you figure out that the reference counter itself is lazily allocated. [14:28] see line 111 [14:28] it increases the ref counter right before that pointer copy [14:28] so, false positive [14:29] yup. [14:29] I'll go through and tame the Coverity [14:31] line 282 is possible UB tho. not legal to compare pointers like that if they aren't from the same array. [14:32] i mean, it works on all implementations I know of, but the standard says it's UB [14:35] well, this is specifically check about str being from the same array [14:35] formally you need to cast to something like intptr_t (which is IB) and do an integer comparison. [14:35] i know what it does. i've written similar code once. [14:36] timofonic (~timofonic@unaffiliated/timofonic) left irc: Ping timeout: 256 seconds [14:37] timofonic (~timofonic@unaffiliated/timofonic) joined #scummvm. [14:38] timofonic (~timofonic@unaffiliated/timofonic) left irc: Remote host closed the connection [14:50] _sev_: for me the String errors reported by Coverity is bogus [14:50] but I would like someone to take a second look on it too [14:51] I suppose we need to put some coverity tag in str.cpp but I don't know which one [14:51] ok so you did second look :) cool [14:58] [scummvm] henke37 opened pull request #2155: COMMON: Dedicated function for in buffer check (master...strPtrCmpUBFix) https://git.io/JvM0A [15:05] speaking of strings, is this correct? https://github.com/scummvm/scummvm/pull/2151/files#diff-269b333a6e284c657a1b22445014839eR52 [15:05] i get lost when mixing between char* and string and [15:08] _sev_ (~sev@scummvm/undead/sev) left irc: Quit: This computer has gone to sleep [15:09] i do not miss figuring out what is or isn't a manually allocated string [15:46] *grumble grumble* another time i want to use a C++11 feature, but couldn't. having to switch to a C header for the same result is not fun. [15:51] _sev_ (~sev@scummvm/undead/sev) joined #scummvm. [15:51] #scummvm: mode change '+o _sev_' by ChanServ!ChanServ@services. [15:54] technically C99 isn't part of C++11 either 😛 [15:55] it's 9 years old. time to get on with the times! [15:57] <_sev_> LePhilousophe: we've cleared it out [15:58] <_sev_> LePhilousophe: thanks for checking! [16:01] [scummvm] SupSuper pushed 1 new commits to master: https://git.io/JvMzg [16:01] scummvm/master 22943ee SupSuper: NETWORKING: Fix delete instead of free [16:17] @Henke37 I have some idea about potentially enabling c++11 [16:17] @Henke37 willing to help? [16:22] i wouldn't know how [16:22] i try to avoid build management stuff [16:23] i think there was a PR for it [16:23] well, it is not about that [16:23] it is about introducing another configure flag, something like --test-c++11 [16:23] and then when this flag is enabled, but into a single file a no-op thing, but which will use most of the interesting c++11 features [16:24] then at the time of release I will ask all the porters to test compiling and running it with this feature [16:24] and we understand which ports will potentially stop working if we enable it for the whole project [16:24] an autoconf flag or something? i don't know autoconf [16:24] I need help with writing that test file [16:25] I can add that configure file, no problem [16:25] we have one example already: the include. [16:25] there should be one class which I will call from main.cpp, and it should do nothing except debug output [16:26] @Henke37 I need .cpp code, not messages on Discord 😄 [16:26] and there is a preprocessor symbol to check the claimed supported c++ version. [16:27] does it mean that you don't have time/desire to help? [16:28] not this exact instant. i'm making dinner [16:59] travis-ci (~travis-ci@ec2-34-235-142-38.compute-1.amazonaws.com) joined #scummvm. [16:59] scummvm/scummvm#12551 (master - 22943ee : SupSuper): The build has errored. [16:59] Change view : https://github.com/scummvm/scummvm/compare/21778dcb8089...22943ee3688e [16:59] Build details : https://travis-ci.org/scummvm/scummvm/builds/664912310 [16:59] travis-ci (travis-ci@ec2-34-235-142-38.compute-1.amazonaws.com) left #scummvm. [17:11] @sev: do you already know which platform could cause problems? [17:14] ajax16384 (~User@109.60.130.33) joined #scummvm. [17:14] #scummvm: mode change '+o ajax16384' by ChanServ!ChanServ@services. [17:15] a complaint I have with Common::String is that it's not thread safe because to the Copy on Write stuff. Interestingly with C++11 the spec for std::string was changed to make CoW implementations non-conformant. Which has the interesting side effect of making std::string more thread safe. I was thinking perhaps we should switch to a non-CoW implementation as well. [17:15] ny00123 (~ny00123@188.120.132.160) left irc: Quit: Leaving [17:15] however I guess we should wait until we switch to C++11 to reduce useless string copies by using move semantics. [17:30] ccawley2011 (~ccawley20@2a00:23c6:341e:8d01:a07a:263a:91d1:883f) joined #scummvm. [17:30] #scummvm: mode change '+o ccawley2011' by ChanServ!ChanServ@services. [17:30] WindlePoons (~WindlePoo@80-110-56-88.cable.dynamic.surfer.at) joined #scummvm. [17:30] WindlePoons (WindlePoo@80-110-56-88.cable.dynamic.surfer.at) left #scummvm. [17:34] BeefEats (~BeefEats@unaffiliated/beefeats) joined #scummvm. [17:36] i've been trying to think of a good c++11 test. but i don't have anything better than this. cpp int main() { return __cplusplus; } [17:37] I believe what sev means is exercising the interesting c++11 features, like using a lambda, range based loop, move semantics, perfect forwarding, .. to check whether all our target platforms can handle them [17:37] the point isn't to test that c++11 exists, but that c++11 works as expected [17:38] and you can't rely on that macro in msvc due to backcompat [17:38] this might help: http://www.glenmccl.com/cpp11.htm [17:39] Nick change: ldevulder_ -> ldevulder [17:59] criezy (~criezy@host31-51-71-219.range31-51.btcentralplus.com) joined #scummvm. [17:59] #scummvm: mode change '+o criezy' by ChanServ!ChanServ@services. [18:14] > I believe what sev means is exercising the interesting c++11 features, like using a lambda, range based loop, move semantics, perfect forwarding, .. to check whether all our target platforms can handle them @Grogbot If a plan is to convert existing codebase to modern cpp, let me know - I'll help as much as I can, i love doing work like this [18:14] travis-ci (~travis-ci@ec2-34-201-108-2.compute-1.amazonaws.com) joined #scummvm. [18:14] SupSuper/scummvm#2 (travis - 357053f : SupSuper): The build has errored. [18:14] Change view : https://github.com/SupSuper/scummvm/compare/00663b85aceb...357053f48612 [18:14] Build details : https://travis-ci.org/SupSuper/scummvm/builds/664958973 [18:14] travis-ci (travis-ci@ec2-34-201-108-2.compute-1.amazonaws.com) left #scummvm. [18:15] travis-ci (~travis-ci@ec2-3-91-18-34.compute-1.amazonaws.com) joined #scummvm. [18:15] SupSuper/scummvm#1 (master - 22943ee : SupSuper): The build has errored. [18:15] Change view : https://github.com/SupSuper/scummvm/compare/21778dcb8089dd2e76520dea1fe2edb1097f4402...22943ee3688eb639528a4878cea65cfac9642599 [18:15] Build details : https://travis-ci.org/SupSuper/scummvm/builds/664958509 [18:15] travis-ci (travis-ci@ec2-3-91-18-34.compute-1.amazonaws.com) left #scummvm. [18:35] whoops sorry about that [18:45] P2E_ (~tgz@209.251.45.7) left irc: Quit: Reconnecting [18:45] P2E (~tgz@209.251.45.7) joined #scummvm. [19:07] Stormkeeper (~kvirc@54.200-30-62.static.virginmediabusiness.co.uk) left irc: Quit: KVIrc 5.0.0 Aria http://www.kvirc.net/ [19:10] Stormkeeper (~kvirc@54.200-30-62.static.virginmediabusiness.co.uk) joined #scummvm. [19:24] ajax16384 (~User@109.60.130.33) left irc: Read error: Connection reset by peer [19:42] SylvainTV (~Sylvain@lfbn-lil-1-811-58.w92-148.abo.wanadoo.fr) joined #scummvm. [19:42] #scummvm: mode change '+o SylvainTV' by ChanServ!ChanServ@services. [20:01] Tomaz^ (~tompsson@h-69-131.A785.priv.bahnhof.se) left irc: Ping timeout: 246 seconds [20:42] @criezy been poking around the mac travis loop and it seems to be related to this: Running ./configure with the last specified parameters seems make keeps trying to reconfigure? might be something off with ccache? [20:52] waltervn (~waltervn@scummvm/undead/waltervn) left irc: Quit: Leaving [20:58] @SupSuper I have zero experience with ccache. Maybe that could be the issue. But I have absolutely no idea. [21:06] Nick change: CommunistWolf -> veggies [21:06] Nick change: veggies -> CommunistWolf [21:21] [scummvm] ccawley2011 opened pull request #2156: GPH/DINGUX: Unify the downscaling code (master...downscale) https://git.io/JvM1r [21:29] Tkachov (~q@84.237.53.159) left irc: Ping timeout: 260 seconds [21:38] yuv422 (~yuv422@119-18-3-11.771203.syd.nbn.aussiebb.net) left irc: Remote host closed the connection [21:38] yuv422 (~yuv422@119-18-3-11.771203.syd.nbn.aussiebb.net) joined #scummvm. [21:38] #scummvm: mode change '+o yuv422' by ChanServ!ChanServ@services. [21:59] cd (~cd@unaffiliated/cd) joined #scummvm. [22:03] hi guys [22:04] Action: yuv422 waves [22:14] ccawley2011 (~ccawley20@2a00:23c6:341e:8d01:a07a:263a:91d1:883f) left irc: Quit: Leaving [22:27] _athrxx (~athrxx@ipbcc00f2d.dynamic.kabel-deutschland.de) joined #scummvm. [22:29] Tkachov (~q@84.237.53.159) joined #scummvm. [22:29] #scummvm: mode change '+o Tkachov' by ChanServ!ChanServ@services. [22:43] vliaskov (~vliaskov@x4db79c9e.dyn.telefonica.de) left irc: Ping timeout: 256 seconds [22:43] _athrxx (~athrxx@ipbcc00f2d.dynamic.kabel-deutschland.de) left irc: Quit: Leaving [23:22] NULUSIOS1 (~nls@mail2.termoil.com) joined #scummvm. [23:27] NULUSIOS1 (nls@mail2.termoil.com) left #scummvm. [23:44] yuv422 (~yuv422@119-18-3-11.771203.syd.nbn.aussiebb.net) left irc: Remote host closed the connection [23:44] yuv422 (~yuv422@119-18-3-11.771203.syd.nbn.aussiebb.net) joined #scummvm. [23:44] #scummvm: mode change '+o yuv422' by ChanServ!ChanServ@services. [00:00] --- Sat Mar 21 2020