IRC logs for #openttd on OFTC at 2024-07-10
            
00:38:35 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
00:45:41 <DorpsGek> [OpenTTD/nml] glx22 commented on pull request #302: Add: 'const' keyword to allow users to define new constants https://github.com/OpenTTD/nml/pull/302#issuecomment-2219214182
01:06:46 <DorpsGek> [OpenTTD/nml] glx22 updated pull request #302: Add: 'const' keyword to allow users to define new constants https://github.com/OpenTTD/nml/pull/302
01:26:46 *** Wormnest has joined #openttd
02:07:40 *** Wormnest has quit IRC (Quit: Leaving)
02:41:26 *** debdog has joined #openttd
02:45:01 *** D-HUND has quit IRC (Ping timeout: 480 seconds)
05:24:05 *** Flygon has joined #openttd
06:37:43 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #302: Add: 'const' keyword to allow users to define new constants https://github.com/OpenTTD/nml/pull/302#pullrequestreview-2168158952
06:59:05 *** jinks has quit IRC (Quit: ZNC - http://znc.in)
06:59:26 *** jinks has joined #openttd
08:21:41 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)
08:24:21 *** mindlesstux has joined #openttd
08:47:30 *** jinks has quit IRC (Remote host closed the connection)
08:49:02 *** jinks has joined #openttd
09:18:38 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852
09:24:33 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852#pullrequestreview-2168544000
09:25:39 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12853: Codechange: Replace default overrides arrays with std::pair and use range-for. https://github.com/OpenTTD/OpenTTD/pull/12853
09:27:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852#pullrequestreview-2168550630
09:29:55 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852#pullrequestreview-2168556546
09:30:26 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852#pullrequestreview-2168557698
09:30:53 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852#pullrequestreview-2168558712
09:31:14 <LordAro> needs ranges, clearly
09:41:32 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12852: Codechange: Use range-for and std::size with _plural_forms. https://github.com/OpenTTD/OpenTTD/pull/12852
09:43:53 <peter1139> Hmm, so I stick my power meter on my heavily laden touring bike or not...
09:45:05 <LordAro> why not lol
09:51:14 <peter1139> Ah, the best progress meter... https://gist.githubusercontent.com/PeterN/8f90592b36343a1a835300ecae20209f/raw/af7ef2bf4f3968d2375e42cc03b2d55fe0103a05/gistfile1.txt
09:51:27 <peter1139> Why not? Hmm, well... not sure.
09:52:36 <peter1139> Moving them from bike to bike means the threads get worn a bit, but also the threads are less likely to get seized... I dunno.
09:53:13 <peter1139> Might help ensure I don't go too hard on the knee? Excuses...
09:53:40 <peter1139> Oof, it's raining :(
09:55:34 <LordAro> can confirm
09:56:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12854: Codechange: Use reference for always_accepted output parameter of AddAcceptedCargo. https://github.com/OpenTTD/OpenTTD/pull/12854
10:05:58 <peter1139> Did not expect to be packing a jumper and waterproofs...
10:06:30 <LordAro> ikr
10:06:58 <LordAro> if the weather could improve drastically for 3 weeks time, that'd be great
10:14:32 *** mindlesstux has quit IRC (Ping timeout: 480 seconds)
10:19:15 *** mindlesstux has joined #openttd
10:30:39 <DorpsGek> [OpenTTD/team] glx22 commented on issue #572: [zh_TW] Translator access request https://github.com/OpenTTD/team/issues/572
10:33:22 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #302: Add: 'const' keyword to allow users to define new constants https://github.com/OpenTTD/nml/pull/302
10:35:32 <yiffgirl> man when you only need to recompile one or a few objects instead of the whole thing from scratch it's orders of magnitude faster
10:55:31 <yiffgirl> https://cdn.discordapp.com/attachments/1008473233844097104/1260549960793980928/Screenshot_from_2024-07-10_11-50-36.png?ex=668fba22&is=668e68a2&hm=3718f7fbd3fd42de85f4c7893e03d69b990c61115c5e5935ea6270325f2e2551&
10:55:31 <yiffgirl> https://cdn.discordapp.com/attachments/1008473233844097104/1260549961582379048/Screenshot_from_2024-07-10_11-51-20.png?ex=668fba22&is=668e68a2&hm=13db1f812120b3cebfca9e08dd248c3a22651a27f7bc5322a340f61e157551de&
10:55:31 <yiffgirl> one of these is current master with tree placement set to original. one of these is current master with tree placement set to improved. can you tell which is which?
11:11:43 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #12835: Fix #12832: Sanitise strings from NewGRF before logging them in Debug output. https://github.com/OpenTTD/OpenTTD/pull/12835
11:13:46 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #12835: Fix #12832: Sanitise strings from NewGRF before logging them in Debug output. https://github.com/OpenTTD/OpenTTD/pull/12835#pullrequestreview-2168790857
11:13:56 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #12854: Codechange: Use reference for always_accepted output parameter of AddAcceptedCargo. https://github.com/OpenTTD/OpenTTD/pull/12854#pullrequestreview-2168791270
11:14:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #12646: Change: Use default NewGRF cargo translation table. https://github.com/OpenTTD/OpenTTD/pull/12646
11:30:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12854: Codechange: Use reference for always_accepted output parameter of AddAcceptedCargo. https://github.com/OpenTTD/OpenTTD/pull/12854
11:49:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #12855: Codechange: Reduce SpriteGroup object size by setting enum base type. https://github.com/OpenTTD/OpenTTD/pull/12855
11:50:04 <peter1139> I had a patch (stash) for that but couldn't find it, so I rewrote it again :p
11:50:39 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #12855: Codechange: Reduce SpriteGroup object size by setting enum base type. https://github.com/OpenTTD/OpenTTD/pull/12855#pullrequestreview-2168861857
11:53:26 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12835: Fix #12832: Sanitise strings from NewGRF before logging them in Debug output. https://github.com/OpenTTD/OpenTTD/pull/12835
11:53:29 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #12832: [Bug]: Several NewGRFs get disabled on load if OpenTTD is started with `-dgrf8` https://github.com/OpenTTD/OpenTTD/issues/12832
11:56:22 *** merni has joined #openttd
11:56:22 <merni> peter1139: Imagine a "peter1139 Patchpack" with all your patches somehow smashed together :p
11:58:49 <LordAro> we generally call it 'master'
12:00:21 <merni> no I mean including the ones which haven't been PRd/merged lol
12:22:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12855: Codechange: Reduce SpriteGroup object size by setting enum base type. https://github.com/OpenTTD/OpenTTD/pull/12855
13:29:17 <peter1139> Okay, well I think that settles it.
13:29:25 <peter1139> I simply cannot get the touring bike in my car.
13:30:13 <peter1139> I would have to remove the front wheel (easy) and the front mudguard (not so easy)
13:44:04 <peter1139> (And maybe the front rack (also not easy))
14:35:08 *** DorpsGek has joined #openttd
14:35:08 *** ChanServ sets mode: +o DorpsGek
14:37:01 <truebrain> right ... seems DorpsGek froze up .. odd
15:05:38 <peter1139> Poor DorpsGek.
15:05:47 <truebrain> bit sad, if you ask me
15:05:49 <truebrain> one job .... πŸ˜›
15:08:14 <peter1139> Hmm, so `std::vector<std::shared_ptr<X>>` vs `std::vector<X>`.
15:08:44 <truebrain> "depends" πŸ™‚
15:08:45 <peter1139> The former is cheaper to sort. But pointers.
15:09:03 <truebrain> sort-by-pointer-value? πŸ˜„
15:09:20 <peter1139> Not quite :p
15:09:40 <peter1139> You can of course just move the pointer instead of the actual object.
15:10:47 <_jgr_> Using shared_pointer implies that you need reference stability and for the items to outlive the vector, just using the value implies that you don't need either of those
15:12:39 <_jgr_> unique_ptr perhaps? πŸ˜›
15:23:25 *** HerzogDeXtEr has joined #openttd
15:26:42 *** nielsm has quit IRC (Ping timeout: 480 seconds)
15:27:34 *** Wormnest has joined #openttd
15:37:03 *** Ox7C5 has joined #openttd
15:41:21 <peter1139> Hmm, shame, there is at least one place where the list is copied, and also where the pointers are used after being inserted into a vector.
15:41:29 <peter1139> s/list/vector/
15:49:20 *** gelignite has joined #openttd
16:23:41 <peter1139> Hmm, maybe a 6.30am start.
16:24:18 *** gelignite has quit IRC (Read error: Connection reset by peer)
16:24:46 <peter1139> Have I been Zorged in the forums?
16:28:14 *** gelignite has joined #openttd
16:36:34 *** Flygon has quit IRC (Read error: Connection reset by peer)
16:38:32 *** Flygon has joined #openttd
16:43:51 <andythenorth> have you?
16:44:09 <andythenorth> oh that cdist thread
16:44:18 <andythenorth> I drafted a reply there, but deleted it
16:44:29 <andythenorth> afaik all this stuff about cdist calculating capacity is woo
16:44:36 <andythenorth> I have notes about it from fonsinchen
16:45:00 <andythenorth> it just uses distance for demand, then it does [something] about symmetric
16:45:15 <andythenorth> cdist still misunderstood?
16:46:57 <merni> andythenorth: Maybe you could share those :p (if you haven't already somewhere)
16:48:29 <merni> "saturation of short paths before using high-capacity paths" does make me think somehow capacity is involved but perhaps not
17:36:59 *** efessel has joined #openttd
17:36:59 <efessel> truebrain: FYI - I've moved some compute resources over to Oracle Cloud from AWS. It's nice having a $0 bill every month.
17:37:20 <truebrain> I can imagine πŸ™‚
17:37:22 <peter1139> You managed to get a response from them?
17:37:39 <truebrain> in fact, we just created an account at OCI. Now to figure out how to do cloudinit via Instance Pool ... #documentation-sucks-yo
17:37:46 <efessel> So far no big complaints, although their free compute means arm compiles which is no big deal
17:37:59 <truebrain> We already run on ARM, as it was cheaper on AWS too πŸ˜›
17:38:15 <truebrain> but glad to hear it is working out for you πŸ™‚
17:38:23 <efessel> I'm using Terraform w/ their provider
17:38:37 <truebrain> yeah, but that is just half of the puzzle πŸ™‚
17:39:06 <efessel> Right, I have't messed with dynamic instances at all
17:39:15 <efessel> terraform runs the cloud init
17:39:17 <truebrain> I like auto-deployments, where everything is installed automatically. And normally you use cloud-init for that. Which is "easy" when spinning up a single instance, but somehow it is hiding from me how to do that with Instance Pooling 😦
17:39:40 <efessel> Yep, I'm a fan of the computer doing all the work too
17:40:04 <truebrain> btw, Pulumi is a great wrapper around Teraform, to make things easier to understand in many ways
17:40:17 <efessel> well terraform doesn't do the cloud init, it's specified in userData prop on the instance config. Not sure how that works with instance pools though.
17:41:08 <truebrain> that is the weird part .. userData exists in Instance, but not in InstanceConfiguration
17:41:16 <truebrain> not sure what that is about .. but it just means I have to actually read πŸ˜„
17:42:23 <efessel> yeah I def had to read teh dox for a bunch of things, assigning a "reserved" public IP isnt' a strait forward process
17:43:06 <efessel> I assume you've updated your account to Pay as you go?
17:43:19 <truebrain> you have to; otherwise there won't be any instances to spin up πŸ˜„
17:43:38 <efessel> Right, which is odd. What's the point of the trial if you can't "try"
17:43:45 <truebrain> I wrote them as such
17:43:46 <audigex> truebrain: Oracle cloud’s interface is awful, puts me off a lot
17:43:54 <truebrain> and I got a weird reply back, dodging the question
17:44:01 <truebrain> so I am sure I am not the only one who complaint about it πŸ˜›
17:44:02 <efessel> Their support is powered by AI
17:44:13 <truebrain> audigex: as with ALL the other Cloud interfaces. None of them are any good, or understandable. None.
17:44:13 <efessel> So you'd be better off with Chat GPT
17:44:31 <efessel> (found that out after a ridiculous back and forth)
17:44:41 <audigex> truebrain: Yeah half the time it feels like they’re just trying to sell training
17:44:41 <audigex> AWS, Azure, Google are all a lot more intuitive but β€œgood” certainly wouldn’t come into it
17:44:48 <truebrain> efessel: At a certain point it told me to learn how to prompt an AI
17:44:50 <truebrain> which made me laugh
17:45:29 <truebrain> this is the world we now live in ... AIs telling us to prompt them better ...
17:46:18 <truebrain> anyway, reason for InstancePool over Instances via Terraform, is that an InstancePool automatically creates a new instance if another one crashes for what-ever reason
17:46:34 <truebrain> self-healing and shit πŸ™‚
17:47:06 <truebrain> `The number of NVMe drives to be used for storage. A single drive has 6.8 TB available.`. Lol.
17:47:43 <efessel> Oh and their block storage restrictions are weird. You *must* use at least 32gb for boot.
17:47:53 <efessel> And if you want to add an additional, minimum is 50gb
17:48:06 <truebrain> lol; well, it is dirt cheap anyway, so I guess it is fine
17:48:55 <efessel> Right, but if you spin up 4 instances, now you need to pay $ for detachable storage (well sure, poo poo for a few bucks a month lol)
17:49:26 <truebrain> from the quick math I did, it is 10 dollar on OCI, and 90 dollar on AWS, for the same hardware πŸ˜›
17:49:33 <truebrain> and that is just the instances
17:49:39 <truebrain> so there is some room in between those two, I guess πŸ™‚
17:49:53 <efessel> Yeah hah not complaining at all, just weird to me
17:50:04 <truebrain> I was a bit annoyed the "Free Tier" storage is just for 2 block devices .. means you can't make a cluster of three (a cluster of two isn't a cluster, mind you) for free πŸ˜›
17:50:34 <efessel> Well if you can't spin up any instances, the point is moot
17:50:48 <truebrain> if you have the PayAsYouGo, you still have the "Free Tier" stuff πŸ™‚
17:50:59 <truebrain> so on my personal account I run 2 instances for free, but I have PAYG active πŸ™‚
17:51:05 <truebrain> I just can't make it three, as that will cost
17:51:19 <truebrain> not for the compute, but for the block storage πŸ˜›
17:51:35 <efessel> huh? you should be able to run 4 ampere for free, no?
17:51:42 <efessel> oh
17:51:42 <truebrain> computes, yes
17:51:44 <truebrain> block storages, no
17:52:01 <efessel> Thought they give you 200gb block storage for free
17:52:07 <truebrain> over 2 block devices
17:52:29 <truebrain> `Up to 2 block volumes, 200 GB total. Plus 5 volume backups.`
17:52:47 <efessel> hmm I didn't read that, where is that ?
17:53:10 <truebrain> https://www.oracle.com/cloud/free/#free-cloud-trial, click Storage
17:53:36 <truebrain> no clue if they actually check for the count, mind you. But it reads on the label you only get 2 free
17:54:41 <efessel> https://docs.oracle.com/en-us/iaas/Content/FreeTier/freetier_topic-Always_Free_Resources.htm#blockvolume
17:54:52 <efessel> hmmm I don't see it in this one
17:55:31 <truebrain> https://cdn.discordapp.com/attachments/1008473233844097104/1260655660157108344/image.png?ex=66901c93&is=668ecb13&hm=ac414fd6a98eecb78cab9d4b990b57431210b937d9e94f19401748ad0ad2e2d8&
17:55:44 <efessel> Yes, I see that from your link
17:56:10 <efessel> but that's the only place I've ever seen that. Wonder if it's changed or a typo
17:56:14 <truebrain> haha, owh, the sneaky extra conditions ... "in the home region" ... lolz
17:56:29 <truebrain> and you cannot change your home region after creation, ofc πŸ˜›
17:56:54 <efessel> Yeah, and I set my home region after finding out there's no AMD Micro instances available 😦
17:57:43 <truebrain> but if the "Two" is old, that would be nice. Means I can run a cluster of three for free πŸ˜„
17:58:10 <efessel> I'm checking my expenses, since I've provisioned three...
17:58:46 <efessel> yup - still at $0
17:58:47 <efessel> https://cdn.discordapp.com/attachments/1008473233844097104/1260656481813135390/image.png?ex=66901d57&is=668ecbd7&hm=283173ea45d924de286d430c3d3b0d581bffe09a6e6d8dcb2c1292fb099d0c8b&
17:58:55 <truebrain> very interesting
17:59:06 <truebrain> also, I guess it means everything will be running in Amsterdam for OpenTTD πŸ˜›
17:59:11 <truebrain> as that is the home region, it seems πŸ™‚
17:59:16 <truebrain> not the best location, compute-wise, but what-ever
17:59:43 <efessel> but maybe they don't consider boot volumes in there
17:59:44 <truebrain> (no zone redundancy in Amsterdam; in Frankfurt there are 3 zones)
18:00:34 <efessel> maybe ask their AI support to clarify
18:00:39 <truebrain> hahaha
18:00:40 <truebrain> no tnx πŸ˜›
18:03:02 <truebrain> `Your account provisioning is in progress.`
18:03:06 <truebrain> that also takes for ever ...
18:03:41 <efessel> Oh yeah - took 8 hours for me I believe.
18:03:55 <efessel> I've read for some people it's taken days
18:04:24 <truebrain> upgrade requested at 12:51 .. so 7 hours in already πŸ™‚
18:05:13 <truebrain> okay, I think the cloud-init can just be done via metadata .. something to try out tomorrow, I guess πŸ˜„
18:07:26 <efessel> Still using Nomad?
18:07:31 <truebrain> yup
18:07:52 <truebrain> Although I would like something a bit easier to manage
18:07:55 <truebrain> but never found any πŸ™‚
18:08:03 <efessel> I thougth Nomad was the easier alternative to K8s πŸ™‚
18:08:08 <truebrain> it absolutely is
18:08:24 <truebrain> like with kubernetes you have three issues: the network/storage, the cluster itself, and the deployments within
18:08:28 <truebrain> with nomad you only have the last one
18:08:41 <truebrain> but they have their own language, which is just a bit annoying to quickly do things
18:08:49 <truebrain> as I always forget their syntax, and have to spend N time reading up again
18:08:57 <truebrain> I rather have something I can just send docker-compose files to or what-ever
18:09:05 <truebrain> something a bit more mainstream
18:09:20 <truebrain> (and no, `helm` and `kustomize` both suck balls too πŸ˜› )
18:09:45 <efessel> I agree, docker-compose would be very nice
18:10:01 <truebrain> You now have software like coolify, which is basicaly self-hosting Vercel
18:10:10 <truebrain> which is nice, but not yet cluster-ready in a sane way
18:12:42 <efessel> I'm still using k3s with success. Agree though - helm/kustomize is a pain, but I think I have them managed via terraform and gitops pretty well. Using ArgoCD to manage the state of the cluster
18:13:09 <truebrain> ArgoCD is nice; if you need to use k8s, ArgoCD is the way to cope πŸ™‚
18:13:10 <efessel> k3s includes traefik which works well
18:15:36 <efessel> also it comes with local storage provisioner, but that won't work across nodes
18:15:49 <truebrain> which makes it non-HA, ofc πŸ™‚
18:15:59 <truebrain> haven't checked what OCI has for that
18:16:18 <efessel> They apparently give you one free k8s service
18:18:06 *** gelignite has quit IRC (Read error: Connection reset by peer)
18:20:16 *** gelignite has joined #openttd
18:49:50 <peter1139> Right.
18:50:38 <peter1139> Helicopters expiring prevent the game from continuing?
18:51:25 <audigex> peter1139: It’s certainly an inconvenience if you’re using them and suddenly you can’t buy any more
18:59:44 <peter1139> No, it *stops* the game.
19:01:00 *** nielsm has joined #openttd
19:03:47 *** Ox7C5 has quit IRC ()
19:10:21 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12853: Codechange: Replace default overrides arrays with std::pair and use range-for. https://github.com/OpenTTD/OpenTTD/pull/12853#pullrequestreview-2170010360
19:12:48 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12851: Codefix: Potential unterminated string returned from convert_to_fs. https://github.com/OpenTTD/OpenTTD/pull/12851#pullrequestreview-2170014932
19:14:24 <audigex> peter1139: If you’re referring to the last reply on the forum I think that’s just a phrasing issue, they mean the game can’t continue *with helicopters*. It’s clumsy wording but they obviously aren’t saying the game stops
19:20:05 <andythenorth> Is there some football to watch?
19:21:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #12853: Codechange: Replace default overrides arrays with std::pair and use range-for. https://github.com/OpenTTD/OpenTTD/pull/12853#pullrequestreview-2170010360
19:21:24 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #12853: Codechange: Replace default overrides arrays with std::pair and use range-for. https://github.com/OpenTTD/OpenTTD/pull/12853
19:30:33 <truebrain> efessel: I can't get boot volumes lower than 50GB. How did you do 32?
19:30:51 <efessel> I got the # wrong - I think its 47
19:31:16 <truebrain> 46.6GB it says, but I can't get it below 50 πŸ˜›
19:31:22 <truebrain> but okay, not 32 😦 Darn ..
19:31:40 <efessel> Boot volume 46.6, but detachable block storage minimum is 50
19:31:52 <truebrain> such a waste of space/money
19:32:30 <truebrain> their NLB is funny. Proxy Support? Only for TCP and IPv4. Not for anything else. Shoo-shoo.
19:32:38 <truebrain> So I need to do the same as on AWS, run my own NLB
19:32:42 <truebrain> at least it is cheaper πŸ˜›
19:32:55 <efessel> Yeah, I read up you did a lot of IPv6 right?
19:33:15 <truebrain> all our services are available on both IPv4 and IPv6
19:33:38 <truebrain> and I don't want my logs to show internal Oracle Cloud IPs; that makes a lot of things much harder πŸ˜›
19:33:45 <truebrain> so we use Proxy Protocol to still see the original IP
19:33:51 <truebrain> but .. OCI is like: only in 25% of the cases
19:34:31 <truebrain> no biggy, I have all the code ready for it anyway
19:44:39 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #12853: Codechange: Replace default overrides arrays with std::pair and use range-for. https://github.com/OpenTTD/OpenTTD/pull/12853#pullrequestreview-2170066914
19:56:39 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12853: Codechange: Replace default overrides arrays with std::pair and use range-for. https://github.com/OpenTTD/OpenTTD/pull/12853
19:57:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #12851: Codefix: Potential unterminated string returned from convert_to_fs. https://github.com/OpenTTD/OpenTTD/pull/12851
19:57:50 *** amal[m] has quit IRC (Quit: Client limit exceeded: 20000)
19:58:25 *** FLHerne has quit IRC (Quit: There's a real world out here!)
20:07:48 *** FLHerne has joined #openttd
20:58:34 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
21:03:15 *** Flygon has quit IRC (Read error: Connection reset by peer)
21:05:07 *** gelignite has quit IRC (Quit: Stay safe!)
21:30:38 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:44:02 *** tokai|noir has joined #openttd
21:44:02 *** ChanServ sets mode: +v tokai|noir
21:50:48 *** tokai has quit IRC (Ping timeout: 480 seconds)