IRC logs for #openttd on OFTC at 2021-06-10
⏴ go to previous day
00:14:08 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:16:12 *** WormnestAndroid has joined #openttd
00:21:07 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:23:01 *** WormnestAndroid has joined #openttd
00:40:04 *** Beer has quit IRC (Quit: Leaving)
00:58:50 *** tokai|noir has joined #openttd
00:58:50 *** ChanServ sets mode: +v tokai|noir
01:05:35 *** tokai has quit IRC (Ping timeout: 480 seconds)
02:48:25 *** Wormnest has quit IRC (Quit: Leaving)
03:00:00 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:15:09 *** Kitrana has quit IRC (Ping timeout: 480 seconds)
04:04:28 *** ChanServ sets mode: +v tokai
04:11:15 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
05:55:16 *** nielsm has quit IRC (Ping timeout: 480 seconds)
06:08:23 *** HerzogDeXtEr has joined #openttd
06:13:46 *** Kitrana has joined #openttd
06:15:47 *** sla_ro|master has joined #openttd
06:19:11 *** Kitrana has quit IRC (Read error: Connection reset by peer)
06:42:50 <peter1138> Wonder if it's worth force-renewing my LE in case of issues...
06:43:02 <peter1138> LetsEncrypt certs, that is.
07:06:05 <Xaroth> be careful, the new LE certs are 1 second shorter than the old ones.
07:09:49 *** tokai|noir has joined #openttd
07:09:49 *** ChanServ sets mode: +v tokai|noir
07:16:46 *** tokai has quit IRC (Ping timeout: 480 seconds)
07:18:21 *** ChanServ sets mode: +v tokai
07:19:27 <TrueBrain> for some reason this completely fell off the radar yesterday .. and nobody reminded me :(
07:21:40 <TrueBrain> now I need a quick review :D
07:24:10 <TrueBrain> LordAro: if you can spare a few, would appreciate it :)
07:25:14 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
07:28:48 <Elouin> Is there a git repo or something with the translations of the description text on gog?
07:29:20 <Elouin> The German Version has some mistakes in it.
07:29:48 <TrueBrain> be mindful, "mistakes" is often an opinion for German these days :)
07:30:05 <TrueBrain> (there are 10+ people translating strings to German :P)
07:30:38 <Elouin> i only see the steam store page in there.
07:30:51 <TrueBrain> owh, it is not renamed yet
07:30:55 <TrueBrain> it should be called "App Store Pages"
07:32:46 <TrueBrain> tnx Xaroth , sadly that is not enough for me to merge :P
07:48:44 <TrueBrain> its warm enough for it ..
07:58:01 <peter1138> OMG. I've just noticed that gitea's wiki is backed by git already.
07:58:48 <TrueBrain> and I just noticed I cannot even write news posts
07:59:03 <peter1138> Oh I did a crap review, oops.
07:59:22 <DorpsGek> [OpenTTD/website] TrueBrain opened pull request #215: Fix: forgot the file-extension with last commit https://git.io/JZlq4
07:59:30 <TrueBrain> nothing you would really notice
07:59:49 <peter1138> (And the gitea wiki files are .md too :D)
08:00:05 <DorpsGek> [OpenTTD/website] PeterN approved pull request #215: Fix: forgot the file-extension with last commit https://git.io/JZlq7
08:00:33 <TrueBrain> I approve of their approach :)
08:00:48 <TrueBrain> sadly, our files are still mediatext
08:00:57 <TrueBrain> but converting mediatext to markdown .. yeah, not it
08:01:05 <peter1138> Yes, I was hesitant to use the wiki as I thought it might be locked away in a weird format, but now I know it's just git, it's perfect.
08:01:32 <TrueBrain> it is nice if products do the sane thing :D
08:02:42 <peter1138> I don't think I can clone issues though :)
08:03:07 <TrueBrain> why would you want to?
08:04:11 <DorpsGek> [OpenTTD/website] TrueBrain merged pull request #215: Fix: forgot the file-extension with last commit https://git.io/JZlq4
08:04:37 <peter1138> Unlikely, just nice when things are open.
08:05:43 <TrueBrain> that moment you move some code, and the game no longer works
08:05:52 <TrueBrain> means you have to stare at the code for a while to find a weird typo
08:09:28 <peter1138> Urgh, one of my colleagues never puts an email subject on.
08:09:50 <LordAro> "oh i'm sorry, i just assumed it was spam"
08:10:01 <TrueBrain> one of "those" people :D
08:13:27 <TrueBrain> meh, URL is over 2 lines on the frontpage now
08:13:37 <TrueBrain> well, not "the worst"
08:14:59 <V453000> how many days does an average month have?
08:15:32 <Rubidium> median is 31, isn't it?
08:15:39 <TrueBrain> @calc (31 + 28.25 + 31 + 30 + 31 + 30 + 30 + 31 + 30 + 31 + 30 + 31) / 12
08:15:39 <DorpsGek> TrueBrain: 30.354166666666668
08:15:46 <TrueBrain> he asked average, not median :P
08:16:13 <V453000> so if I have a date(1990, 6, 29) for example
08:16:21 <V453000> and want to convert it to some days value
08:16:59 <V453000> it's not so simple, is it
08:17:11 <V453000> because it's not the average, it's just dependent on the current position in the month
08:17:25 <TrueBrain> you talking in general, or in OpenTTD?
08:17:34 <TrueBrain> as OpenTTD internally uses "days since 1-1-0"
08:17:48 <TrueBrain> and a lot of code to get from that to YYYY-MM-DD :)
08:18:21 <TrueBrain> so doesn't NML etc already have something for this?
08:18:28 <TrueBrain> (I honestly don't know, but I would assume so :P)
08:18:42 <V453000> NML has a literally date(1990, 6, 29)
08:18:48 <V453000> now I wonder if I can do date() + date()
08:19:04 <V453000> or only date() + days
08:19:40 <Rubidium> if it were to work, it would probably first convert the date to days from epoch and then add those days together
08:20:48 <Rubidium> so 1-1-2004 + 1-1-1 would become 31-12-2004
08:21:36 <V453000> I don't think I understand that XD
08:22:17 <Timberwolf> Ouch, the epoch is non-zero so the calculations will be a year "off".
08:22:18 <Rubidium> year 0 is not a leap year, 2004 is a leap year
08:22:32 <Timberwolf> If I read that correctly.
08:22:48 <Timberwolf> Oh wait - the leap year thing.
08:22:49 <Rubidium> so 1-1-0 to 1-1-1 is 365 days, 1-1-2004 to 1-1-2005 is 366 days
08:23:27 <V453000> well this did result in 3066, idk about the days date(2066, 1, 1) + date(1000,1,1);
08:24:19 <TrueBrain> adding two dates together doesn't really mean anything honestly .. I would hope NML supports date(Y, M, D) + D
08:24:34 <TrueBrain> okay, posted GOG news on Socials, except tt-forums
08:24:46 <TrueBrain> I guess I should be nice and do that too ..
08:24:51 <Eddi|zuHause> since date() is a macro that returns an integer, that should work
08:25:44 *** andythenorth has joined #openttd
08:25:53 *** andythenorth has quit IRC ()
08:26:02 <Timberwolf> Heh. I'm so used to dealing with ISO style dates I read "1-1-0" as "0th January, 1"
08:26:03 <V453000> yeah, I imagine under the hood both are just converted into integers, however I also guess that I'm always adding 1 month and 1 day extra? :)
08:27:01 <TrueBrain> and I have the first remark how we should have done GOG before Steam :P
08:27:08 <TrueBrain> people gatekeep the weirdest shit
08:27:39 <Eddi|zuHause> honestly, that's the point where you just softban people
08:28:11 <Eddi|zuHause> let them think they posted something, but nobody can read it
08:28:19 <Timberwolf> Discord/Reddit has some weird hangups.
08:29:03 <Timberwolf> I've been finding that most of the time when someone puts up a station design and someone else "corrects" it, the corrected version is somewhere between 20%-50% worse for throughput.
08:29:34 <TrueBrain> okay, posted on tt-forums too now
08:31:31 <peter1138> Game features: Cloud saves, mmm
08:31:55 <TrueBrain> GOG is funny, in a lot of things you do not get a choice :)
08:32:57 <TrueBrain> I still hope someone looks at how/what to enable for Steam Cloud save :) But so far, crickets
08:32:58 <peter1138> Me: use path signals, they much easier
08:33:15 <Eddi|zuHause> cloud saves weird me out... but i do think for many people they are a benefit
08:33:19 <peter1138> Reddit: but you need block/pre signals for prio or logic! absolutely impossible to play without!
08:33:43 <Timberwolf> AFAICT, it's because most of the popular station designs date back to the block signals era, where they were often massively better, and people have ported them to PBS over the years without revisiting what that does to the throughput.
08:34:45 <Eddi|zuHause> no, that's because they make assumptions on how "most people" play the game that just aren't based on reality
08:34:49 <Timberwolf> Hey, I got a clocked exit roro with acceleration tracks to give me the same throughput as a compact balanced PBS roro, it only took about 100 additional tiles and an hour of balancing! It's not that bad a station! :p
08:36:05 <Eddi|zuHause> that's also what always weirded me out about openttdcoop. they all talked like their way is the only way to play the game
08:36:36 <peter1138> I'm not even sure what prios achieve. They seem to specifically designed to delay trains from one path.
08:38:05 <Eddi|zuHause> well, yes. i do get their idea. once a train (on the "main line") is at full speed, it should never ever stop on a red signal
08:38:56 <Timberwolf> These people, with their "identifiable" "main" line.
08:39:23 <Eddi|zuHause> ... is exactly why i never have a use for "priority signals"
08:39:34 * Timberwolf is of the, "oh everything appears to be going via there, not sure why but I guess I'll make it 6 tracks"
08:40:31 <peter1138> Alright, time to find some convincing uses of presignals on the reddit server.
08:40:32 <Eddi|zuHause> i'd rather build more 2-track lines
08:41:01 <peter1138> Rau's Labs. Good candidate, I assume that is the guy that replied to me.
08:42:07 <Eddi|zuHause> in an ideal world, you could achieve the "no stopping on red" with timetables
08:42:33 <V453000> priorities are basically just detections, if you're trying to join 1 line onto 4, you need to somehow detect if the other line is full. Using too long priorities isn't a great idea, but many people do it htat way
08:43:09 <Eddi|zuHause> V453000: i'm pretty sure most people that use them are just cargoculting, without really understanding them
08:44:26 <peter1138> JGR's braking patch would sort of achieve priority. I think a fast train would reserve farther ahead, beyond one signal.
08:45:17 <Eddi|zuHause> that's an "almost, but not quite" solution
08:48:49 <Eddi|zuHause> what timetables need, in my eyes, is: 1) a complete GUI overhaul, 2) a better way to set synchronisation points between separate lines, and 3) a conditional speed limit (e.g. 80km/h when on time, 100km/h when delayed more than 2 days)
08:49:15 <peter1138> Registered : Jan 10 11:44:29 2010 (11y 21w 6d ago)
08:49:27 <peter1138> Finally remembered to drop my freenode nickserv account...
08:50:24 <V453000> How are timetables related to merging tracks? Are you trying to let a delayed train go first, or hope to time them so precisely that they just avoid each other?
08:50:25 <Eddi|zuHause> i never had a freenode account. made it pretty unusable after that wave of spam attacks, where every channel and their mom set "registered users only". never really recovered from that
08:50:51 <Eddi|zuHause> V453000: i did the "so precisely that they avoid each other" successfully on a maglev track
08:51:02 <Eddi|zuHause> but it was a complete pain to set up
08:51:14 <TrueBrain> how is freenode doing .. :P
08:51:33 <Eddi|zuHause> and that was a "simple" A-B, A-C merge
08:51:49 <V453000> yes, that sounds close to impossible to expand, autoreplace, or anything
08:52:11 <Eddi|zuHause> that was at a point where no autoreplace would ever occur
08:52:50 <peter1138> TrueBrain, so many idle accounts from bouncers or matrix etc, I bet :-)
08:55:26 <V453000> sure, so what would you do in a more general case?
08:56:12 <peter1138> Build point to point tracks :)
08:56:37 <FLHerne> TrueBrain: Yeah, I don't think that graph accurately represents user activity -- anecdotally all but one of the channels I was in has moved wholesale
08:56:44 <V453000> that's probably the n1 most efficient solution, yes
08:56:58 <TrueBrain> FLHerne: we might just operate in a world that is biased towards these things :P
08:57:23 <FLHerne> 100% of Libera users have been active in the last few weeks; that's clearly not true of freenode
09:02:24 <_dp_> peter1138, there's literally just that single weird guy on reddit who cares about presignals xD
09:12:51 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9339: Codechange: add and use the ability to store (lists of) structs in savegames https://git.io/JZLZC
09:13:08 <TrueBrain> there, stations also converted ... now I need to do a pass off: store length-byte beforre list, instead of in a random other place
09:14:20 <TrueBrain> time to run this against 550 savegames :)
09:16:01 <peter1138> I apologise for the weird bit about stationspecs that I added back in the day...
09:17:09 <TrueBrain> that was the only thing in the whole station_sl.cpp that wasn't weird :P
09:17:24 <TrueBrain> its a simple list with a simple structure :)
09:17:39 <TrueBrain> for (StationCargoPacketMap::ConstMapIterator it = ge->cargo.Packets()->begin(); it != ge->cargo.Packets()->end(); ++it) {
09:17:39 <TrueBrain> SlObject(const_cast<StationCargoPair *>(&(*it)), this->GetDescription());
09:17:41 <TrueBrain> who ever wrote that
09:17:47 <TrueBrain> now THAT is some seriously unreadable lines ..
09:18:16 <TrueBrain> and SwapPackets .. like wtf ..
09:18:25 <TrueBrain> no, those parts are completely unclear in why/what they do
09:18:52 <TrueBrain> "Don't run when savegame version is higher than or equal to 123"
09:19:05 <TrueBrain> like .. I had to go in the history to figure out WHY that is the case
09:19:13 <TrueBrain> I can read the line of code that it was doing exactly as the comment said
09:19:14 <milek7> what's wrong with that for loop?
09:19:41 <peter1138> Could be improved with a range iterator? :D
09:19:44 <TrueBrain> so yeah peter1138 , I was happy when I arrived at SpecList :)
09:19:53 <peter1138> Oof. Simpler times :)
09:20:07 <peter1138> That may predate the C++ change, I guess.
09:20:28 <TrueBrain> some parts are, yes
09:20:31 <TrueBrain> and never got converted :)
09:20:36 <TrueBrain> other parts predated C++11
09:20:44 <Timberwolf> TrueBrain: on savegames, I have some old ones which were generated in a patchpack which added a couple of settings and bumped the savegame version, which now have a weird property that they load in a current version, but all the newgrf settings are missing. Is there any way to rescue them?
09:20:59 <Timberwolf> It's weird they load at all, I'd have thought something would be invalid.
09:21:25 <TrueBrain> Timberwolf: strongly depends on the changes :)
09:21:42 <peter1138> What truebrain is doing now should prevent that "sort of works" thing, I think.
09:21:48 <peter1138> Well, going forwards.
09:21:53 <TrueBrain> peter1138: absolutely
09:22:06 <TrueBrain> adding/removing of "things" in structs becomes "free" and compatible
09:22:16 <TrueBrain> modifying remains the hard part :)
09:22:20 <peter1138> Doing this should've been step 1 of changing from TTD's file format :D
09:22:34 <TrueBrain> the world has changed a lot in 17 years :P
09:22:44 <TrueBrain> non-self-describing formats are a bit .. meh, these days :)
09:23:07 <TrueBrain> but, after my works, except for the MAP chunks, they should all be self-describing :D
09:23:26 <Timberwolf> This of course is the classic preservation problem, you don't just need the media but also the player.
09:24:23 <Timberwolf> I vaguely recall thinking this at the time, "oh dear, if I want to play these games again I'll need the exact patched version which created them"
09:24:34 <TrueBrain> Timberwolf: either way, to recover such savegame, if it is only settings, I would modify the source to have those extra settings for that savegame range, so they are loaded, just not saved
09:24:40 <TrueBrain> meaning that loading should "just work" fine again
09:24:47 <TrueBrain> and after saving, those settings are gone
09:25:02 <TrueBrain> but it is not trivial, and needs knowledge of the changes of that savegame version
09:25:04 <peter1138> Grr, why do my rudder pedals always shift left. And why do I care when I'm not using them.
09:25:27 <peter1138> Self-describing map chunks... that'll be a task.
09:25:38 <TrueBrain> and the fact that they load, can have everything to do with pure luck :)
09:25:44 <TrueBrain> we read a lot of strings, which read to the next \0
09:25:50 <TrueBrain> which tends to reset "broken" stuff
09:26:19 <TrueBrain> no, wait, that is not true for savegames
09:26:22 <TrueBrain> only on the network
09:26:23 <Timberwolf> Quite. Interestingly OpenTTD handles the lack of newgrf config by applying whatever happens to be the current new-game profile.
09:26:51 <Timberwolf> So I might be better off guessing the original newgrf set, which is probably going to be predictable (and possibly even documented somewhere)
09:26:53 <TrueBrain> to me, NewGRF config just in general feels completely over complicated and not understood by any developer :P
09:28:39 <peter1138> NewGRF config was originally just numeric parameters after the newgrf filename in the config file.
09:28:42 <TrueBrain> Timberwolf: I have no clue why that savegame would load correctly .. only by pure luck, I can imagine
09:28:48 <peter1138> Then it got beefed out to be described by NewGRF.
09:29:41 <Timberwolf> Yeah, at first I thought, "ooh, I've found something which doesn't load properly" then realised it would have been from the days when I was combining hertogjan's old physics patch and a few of my own things.
09:30:05 <Timberwolf> So that it loads at all is quite spectacular.
09:30:35 <peter1138> Ah yes, old games could use newgrfs but nothing was stored about them, so it used the new-game newgrf config because that's how it worked.
09:30:54 <TrueBrain> oops, I have 9 open PRs, I have to be careful not to become "one of those" people :D
09:31:01 <peter1138> Not sure if there's a version cut off for that behaviour. I think I saw one for not-before but don't know about not-after. (LetsEncrypt lol)
09:31:19 <peter1138> TrueBrain, it's the *other* people's fault for not reviewing and approving, right...
09:31:41 <TrueBrain> yeah, as it is totally realistic for someone with an actual job to keep up with the SaveLoad changes I have been making :P
09:33:12 <Timberwolf> peter1138: That's another possibility, these are *old* games. They're definitely from a patched version, but I do recall making one which was save-compatible with trunk by just hardcoding all the parameters in the patches.
09:35:38 <Eddi|zuHause> i've made a special modified version once that converted a game from some old patched version to be trunk-loadable
09:36:42 <peter1138> I found a nice issue with my actul job system last night :(
09:37:40 <peter1138> ... involving database transaction locks, and files being removed from /tmp.
09:39:23 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9314: Codechange: let the setting name be std::string https://git.io/JZ82W
09:39:53 <Eddi|zuHause> i once learned the only time when it's safe to remove files from /tmp is during a reboot
09:40:30 <peter1138> That was why the file was removed :-)
09:41:52 <peter1138> It's a nice plan. Multiple instances, all running fairly independently except backed by one database. Instances can be stopped and started as needed. Except if this file is missing it causes a transaction lock to occur, and that fucks up all the other instances. Not completely, but just enough to cause things to not work right...
09:42:33 <peter1138> I am, of course, reworking it so it no longer happens.
09:44:07 <TrueBrain> we expected nothing less from you :)
09:47:24 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9315: Codechange: use std::string with network's ContentInfo https://git.io/JZ8rI
09:47:47 <TrueBrain> okay, savegame-regression-test-suite-thingy says my PR has no regressions \o/
09:47:52 <_dp_> hm... there is an overflow because line height is more than font height... should I fix the line, the overflow or both...
09:48:05 <LordAro> everyone else looking at the clouds where the eclipse should be?
09:48:46 <TrueBrain> like ... completely blue
09:48:59 <peter1138> _dp_, there's quite a lot of places I found that assume one thing is always bigger than the other. I've fixed them in my mammoth scaling PR, but need to split that up a bit, when I have brain power.
09:50:17 <_dp_> so... should i fix the line, the overflow or wait for peter to do it xDD
09:50:48 <TrueBrain> LordAro: it is especially fun to see this on my solar panels .. it is already going down, the power produces, while it is not even midday :)
09:58:25 <V453000> Floating-point operations are only possible when both operands are compile-time constants. Hm
09:58:59 <V453000> I guess NML doesn't have a round() or something?
10:01:31 <V453000> I have shit like this introduction_date: date(0,1,1) + (param_date_start * 365.25) + (((1982 - 1900)/199)*(param_date_end-param_date_start) * 365.25);
10:05:53 *** tokai|noir has joined #openttd
10:05:53 *** ChanServ sets mode: +v tokai|noir
10:09:13 *** Smedles has joined #openttd
10:11:19 <V453000> guess I need to convert it into a "start date" and a "length multiplier", multiplying the dys
10:11:23 <V453000> that will be good enough
10:12:33 *** tokai has quit IRC (Ping timeout: 480 seconds)
10:12:50 *** ChanServ sets mode: +v tokai
10:19:44 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
10:19:45 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9339: Codechange: add and use the ability to store (lists of) structs in savegames https://git.io/JZLZC
10:56:45 <TrueBrain> (the green line are my solar panels .. the blue dip is me cooking)
11:04:15 <TrueBrain> well, "cooking" .. I made a tosti :P
11:11:18 * Timberwolf flashbacks to staring at Factorio solar output vs. charge graphs.
11:17:04 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:18:41 *** WormnestAndroid has joined #openttd
11:43:33 <FLHerne> V453000: `date()` can take non-constant parameters
11:45:31 <FLHerne> so `(param_date_start * 365.25)` can be `date(param_date_start, 1, 1)` or so
11:46:37 <FLHerne> [only the year can be non-constant in `date()`]
12:07:26 <jez> is anyone here a moderator at tt-forums.net?
12:08:32 <jez> i'm working with an oldish patch pack that uses a patch from another patch pack. that other patch pack has a post that'd been updated a bunch of times with newer versions of the .diff for his patch pack. at one point, he fixed a crash bug that's still in the version of that patch pack that the one i'm using uses. i need to get a diff of the two revisions before and after he applied the fix for the bug.
12:09:14 <jez> to do that, i need to get access to the history of the edits for that post so i can get the diffs for r51 and r52 (sadly he didn't just give access to his hg repo)
12:09:48 <jez> as i can only see the latest edit, i can only get r53. was hoping admins might be able to access historical versions
12:11:59 <LordAro> pretty sure phpbb doesn't store edit history
12:13:03 <FLHerne> orudge should have backups though :p
12:13:32 <LordAro> the file will probably still be there, but good luck finding the id
12:14:14 <jez> the guy hasnt been active for 6 years either so he probably isnt contactable :-(
12:22:46 <_dp_> may be jgrpp has something on it but that's a lot of stuff there to filter through
12:24:33 <milek7> probably easier to just fix the bug ;p
12:24:41 <milek7> than hunting old attachments
12:25:17 <_dp_> yeah, that would be my thinking too xD
12:26:04 <_dp_> you may need some good C++ knowledge to fix a crash though
12:44:49 <jez> ok, looks like i managed to fix it. a line needed moving.
12:45:03 <jez> surprised it hadn't been causing more crashes than it did actually
12:49:40 <peter1138> "How did that ever work" is fairly common :D
12:51:04 <FLHerne> The really annoying thing is when it couldn't have possibly worked, and then you fix it, and the new version that's obviously correct doesn't
13:07:15 <jez> i've successfully patched OpenTTD to allow the jukebox to play WAV files
13:07:33 <jez> it's not all that difficult. dunno why it isn't in trunk. i have nice versions of the tracks playing now :-)
13:07:50 <jez> im also using the original's opening theme instead of TTD's, which i prefer
13:08:32 <jez> introduce a new .obm for wavs, remove MD5s, disable MD5 check for WAV music files
13:08:46 <_dp_> what's even the point of playing wav?
13:08:57 <_dp_> mp3 would make some sense but there're better programs for it
13:09:07 <jez> so you can play custom jokebox music (in my case, a nicer-sounding MT-32 version than than windows MIDI rendering)
13:09:28 <jez> MP3 is harder because directmusic doesn't support it. wav was easy because it was already supported.
13:09:58 <jez> there aren't "better programs for it" because there's nothing better than having the in-game music experience of the right music being played at the right time :-P
13:10:16 <jez> also, i patched it to mute sounds on the opening screen. no idea why that isn't in trunk either. sound fx on the opening screen are annoying
13:10:51 <_dp_> except openttd doesn't do any dynamic music and just does playlists like any other program :p
13:12:21 <jez> it plays the title screen music on the title screen
13:25:41 <V453000> FLHerne: that does make it 1 step less stupid, thanks! I still changed the system to multiply only, I just have something wrong in my conversion for now
13:26:10 *** WormnestAndroid has quit IRC (Remote host closed the connection)
13:26:23 *** WormnestAndroid has joined #openttd
13:26:34 <V453000> the 199 was the default range length, so it was getting a "position in range of progression"
13:27:39 <TrueBrain> can someone turn down the thermostat outside? I am not enjoying this
13:29:08 <FLHerne> V453000: What do you have now?
13:29:17 <dwfreed> TrueBrain: heat advisory here today; anticipated heat indexes of 106 F
13:30:02 <V453000> introduction_date: date( param_date_start, 1, 1) + ( 18 * param_date_multiplier);
13:30:17 <V453000> the 18 is just somehow gotten wrong by the python that ... gets it
13:31:41 <V453000> it's supposed to be conversion from original introduction dates to this parameter based sistem
13:31:54 <FLHerne> Well, 18 days doesn't seem like an obviously-meaningful time period
13:32:07 <V453000> so it's something like "number of days since the game start / 10"
13:32:34 <V453000> but as I'm saying, the 18 is not supposed to be this number. :)
13:32:44 <TrueBrain> make it 42, and all is well :D
13:33:08 <FLHerne> What's param_data_multiplier?
13:33:43 <V453000> a parameter that can extend the period in which engines are introduced
13:34:04 <FLHerne> With what value range?
13:34:26 <V453000> 10 is default, which means 199 year total range
13:34:43 <V453000> so you can shorten it to 19.9 years, or make it longer to some wtf number
13:35:07 <V453000> I think 666 is currently the max but that's quite arbitrary
13:35:13 <FLHerne> ...so you have more code somewhere? I don't see how that's connected to what you copied just now
13:35:39 <FLHerne> <V453000> introduction_date: date( param_date_start, 1, 1) + ( 18 * param_date_multiplier);
13:35:54 <V453000> yes that's the intro date property of one of the vehicles
13:35:54 <FLHerne> So if param_date_multiplier is 10, you're adding about half a year
13:36:10 * _dp_ was quite surprised to find out that some openttd games overflow postgres date type
13:36:11 <V453000> yes, which is incorrrect, as 18 is wrong
13:36:36 <FLHerne> That 18 is wrong, I suppose, which was why I asked in the first place
13:36:45 <V453000> I am saying that from the start
13:36:55 <FLHerne> Oh, "somehow gotten wrong by the python that ... gets it" is your code
13:37:42 <FLHerne> Ok, somehow I misread that as saying nmlc was broken :p
13:37:57 <FLHerne> (which tbf it usually seems to be with units)
13:38:45 <FLHerne> But yeah, that looks sane except that the numbers should be bigger than 18 :D
13:42:21 <V453000> yeah I was keeping a /199 in my python formula while it shouldn't have been there anymore
13:46:27 <V453000> it's working now I think :)
14:48:35 *** Kitrana has joined #openttd
14:53:29 <peter1138> _dp_, stupid database designers!
14:54:44 *** Kitrana1 has joined #openttd
14:55:15 *** Kitrana2 has joined #openttd
15:01:06 *** Kitrana has quit IRC (Ping timeout: 480 seconds)
15:02:46 *** Kitrana1 has quit IRC (Ping timeout: 480 seconds)
15:30:43 *** Wormnest has joined #openttd
15:34:56 *** Progman has joined #openttd
15:43:09 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #9337: Pressing a station name in the station list doesn't move you to the correct station https://git.io/JGDBu
15:58:52 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on issue #9337: Pressing a station name in the station list doesn't move you to the correct station https://git.io/JGDBu
16:04:14 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on issue #9337: Pressing a station name in the station list doesn't move you to the correct station https://git.io/JGDBu
16:31:36 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #9314: Codechange: let the setting name be std::string https://git.io/JZR2z
16:39:19 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on issue #9337: Pressing a station name in the station list doesn't move you to the correct station https://git.io/JGDBu
16:41:21 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9314: Codechange: let the setting name be std::string https://git.io/JZRrp
16:42:28 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9314: Codechange: let the setting name be std::string https://git.io/JZRoa
16:43:59 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
16:45:50 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #9315: Codechange: use std::string with network's ContentInfo https://git.io/JZR6U
16:50:38 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9315: Codechange: use std::string with network's ContentInfo https://git.io/JZRij
16:54:04 <DorpsGek> [OpenTTD/OpenTTD] V453000 commented on pull request #9346: Feature: [NewGRF] Maximum curve speed modifier for rail vehicles https://git.io/JZRXG
16:59:33 *** gelignite has joined #openttd
17:02:39 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9338: Several code refactors of the SaveLoad code https://git.io/JZRD1
17:02:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #9314: Codechange: let the setting name be std::string https://git.io/JGCaz
17:17:03 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9314: Codechange: let the setting name be std::string https://git.io/JZR7G
17:18:47 <TrueBrain> w00p, tnx for that review Rubidium :)
17:20:23 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZR5z
17:20:44 <TrueBrain> can we switch to Rust? I like variable-lifetime to be more explicit :D
17:23:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9343: Add: adhere the autosave_on_exit setting for Null videodriver https://git.io/JZRdV
17:23:31 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZGDH
17:24:52 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZGDH
17:26:44 *** tokai|noir has joined #openttd
17:26:44 *** ChanServ sets mode: +v tokai|noir
17:29:56 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZGDH
17:30:32 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZGDH
17:30:33 <TrueBrain> ugh, okay, enough bikeshedding .. :P Sorry for the spam :)
17:30:34 *** frosch123 has joined #openttd
17:31:25 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZRA7
17:31:44 <TrueBrain> ffs, and now the English is ugly :P
17:33:33 *** tokai has quit IRC (Ping timeout: 480 seconds)
17:34:04 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZGDH
17:34:36 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZRpO
17:34:56 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9343: Add: adhere the autosave_on_exit setting for Null videodriver https://git.io/JZG9y
17:38:03 <TrueBrain> frosch123: why don't containers extend the lifetime of what they contain? (talking about std::span and std::string_view here) .. I don't like lifetimes in C++, they are annoying :P
17:40:58 <glx> hmm I guess I'll have some conflicts to fix
17:43:08 <glx> oh a nice one in cheat_sl, almost all the file is in conflict
17:44:59 <TrueBrain> "getting here" isn't fine frosch123 ? Hmm
17:45:02 <frosch123> TrueBrain: they are not containers, they are views
17:45:08 <TrueBrain> frosch123: yeah, that is what I meant
17:45:22 <frosch123> TrueBrain: i guess depends on the meaning?
17:45:40 <TrueBrain> in this context, I was sure "getting here" was fine :)
17:45:48 <TrueBrain> as that is how you say it in Dutch :P
17:46:05 <frosch123> "getting to this situtation" or "getting onto gog"
17:46:40 <TrueBrain> not sure if it is "here" or "there" :P
17:46:51 <TrueBrain> but I understand, it points to something unclear
17:46:55 <TrueBrain> so that is confusing anyway :D
17:47:44 <TrueBrain> anyway, more importantly: just surprised views don't extend the lifetime honestly .. sometimes is a bit weird
17:47:52 <TrueBrain> as things that have infinite lifetime is fine
17:48:03 <TrueBrain> but things that do not, like a std-string, need more care
17:48:22 <TrueBrain> confusing, for a person like me :P
17:48:26 <frosch123> c-strings have no infinite lifespan. or do you mean string literals?
17:48:46 <TrueBrain> "abc" <- that, over "abc"s, as example given by cppreference
17:48:54 <Rubidium> frosch123: c-string literals ;)
17:48:56 <TrueBrain> call them how ever you want :)
17:49:03 <TrueBrain> the thing ending up in the .data :P
17:49:06 <frosch123> there is also "abc"sv :)
17:50:03 <TrueBrain> frosch123: wtf does the "v" do?
17:50:26 <frosch123> computes the length at compile time, and then stores data+length in .data
17:51:04 <frosch123> though i would not be surprised, if optimisers figured that out themself
17:51:14 <TrueBrain> I would hope so ...
17:51:39 <frosch123> but you discovered that std::vector consturctor is not yet constexpr :)
17:51:58 <TrueBrain> yes .... painfully ... we found that out the other day :P
17:52:04 <TrueBrain> as you said it was, but it really wasn't :P
17:52:12 <TrueBrain> I TRUSTED YOU! :P :D
17:52:24 <glx> isn't it constexpr in c++20 ?
17:52:33 <TrueBrain> glx: it is; just in no compiler :P
17:53:31 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
17:53:47 <TrueBrain> glx: starting to look like something solid :D (your branch)
17:54:26 <glx> I still don't like fix_pointers variable
17:54:59 <glx> but I managed to remove the need for save variable :)
17:55:35 <TrueBrain> converting all chunks did that for you? :)
17:55:53 <TrueBrain> what FixPointers soon (tm) can do, is just run over all chunks
17:56:03 <TrueBrain> as there is no real reason not to, as it only works on a few types anyway
17:56:08 <TrueBrain> just a bit pedantic atm it is selective
17:56:18 <glx> for now the variable is just for the DEBUG
17:57:30 <TrueBrain> btw, it is likely CH_READONLY will have to become a pointer
17:57:38 *** WormnestAndroid has joined #openttd
17:57:46 <TrueBrain> as I still might need to know if the load code is CH_RIFF or CH_ARRAY
17:57:58 <TrueBrain> but that is a bit further away on my TODO list, so who knows if that is still true by then :)
17:59:13 <TrueBrain> holy crap ... it really is too warm
18:01:41 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #9315: Codechange: use std::string with network's ContentInfo https://git.io/JGCyL
18:02:42 <TrueBrain> btw, frosch123 , {} or "" .. did we settle on either one?
18:05:17 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9315: Codechange: use std::string with network's ContentInfo https://git.io/JZ0tg
18:08:56 <frosch123> TrueBrain: i would choose "" because it is more verbose, and you can easier find the string in a long list of values, like in the SLE_ macros.
18:09:21 <frosch123> but i agree that you may save one cpu cycle in unoptimised builds by using {} :p
18:09:27 <LordAro> there's quite a lot of {} scattered around now
18:09:48 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #9315: Codechange: use std::string with network's ContentInfo https://git.io/JGCyL
18:10:43 <TrueBrain> LordAro: yup. But, I agree, "" is a lot more verbose
18:11:30 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZ0m9
18:11:54 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9342: Change: allow pause/unpause console command in single player too https://git.io/JZGDH
18:25:45 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #9347: [Codechange] Remove C-string Packet::Recv_string https://git.io/JZ0Zs
18:27:50 *** ChanServ sets mode: +v tokai
18:29:29 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #9347: Codechange: [Network] Remove C-string Packet::Recv_string https://git.io/JZ0nK
18:29:32 <TrueBrain> seems Rb is buying us cake :P
18:34:38 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
18:43:15 <frosch123> what's the occassion? it's way beyond 25k commits, and way below #10000 ?
18:43:35 <TrueBrain> removing c-string function deep in the network layer :)
18:43:55 <LordAro> and probably not breaking anything in the process!
18:44:23 <frosch123> savegame version 300 is coming up, but that's nothing to party :p
18:44:34 <frosch123> too bad some patchpack used up version 255
18:44:59 <frosch123> would have been a nice fool's news: we are out of savegame versions, no more features can be accepted
18:45:23 <TrueBrain> sorry I took that from you :P
18:51:20 *** Kitrana2 has quit IRC (Ping timeout: 480 seconds)
19:01:21 *** crem has quit IRC (Quit: WeeChat 3.0)
19:04:07 <frosch123> hmm, black cannot wrap long f strings
19:04:25 <TrueBrain> he doesn't force breaks on long sequential things, no
19:04:31 <TrueBrain> similar for long integers, etc
19:04:33 <TrueBrain> you have to help him :)
19:05:01 <TrueBrain> I want to make a very racist joke now, but I will leave that be :P
19:07:28 <glx> [19:57:47] <TrueBrain> as I still might need to know if the load code is CH_RIFF or CH_ARRAY <-- but chunk type as specified in the struct doesn't mean anything at load time, it relies on what the save says
19:07:53 <TrueBrain> glx: absolutely! But I intend to change that drasticly
19:07:59 <TrueBrain> there is a lot of freedom atm
19:08:06 <TrueBrain> a CH_ARRAY can store what-ever-the-fuck-it-wants-even-not-an-array
19:08:41 <TrueBrain> I kinda want to .. nudge things in the right direction, but making that a bit more abstracted away :)
19:09:02 <frosch123> i replaced it with str.format(), way shorter string :p
19:10:09 <TrueBrain> youi and str.format :P
19:14:41 <TrueBrain> glx: the first one, after my changes, it can just call FixPointer always
19:14:42 <glx> and debug levels are weird, save and load are level 2, pointers are level 3
19:14:57 <TrueBrain> so that at least makes it a debug-only
19:15:11 <TrueBrain> owh, and the debugs also move up ofc
19:15:18 <TrueBrain> so yeah .. after my changes, that shouldn't be an issue anymore :)
19:15:40 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
19:15:49 *** WormnestAndroid has joined #openttd
19:24:10 *** Kitrana has joined #openttd
19:35:22 *** Kitrana has quit IRC (Read error: Connection reset by peer)
19:43:37 <TrueBrain> glx: what I really like about your work, as that means we can move the _desc tables inside those classes too :)
19:43:44 <TrueBrain> that makes things a lot more .. contained :D
19:44:22 <TrueBrain> there is also no rush :)
19:53:23 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #9347: Codechange: [Network] Remove C-string Packet::Recv_string https://git.io/JZ0Zs
19:55:33 <Rubidium> but I'll buy some cake and you can get it if you can find it ;)
19:59:56 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZ0Nc
19:59:59 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9336: Fix: [Script] Ensure the saved script strings are properly validated and terminated when being read from the save game https://git.io/JZ0NW
20:00:03 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9330: Fix #9329: [CMake] FindVersion.cmake relied on internal cmake variable https://git.io/JZ0N8
20:00:29 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZ0Nw
20:01:45 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZ0Av
20:03:27 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #9336: Fix: [Script] Ensure the saved script strings are properly validated and terminated when being read from the save game https://git.io/JGDIR
20:04:50 <frosch123> cargodist disables subsidies, right?
20:10:36 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain dismissed a review for pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZ0Nc
20:10:39 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZG1r
20:12:48 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZ0jZ
20:13:29 *** gelignite has quit IRC (Quit: Stay safe!)
20:14:12 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #9330: Fix #9329: [CMake] FindVersion.cmake relied on internal cmake variable https://git.io/JGV2l
20:14:15 <DorpsGek> [OpenTTD/OpenTTD] glx22 closed issue #9329: No rule to make target 'generated/ottdres.rc' when Cross-compiling with CMake + MINGW https://git.io/JGVvY
20:17:40 *** andythenorth has joined #openttd
20:19:24 <TrueBrain> michi_cc: personally, I think it makes sense to have subsidy configurable in length, as that allows for certain types of gameplay; and yes, that is one more setting .. but ... yeah, I also can't think of another way :P
20:26:27 <michi_cc> TrueBrain: Can you have a tiny look, even if I could technically still approve it :)
20:28:06 <TrueBrain> "attract a {NUM} year subsidy" <- doesn't need plural?
20:28:07 <michi_cc> The one permission loop-hole: just pick some random PR not by you, force-push stuff and approve and merge because you aren't the PR author :)
20:28:23 <TrueBrain> michi_cc: a few of us can also just hit "merge"
20:28:34 <TrueBrain> I mean .. the system is not meant to avoid abuse like that :P
20:28:38 <michi_cc> IMHO (not English etc), I would indeed say "a 3 year subsidy"
20:28:51 <glx> TrueBrain: the red button ?
20:28:53 <michi_cc> Could be wrong English, though :)
20:29:08 <TrueBrain> "the next {NUM} year{P "" s}!" <- reads fine for 2+, but for 1 reads weird :P
20:29:15 <_dp_> TrueBrain, throw out subsidies into gamescript :p
20:30:41 <michi_cc> Make plural system even more powerful by allowing {P "year" "{NUM} years"} :)
20:34:46 <TrueBrain> the DParam thing is very weird :P
20:37:07 <DorpsGek> [OpenTTD/OpenTTD] michicc commented on pull request #9344: Change: Reduce real sprite groups if possible. https://git.io/JZEty
20:38:31 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9081: Feature: Configurable subsidy duration https://git.io/JZEqG
20:39:03 <TrueBrain> 1 comment of every type: rebase-error, coding style, missing-feature-usage, wrong doxygen and nitpick :D
20:39:57 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9081: Feature: Configurable subsidy duration https://git.io/JZEq7
20:40:57 <TrueBrain> good point michi_cc .. so lets just ignore that comment of mine :)
20:41:50 <TrueBrain> owh, and STR_JUST_INT is "wrong" , I think .. shouldn't the setting read: "1 year" instead of "1"?
20:42:40 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9081: Feature: Configurable subsidy duration https://git.io/JZEmd
20:48:59 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #9268: [MinGW] Fix MinGW32 build failure and almost all MinGW32/MinGW64 warnings https://git.io/JsGOr
20:55:51 *** Samu has quit IRC (Quit: Leaving)
21:04:54 <michi_cc> TrueBrain: The subsidy PR changes the number of params for the strings (leading to strgen warnings). Will eints sort that our or is there some manual work needed?
21:05:09 <TrueBrain> I would like to say the first
21:05:16 <TrueBrain> but I sneaky look at frosch123 to get confirmation
21:05:30 <frosch123> eints will mark all translations "invalid"
21:05:39 <frosch123> it will remove them from git on next commit
21:05:45 <frosch123> but keep them visible for the translators to fix
21:09:56 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:10:52 <TrueBrain> peter1138: michi_cc used a class enum !! :D
21:11:43 <TrueBrain> michi_cc: I can nitpick about a dot at the end of a tooltip
21:11:48 <TrueBrain> not sure I can muster that
21:13:04 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZE8E
21:13:27 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9268: [MinGW] Fix MinGW32 build failure and almost all MinGW32/MinGW64 warnings https://git.io/JZE8X
21:13:38 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9341: Add: '-X' option to ignore global folders in the search path https://git.io/JZG1r
21:17:33 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #9268: [MinGW] Fix MinGW32 build failure and almost all MinGW32/MinGW64 warnings https://git.io/JsGOr
21:29:03 *** frosch123 has quit IRC (Quit: be yourself, except: if you have the opportunity to be a unicorn, then be a unicorn)
21:32:04 <glx> I want to see what it does now :)
21:33:10 <TrueBrain> another incredibly slow target ... well, after MacOS, we can handle that :D
21:35:14 *** sla_ro|master has quit IRC ()
21:38:30 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:45:03 <glx> finally a no warning mingw
21:45:33 <TrueBrain> you have to buy us cake too!
21:45:36 <TrueBrain> two cakes in 1 day :)
21:56:38 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
22:01:03 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZEX3
22:01:04 <glx> oh nice one, but I don't really understand this code :)
22:03:13 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZEXj
22:05:25 <DorpsGek> [OpenTTD/OpenTTD] James103 commented on pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZE1y
22:05:47 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain dismissed a review for pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZEXj
22:05:50 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZEX3
22:06:02 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZE1A
22:06:16 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZEMf
22:06:41 <glx> ah 7 is enough, I usually use 8
22:11:03 *** andythenorth has quit IRC (Quit: andythenorth)
22:14:33 <TrueBrain> glx: now the question, does MacOS finish before or after MSYS?
22:15:57 <TrueBrain> they get 90s slack, as the cache was cold
22:16:40 <TrueBrain> omg, with the 90s they even won
22:17:29 <TrueBrain> wow, annotation check is .. slow
22:18:27 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #9349: Fix #9348, 4d74e5: don't try to sell shares of spectators https://git.io/JZEX3
22:18:27 <TrueBrain> runner was slow with reporting, it seems
22:18:45 <TrueBrain> right, record for quickest bug-fix?
22:32:49 *** Smedles has joined #openttd
22:53:42 *** Progman has quit IRC (Remote host closed the connection)
23:36:26 *** WormnestAndroid has quit IRC (Remote host closed the connection)
23:49:40 *** WormnestAndroid has joined #openttd
continue to next day ⏵
