IRC logs for #openttd on OFTC at 2025-10-30
            
01:58:39 *** tokai has joined #openttd
01:58:39 *** ChanServ sets mode: +v tokai
02:05:34 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
02:05:59 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit opened pull request #14744: Feature: Signs, waypont and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744
02:19:22 <DorpsGek> [OpenTTD/OpenTTD] EmperorJake commented on pull request #14744: Feature: Signs, waypont and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3465867701
02:23:02 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit commented on pull request #14744: Feature: Signs, waypont and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3465873764
03:00:05 *** herms2 has quit IRC (Quit: bye)
03:04:00 *** herms2 has joined #openttd
03:57:42 *** gnu_jj has joined #openttd
04:01:19 *** gnu_jj_ has quit IRC (Ping timeout: 480 seconds)
04:40:24 *** Zathras_11 has joined #openttd
04:43:56 *** Zathras has quit IRC (Ping timeout: 480 seconds)
05:20:13 *** keikoz has joined #openttd
06:02:33 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #14744: Feature: Signs, waypont and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3466245346
07:08:34 *** tokai has quit IRC (Ping timeout: 480 seconds)
07:24:51 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14744: Feature: Signs, waypont and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3466433489
07:50:25 <LordAro> memes
07:54:38 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3466518616
07:58:33 <andythenorth> coffee?
08:06:08 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#pullrequestreview-3397884888
08:06:50 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#pullrequestreview-3397886954
08:25:14 *** Borg has joined #openttd
08:35:02 *** toktik has joined #openttd
08:50:32 <mmtunligit> Thanks aro, I thought I had those braces but I guess I forgot. And as for the variable I’d tried declaring it like that before but I got errors, I didn’t have the ‘const’ though which I assume would affect it
08:58:36 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit commented on pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3466735315
09:07:09 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3466775404
09:19:37 <_zephyris> ☕
09:29:18 <LordAro> https://github.com/curl/curl/issues good news, curl is finished
09:37:18 <_zephyris> If anyone with NML commit rights is bored today, would you mind merging #393? https://github.com/OpenTTD/nml/pull/393
09:39:04 <Borg> okey, stuff done :) I added nice tri-color line to indicate relation between signals.. stuff works :>
09:42:20 *** toktik has quit IRC (Remote host closed the connection)
09:48:53 <DorpsGek> [OpenTTD/OpenTTD] EmperorJake commented on pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744#issuecomment-3467014039
09:54:58 <pickpacket> LordAro: no more protocols are allowed to be invented
10:10:28 *** toktik has joined #openttd
10:15:38 *** SigHunter_ has joined #openttd
10:17:21 *** SigHunter has quit IRC (Ping timeout: 480 seconds)
10:26:02 *** SigHunter has joined #openttd
10:30:25 *** SigHunter_ has quit IRC (Ping timeout: 480 seconds)
10:33:11 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744
10:39:20 <xarick> ih
10:45:55 <kuhnovic> elloh
11:34:26 <xarick> make yapf faster!
11:36:23 <xarick> would a safe btree shenanigans help?
11:40:20 <xarick> has JGR made some changes to yapf regarding containers and hashtables etc...?
12:30:37 *** lobster has quit IRC (Read error: Connection reset by peer)
12:31:19 <kuhnovic> Why does YAPF need to become faster? Is it a bottleneck?
12:32:37 *** lobster has joined #openttd
12:48:10 <Borg> does Ship still do circles when they have single order at destination dock? (asking for newest OpenTTD)
12:49:04 <Borg> an idea popped in.. to send such vehicle to depot...
12:55:44 <andythenorth> kuhnovic: if I start a ship in an otherwise empty game, my game ffwd rate drops from about 100x to about 100x
12:56:12 <andythenorth> must be YAPF
12:56:13 <Borg> andythenorth: YAPF or NPF?
12:56:21 <andythenorth> lol YAPF of course
12:56:24 <andythenorth> NPF isn't a thing
12:56:30 <Borg> I actually still use NPF for ships
12:56:44 <Borg> I see no reason to use YAPF for ships
12:57:02 <Borg> YAPF shines for road/rails
12:57:10 <Borg> and let it do so there
12:57:11 <_glx_> NPF doesn't exist anymore
12:57:17 <Borg> oh lol ;D
12:57:50 <Borg> so, OpenTTD is left w/ one path finding now?
12:57:55 <andythenorth> I mean....the colour news window opening completely tanks FFWD rate, but let's look again at making YAPF more fragile and broken
12:58:06 <andythenorth> which is where 99% of optimisations seem to focus
12:58:11 <_glx_> Yes, no need for other PF
12:58:13 <alfagamma7> Hmm
12:58:19 <alfagamma7> I want to test something
12:58:22 <andythenorth> meanwhile FIRS tile animation was dropping FPS by approx 50%
12:58:23 <Borg> _glx_: congratulations.. you just fucked up the game
12:58:34 <andythenorth> but let's optimise arcane pathfinding cases
12:58:39 <Borg> if you think you can build ONE pathfinder for everything...
12:58:45 <alfagamma7> Do custom emojis being sent from here appear on IRC ?
12:58:48 <Borg> I think you are mistaken.. but okey.. what do I know
12:58:53 <alfagamma7> Sending one now
12:59:00 <alfagamma7> :Aaarg:
12:59:04 <_glx_> And we removed the need for buoys
12:59:31 <Borg> anyway.. can anyone confirm my question about ships w/ one order?
13:06:16 <andythenorth> borg you don't have the game running? 🙂
13:07:16 <andythenorth> yes ships with one dock order still circle back and forth to the dock
13:08:25 <Borg> andythenorth: ok thanks
13:08:37 <Borg> so lets fix it....
13:11:21 <andythenorth> what's to fix? 🙂
13:11:27 <andythenorth> it's correct behaviour for the orders
13:12:55 <Borg> but its hella annoying
13:13:07 <Borg> for any other vehicle.. they will sit on loading
13:13:14 <Borg> ships will horn and do circles
13:13:27 <Borg> im going to send them to depot (the one that is in orders, maintenance)
13:13:37 <Borg> or find_closest_depot() and send them there
13:14:17 <Borg> and they will stop there.. until player fix its order list
13:14:47 <xarick> kuhnovic: i'm looking for alternative stuff, but unsure if I'm even checking the right things <https://github.com/abseil/abseil-cpp/blob/master/absl/container/node_hash_set.h>
13:33:41 <kuhnovic> xarick: I've looked into that at some point. You can go full rabbit hole here https://martin.ankerl.com/2022/08/27/hashmap-bench-01/
13:35:34 <kuhnovic> But if you do some profiling you'll find that the map performance isn't really a bottleneck. I know JGRPP uses a different map, I can imagine that's needed because of the enormous map sizes it supports.
13:44:26 <andythenorth> FIRS is the bottleneck
13:44:59 <andythenorth> also, it may have been fixed, but opening the train window with Iron Horse dumped FPS to about 20% of usual
13:47:12 <andythenorth> yup still happens
13:47:40 <andythenorth> and there's some weird issue where the news queue affects FPS, but I can't remember what was counter-intuitive about it
13:47:43 <andythenorth> or the repro
13:47:59 <andythenorth> but anyway, carry on optimising corner cases 😛
13:52:31 <peter1138> Well.
13:52:54 <kuhnovic> Bad horse
13:53:45 <peter1138> correct horse battery staple?
13:55:19 <peter1138> But also, working with a 7ish year old version of the game.
14:05:17 <kuhnovic> "the good old days"
14:06:04 <andythenorth> when NPF still existed 🙂
14:06:24 <andythenorth> lunch?
14:09:19 <alfagamma7> quite
14:28:51 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
14:38:44 <_jgr_> Replacing containers with ones which perform better and/or use less memory seems like a free lunch to me, I'm not so worried if it isn't an absolutely critical path
14:42:54 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1433466196568576162/Captura_de_ecra_2025-10-30_143655.png?ex=6904caed&is=6903796d&hm=68ec7aabe7bfa0ea96518fd62355a7f394912fb5913c3fe776f1d2665ff5316d&
14:42:54 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1433466197005045953/Captura_de_ecra_2025-10-30_144037.png?ex=6904caed&is=6903796d&hm=40b791e593f59c1554a492a712c8c97a1dd543d41f482913415b520be197b625&
14:42:54 <xarick> JGR's valuate is just fast
14:43:11 <xarick> or rather... the AddItem thing
14:43:22 <xarick> SetValue
14:44:53 <xarick> AddOrSetItem
15:05:48 <xarick> GetAllRailTypes doesn't belong to ScriptRail
15:05:53 <xarick> but to ScriptEngine
15:06:04 <xarick> someone made a mistake
15:07:00 <andythenorth> file a PR 🙂
15:07:12 <_jgr_> No, see the recent engine rail types PR
15:11:57 <xarick> no, it's a mistake
15:12:46 <xarick> <https://github.com/OpenTTD/OpenTTD/commit/313c6c45aa307835f4eb141e11c3bf68038bc3aa#diff-08e7a64742912ad1b5855d1a29ecdd1000feb2ac23812005d93917129fa150f5>
15:12:46 <xarick> ScriptEngine::GetAllRailTypes, but the changelog says * \li AIRail::GetAllRailTypes
15:14:24 <_jgr_> The changelog is not important, but you can PR a fix for that, sure
15:14:56 <xarick> even regression uses `local railtypes = AIEngine.GetAllRailTypes(i);`
15:16:41 <peter1138> If the regression test used the wrong thing it wouldn't work, so yes, of course.
15:18:36 <andythenorth> I had lunch
15:19:04 <peter1138> Congratulations.
15:19:10 <xarick> this is weird to return it as a bitmap
15:19:22 <xarick> there is AIRailTypeLists
15:19:26 <peter1138> That commit was changed.
15:19:29 <andythenorth> thanks peter1138
15:19:37 <xarick> ah
15:19:38 <andythenorth> just keeping the channel actually relevant
15:19:40 <andythenorth> and productive
15:19:50 <peter1138> Oh right. Bike ride?
15:20:04 <andythenorth> could be
15:20:05 <mmtunligit> i just got back from one
15:20:17 * andythenorth making grfs
15:20:24 <peter1138> https://github.com/OpenTTD/OpenTTD/pull/14617
15:20:39 <peter1138> I didn't touch the changelog though :-)
15:21:09 * rito12_51026 watching the storm through the window
15:23:23 <Borg> okey! fixed
15:23:30 <Borg> Ships will head to nearest depot...
15:23:35 <peter1138> By rebase to master?
15:23:38 <peter1138> *rebasing
15:23:48 <Borg> of course it took bloody long to figure out where to put the code
15:23:59 <peter1138> Unfathomable.
15:24:28 <andythenorth> I am unable to fathom
15:24:37 <andythenorth> how many fathoms deep is OpenTTD sea?
15:24:59 <peter1138> It's not. It's all shallow.
15:25:17 <andythenorth> maybe it's wide
15:25:47 <Borg> the side effect is.. if its unable to find Depot it will get lost ;)
15:25:50 <Borg> not bad too...
15:25:56 <Borg> at least it will not run circles horning
15:28:52 <Borg> _jgr_: Signal Links works like a charm :) thanks for hints
15:29:12 <Borg> if you want the code (changes arent that invasive), I can hand you a diff
15:34:55 <Rubidium> andythenorth: an unfathomable amount I'd say
15:35:25 <andythenorth> indeed
15:36:02 <xarick> there actually exists an AIRail.GetRailType but that's for tiles
15:36:30 <xarick> AIEngine.GetRailType is for engines
15:43:05 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened pull request #14745: Doc 313c6c4: [Script] GetAllRailTypes and GetRailType are from ScriptEngine https://github.com/OpenTTD/OpenTTD/pull/14745
15:47:23 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1433482424364433418/image.png?ex=6904da0a&is=6903888a&hm=0d40e0ecbe73550472062637c80dc8de86ed17c52014ba372d5d65c96357dd50&
15:47:23 <xarick> doxygen doesn't put a link
15:51:54 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14745: Doc 313c6c4: [Script] GetAllRailTypes and GetRailType are from ScriptEngine https://github.com/OpenTTD/OpenTTD/pull/14745#pullrequestreview-3400364827
16:05:15 <xarick> hmm
16:23:39 <xarick> ram prices skyrocketing thx to AI 🙁
16:39:58 <Rubidium> if you wouldn't have used AI, the prices would've gone up less...
17:10:17 <xarick> 🙁
17:21:02 <mmtunligit> if i have a really long test in an if statement can i split it across two lines?
17:26:50 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #14311: Add: [Script] ScriptList::[Get/Set]SorterType and ScriptList::[Get/Set]SorterDirection https://github.com/OpenTTD/OpenTTD/pull/14311
17:35:20 <LordAro> mmtunligit: yes
17:35:29 <LordAro> not that OTTD really has line length limts
17:35:32 <LordAro> +i
17:36:47 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #14745: Doc 313c6c4: [Script] GetAllRailTypes and GetRailType are from ScriptEngine https://github.com/OpenTTD/OpenTTD/pull/14745
17:42:02 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14740: Codechange: Use helper function for company recolour offset https://github.com/OpenTTD/OpenTTD/pull/14740#pullrequestreview-3400869189
17:44:43 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14735: Codefix: Clear unused map bits in MakeXXX() map helpers. https://github.com/OpenTTD/OpenTTD/pull/14735#pullrequestreview-3400877463
17:49:39 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14736: Codechange: Make driver probe behave the same as loading by name. https://github.com/OpenTTD/OpenTTD/pull/14736#pullrequestreview-3400899619
17:51:25 <xarick> why can't we build bridges over depots?
17:52:06 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14732: Fix #14701: Company colour remap for sprites in badge filter dropdowns. https://github.com/OpenTTD/OpenTTD/pull/14732#pullrequestreview-3400913113
17:54:27 <xarick> sqvm overheard is quite intense 🙁
17:57:34 <andythenorth> is it more or less than FIRS?
18:03:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725#pullrequestreview-3400918018
18:04:31 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #14311: Add: [Script] ScriptList::[Get/Set]SorterType and ScriptList::[Get/Set]SorterDirection https://github.com/OpenTTD/OpenTTD/pull/14311
18:08:54 <talltyler> I don’t know of a technical reason we can’t bridge over depots, just “nobody has opened a PR yet”
18:09:24 <talltyler> Could likely follow the “bridges over locks” PR as an example of what needs changing
18:33:26 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744
18:40:57 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14744: Feature: Signs, waypoint and station names may be moved https://github.com/OpenTTD/OpenTTD/pull/14744
19:32:22 <DorpsGek> [OpenTTD/nml] zephyris commented on pull request #393: Add: Support for Action5 type 1B bridge deck overlays https://github.com/OpenTTD/nml/pull/393#issuecomment-3469744452
19:32:44 <andythenorth> naptime?
19:35:45 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14736: Codechange: Make driver probe behave the same as loading by name. https://github.com/OpenTTD/OpenTTD/pull/14736#issuecomment-3469756152
19:40:08 <rito12_51026> not yet
19:41:13 <rito12_51026> there is stuff to do
19:53:47 <xarick> bad code style 🙂
19:57:15 <locosage> wait, was GRFv9 not implemented in 15.0?
19:58:21 <locosage> I thought it was merged but #13309 is closed without any comment
20:01:16 <_jgr_> Some GRF features have been merged but there's been no version bump
20:05:41 <locosage> breaking changes were the most interesting ones
20:06:54 <digitalfox> Any dev avalaible with commit powers to make nml 393 happen? 🙂
20:06:54 <digitalfox> Please 🙏
20:09:48 <xarick> JGR made a codestyle issue
20:10:57 <_jgr_> xarick: You'll need to be a bit more specific...
20:11:30 <xarick> perhaps not
20:12:03 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1433549030696157215/image.png?ex=69051812&is=6903c692&hm=63b95974f08fd49a7f32b394d25dfca7c716b4400f230fc649ef65d5542b45e3&
20:12:12 <xarick> !res.second
20:13:06 <xarick> ah, second is a bool apparently
20:13:20 <xarick> my bad
20:22:34 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14736: Codechange: Make driver probe behave the same as loading by name. https://github.com/OpenTTD/OpenTTD/pull/14736
20:22:55 *** Wolf01 has joined #openttd
20:30:22 <xarick> is RemoveList tested in regression?
20:31:39 <xarick> yes
20:43:42 *** toktik has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** Smedles has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** WormnestAndroid has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** Extrems has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** felix has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** dwfreed has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** Ttech has quit IRC (charon.oftc.net resistance.oftc.net)
20:43:42 *** twpol has quit IRC (charon.oftc.net resistance.oftc.net)
20:44:39 <peter1138> Nap?
20:44:51 *** toktik has joined #openttd
20:44:51 *** Smedles has joined #openttd
20:44:51 *** WormnestAndroid has joined #openttd
20:44:51 *** Extrems has joined #openttd
20:44:51 *** felix has joined #openttd
20:44:51 *** dwfreed has joined #openttd
20:44:51 *** Ttech has joined #openttd
20:44:51 *** twpol has joined #openttd
20:45:31 *** ChanServ sets mode: +o orudge
20:45:31 *** ChanServ sets mode: +v peter1138
20:45:31 *** ChanServ sets mode: +v planetmaker
20:45:31 *** ChanServ sets mode: +v DorpsGek
20:50:35 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725
20:51:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14746: Codechange: Use enum/EnumBitSet for livery in use flags. https://github.com/OpenTTD/OpenTTD/pull/14746
20:51:19 <peter1138> Hmm.
20:54:39 <andythenorth> nap?
20:56:32 <andythenorth> 393 is https://github.com/OpenTTD/nml/pull/393
21:05:09 *** Borg has quit IRC (Quit: leaving)
21:07:19 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1433562940770095165/image.png?ex=69052507&is=6903d387&hm=33bf2c481a60e51ca30df5cfee5f9b1e68e35310457781d98b84724d9519fa2a&
21:07:19 <xarick> _jgr_: why was this done this way? Wouldn't this work:
21:07:19 <xarick> ```for (const auto &it : list->items) {
21:07:19 <xarick> this->AddOrSetItem(it.first, it.second);
21:07:19 <xarick> }```
21:13:10 <_jgr_> xarick: It ought to work, yes
21:16:52 <_glx_> why the local variable ?
21:17:17 <locosage> hm, industry var66 doesn't seem to return closest town as it says
21:17:37 <locosage> or, rather, it picks closest by manhattan distance even though result is euclidean
21:19:32 <_glx_> it uses same closest for var65 and var66 I think
21:22:05 <_jgr_> git blame for the temporary points to 1f1345de098 from more than 5 years ago, I've had no reason to change it
21:25:05 <_jgr_> The commit in master which removes it not even a month old yet
21:26:00 <_glx_> yeah, understandable
21:27:07 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14740: Codechange: Use helper function for company recolour offset https://github.com/OpenTTD/OpenTTD/pull/14740#issuecomment-3470331424
21:29:31 <xarick> auto is scary powerful
21:31:22 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
21:32:27 *** WormnestAndroid has joined #openttd
21:34:51 <rito12_51026> PeterNviaGitHub: eh
21:44:43 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
21:48:39 <xarick> RetargetIterator is a little bit... out of place
21:50:37 *** WormnestAndroid has joined #openttd
22:05:56 <peter1138> Never heard of it.
22:07:59 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725#pullrequestreview-3401809886
22:10:41 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit commented on pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725#pullrequestreview-3401828491
22:45:56 *** bryjen has joined #openttd
22:46:12 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1433587824174563472/image.png?ex=69053c34&is=6903eab4&hm=34e6ebf0afd856e0b82715f538198d58fa2305f4004fffe16e742454c08515da&
22:46:12 <xarick> how cursed is this coding style?
22:46:46 <xarick> it compiles
22:46:54 <xarick> regression tests pass
22:47:15 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:55:00 *** Flygon has joined #openttd
22:58:13 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725
23:05:54 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725
23:06:22 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14725: Change: Clamp terraform toolbar to main toolbar. https://github.com/OpenTTD/OpenTTD/pull/14725#pullrequestreview-3401994253
23:07:46 <xarick> something interesting with safe btree stuff
23:09:02 <xarick> AddList performance is faster when both lists are SORT_BY_ITEM
23:11:13 <xarick> ah, I see
23:11:20 <xarick> InitValues is not called
23:22:07 <peter1138> 22:46 < xarick> how cursed is this coding style?
23:22:09 <peter1138> Too cursed.
23:22:35 <peter1138> Don't accept or return auto as a function parameter unless it's a generic function.
23:36:42 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
23:45:24 *** Zathras_4 has joined #openttd