IRC logs for #openttd on OFTC at 2020-06-23
⏴ go to previous day
00:49:41 *** Feuersalamander has joined #openttd
01:12:38 *** Wormnest has joined #openttd
01:28:29 *** Feuersalamander is now known as grossing
03:34:54 *** Wormnest has joined #openttd
04:59:33 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8224: Cleanup: Remove redundant implementation of TakeScreenshot https://git.io/Jfxcf
05:02:02 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8224: Cleanup: Remove redundant implementation of TakeScreenshot https://git.io/JfxcI
07:58:41 *** andythenorth has joined #openttd
08:05:47 *** sla_ro|master has joined #openttd
09:30:15 *** WormnestAndroid has quit IRC
09:30:37 *** WormnestAndroid has joined #openttd
09:45:42 *** Progman has joined #openttd
10:36:01 *** gelignite has joined #openttd
13:08:45 *** zwamkat has joined #openttd
13:55:45 *** snail_UES_ has joined #openttd
14:22:57 *** arikover has joined #openttd
15:08:51 <supermop_Home> why did i go running when it is 80 out and 73% humidity right before a meeting
15:21:25 *** zwamkat is now known as there_is_no_zwamkat
15:31:15 <Flygon> I question HOW you can exercise in 80c.
15:31:42 <LordAro> Flygon: freedom units
15:45:04 *** there_is_no_zwamkat is now known as zwamkat
16:20:06 <FLHerne> It's actually 80 Kelvin
16:20:31 <FLHerne> Which is a problem, because giant puffy jackets aren't formal meeting attire
16:21:09 <DorpsGek_III> [OpenTTD/OpenTTD] abmyii opened pull request #8235: Fix for 'Huge screenshot' warning being shown incorrectly https://git.io/JfxQt
16:23:17 <DorpsGek_III> [OpenTTD/OpenTTD] James103 commented on pull request #8235: Fix for 'Huge screenshot' warning being shown incorrectly https://git.io/JfxQc
16:26:16 <supermop_Home> impressive humidity for 80 kelvin
16:30:39 <DorpsGek_III> [OpenTTD/OpenTTD] abmyii commented on pull request #8235: Fix for 'Huge screenshot' warning being shown incorrectly https://git.io/JfxQM
16:32:47 <DorpsGek_III> [OpenTTD/OpenTTD] abmyii updated pull request #8235: Fix for 'Huge screenshot' warning being shown incorrectly https://git.io/JfxQt
17:18:10 <dP> why Backup<> always require .Restore() instead of just doing the job as raii pattern?
17:19:29 <dP> it actually even does the job but whines in a console for some reason
17:20:28 <nielsm> hm yesterday when I looked at ther code it seemed SetPIP() would work on NWID_HORIZONTAL and NWID_VERTICAL containers, but apparently it doesn't
17:27:16 <dP> comment is hilarious: "We cannot assert here, as missing restoration is 'normal' when exceptions are thrown."
17:27:30 <dP> it's our intended use but we're not happy about it so let's just spam console
17:31:25 <supermop_Home> FLHerne my formal meeting attire is mostly just my underwear
17:34:32 *** Wormnest has joined #openttd
17:34:50 <LordAro> dP: explicit is better than implicit?
17:35:25 <LordAro> it'll predate C++, and could well have just be "converted" without significantly thinking about it / before RAII became popular
17:36:19 <dP> LordAro, explicit to a point of spamming comments to the console?
17:37:30 <LordAro> spamming the console does seem unnecessary
17:38:55 <dP> also forces a bunch of .restores everywhere throughout the code
17:40:19 <nielsm> the question is, is there ever any time you would _not_ want to call Restore()?
17:40:29 <nielsm> when is it correct to not call restore?
17:40:45 <dP> nielsm, it calls it anyway
17:40:46 <nielsm> if never, then don't bonk the programmer for making a "mistake" the computer could fix
17:41:01 <nielsm> "you forgot to say the magic word"
17:41:23 <LordAro> i'd rather some sort of "context manager" for it, rather than RAII i think
17:41:44 <LordAro> otherwise you'd just end up with the inverse problem - the backup would be "left" too long and would affect unintended things
17:41:46 <dP> LordAro, raii is a context manager in c++
17:41:58 <LordAro> i mean an explicit one
17:42:12 <LordAro> OrderBackup([&](){ ... });
17:42:42 <LordAro> rather than just OrderBackup foo; do stuff; ..?? ..
17:43:01 <dP> LordAro, if only c++ lambdas weren't that awful
17:43:13 <dP> LordAro, also you can just {Backup<>; ... }
17:43:25 <LordAro> true, but syntax is just syntax
17:43:35 <LordAro> dP: that's my point, the {} would inevitably be forgotten
17:43:51 <dP> LordAro, Backup can also be forgotten :p
17:43:51 <LordAro> it's just as manual as having to .Restore(), but is even less clear what's happening
17:44:14 <LordAro> the bugs would be even more subtle
17:58:19 <dP> was googling for raii usage examples until I realized that it's pointless talking best practices when imlementing a thing whose only purpose is handling freaking globals
18:06:03 *** HerzogDeXtEr has joined #openttd
18:10:52 <LordAro> you must be new to reddit
18:29:10 <nielsm> hm I wonder why I didn't also apply ScaleGUITrad to PIP spacing back when I first did that "fix scaling" patch
18:29:32 <nielsm> because I distinctly remember considering, probably trying it, then rejecting it
18:47:35 *** supermop_Home has joined #openttd
19:12:46 <FLHerne> nielsm: In your 'after' screenshots of the tree GUI, the tree sprites overhang the edges of their buttons in an unpleasant way?
19:13:00 <FLHerne> Smarter layout is nice though :-)
19:17:00 <nielsm> look at the window titles above
19:17:04 <nielsm> the right one is before
19:18:25 *** Speeder has joined #openttd
19:19:45 <nielsm> actually I'm fighting some more sizing problems with it
19:41:12 *** frosch123 has joined #openttd
19:52:49 *** b_jonas has joined #openttd
20:52:50 <Wolf01> NAS died :( Luckily I had a backup of photos and documents
21:09:47 <frosch123> i only hear about nas when they die
21:09:55 * andythenorth should make dinner
21:10:38 <frosch123> it takes some time to make beer
21:12:03 <Wolf01> Or you can get instant beer
21:25:40 <andythenorth> I got instant beer
21:25:47 <andythenorth> it comes in a bottle
21:29:06 <frosch123> can you 3d-print it?
21:36:17 <nielsm> hmm, I wonder if I could abuse the autorail tool graphics for tree painting
21:54:53 <DorpsGek_III> [OpenTTD/OpenTTD] abmyii opened pull request #8236: Fix #7772: Show destination on mouseover when vehicle stopped (and not in depot) https://git.io/JfpUS
22:21:32 *** virtualrandomnumber has joined #openttd
22:29:56 *** gelignite has joined #openttd
22:31:47 <dP> is it possible to make orders that change destination mid-route?
22:32:10 <dP> e.g. head to one station for 10 days and turn to another after that
22:34:52 <nielsm> not in vanilla at least
22:35:14 <nielsm> maybe it'd be possible to have a conditional order that tests days since departure
22:35:42 <nielsm> you'd still have to pass through a waypoints the train can route towards before it tests the conditional order
22:36:47 <dP> what happens if conditional order is timetabled?
22:38:07 <nielsm> don't think you can do that
22:38:27 <dP> I can but no idea what it does
22:38:55 <b_jonas> dP: no, but you can use a go via order usually
22:39:11 <b_jonas> to give an intermediate station where the train need not stop
22:39:17 <b_jonas> make it go non-stop via
22:39:19 <dP> well, I kinda wanted it for airplanes...
22:39:42 <b_jonas> dP: did someone lobby for a no-fly zone?
23:07:08 *** virtualrandomnumber has quit IRC
23:31:08 <andythenorth> is today the day to learn GS?
23:32:12 <FLHerne> No, today is the day to spend ages trying to bisect a failing test, only to find that the Python devs made backwards-incompatible API changes in a patch release
23:34:22 <FLHerne> To be fair, the previous behaviour was obviously broken
23:35:25 <FLHerne> But we had a workaround, which goes horribly wrong if the behaviour stops being obviously broken :p
23:39:32 <dP> I tried to refactor town growth code but gave up, that shit's hopeless...
23:42:34 <dP> andythenorth, that shit's even more hopeless :p
continue to next day ⏵
