IRC logs for #openttd on OFTC at 2026-10-04
            
00:39:34 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
00:39:52 *** MinchinWeb[m] has joined #openttd
01:57:11 *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)
01:57:28 *** MinchinWeb[m] has joined #openttd
02:39:15 *** Flygon has joined #openttd
02:53:19 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
02:53:21 *** WormnestAndroid has joined #openttd
02:54:58 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
02:55:00 *** WormnestAndroid has joined #openttd
02:58:15 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
02:58:16 *** WormnestAndroid has joined #openttd
03:13:50 *** Phileman has joined #openttd
03:17:11 *** Philemon has quit IRC (Ping timeout: 480 seconds)
03:58:20 <DorpsGek> [OpenTTD/OpenTTD] perryprog commented on issue #16053: [Crash]: Screensaver mode crashes shortly after panning https://github.com/OpenTTD/OpenTTD/issues/16053
04:58:33 <Rubidium> jfkuayue: use it to come up with reasons not to use it?
05:04:58 *** wensimehrp has joined #openttd
05:04:58 <wensimehrp> jfkuayue: https://lore.kernel.org/linux-media/CAHk-=wi4zC+Ze8e+p3tMv8TtG_80KzsZ1syL9anBtmEh5Z40vg@mail.gmail.com/
05:04:58 <wensimehrp> > Linux is not one of those anti-AI projects, and if somebody has issues
05:04:58 <wensimehrp> > with that, they can do the open-source thing and fork it.
05:04:58 <wensimehrp> >
05:04:58 <wensimehrp> > Or just walk away.
07:17:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #16059: Codefix: [NewGRF] Only allow disabling original airports/industries https://github.com/OpenTTD/OpenTTD/pull/16059
07:21:31 *** Wolf01 has joined #openttd
07:57:34 <peter1138> /home/petern/src/openttd/src/core/pool_type.hpp:63:66: warning: β€˜((const PoolID<unsigned int, VehicleIDTag, 1044480, 1048575>*)this)[21].PoolID<unsigned int, VehicleIDTag, 1044480, 1048575>::value’ may be used uninitialized [-Wmaybe-uninitialized]
07:57:39 <peter1138> 63 | constexpr TBaseType base() const noexcept { return this->value; }
07:58:19 <peter1138> Other than removing -Wmaybe-uninitialized... what's needed to fix this :o
07:58:48 <peter1138> (gcc 14.2.0)
08:09:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060
08:10:40 *** MinchinWeb[m] has quit IRC (Remote host closed the connection)
08:10:56 *** MinchinWeb[m] has joined #openttd
08:22:22 *** jfs_ has joined #openttd
08:27:25 *** SigHunter has quit IRC (Remote host closed the connection)
08:31:46 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
08:33:20 *** SigHunter has joined #openttd
08:43:22 *** Wolf01 has joined #openttd
09:28:09 <peter1138> Tree classes though.
09:28:37 <peter1138> I used a bitmask for convience, but that's not exactly friendly for NewGRFs.
09:32:10 <reldred> treeeeees
09:32:26 <peter1138> A bitset is fixed length. A vector is variable length but can't be defined static constexpr (not a huge problem but)
09:34:55 <peter1138> The classes are only evaluated when generating tree tile types, so performance isn't a huge concern.
10:13:52 <peter1138> 4-letter labels?
10:23:27 <peter1138> More memory though.
10:24:00 <reldred> my normal facetious response would be 'memory is cheap' but it really isn't anymore 😒
10:24:47 <peter1138> It's a non-event here unless you were defining thousands of trees with thousands of classes :D
10:25:18 <reldred> depends how nuts zyliety goes with it once you get it into the general populations hands πŸ˜›
10:26:09 <peter1138> But 32-bit class mask is 4 bytes for every combination. 4-byte labels is 24 bytes for the vector, and then 4 bytes for each entry. There's 4 class lists per tree type so that's already 16 bytes -> 96 bytes, ZOMG!
10:27:46 <peter1138> Probably 32 different tree classes is fine anyway, but.
10:28:42 <peter1138> They're used to mix different but similar trees when building the tree tiles.
10:30:48 <peter1138> Might rename from class actually, as that traditionally is used for things like objects and stations to categorise in the UI.
10:44:45 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/df2fd61b29485dd9fdde1c5a18657212c9175617
10:44:46 <DorpsGek> - Update: Translations from eints (by translators)
11:00:21 <peter1138> Well, labels mean authors get to fight over what 4-byte labels mean instead of what limited individual bits mean.
11:26:15 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:26:17 *** WormnestAndroid has joined #openttd
11:55:59 <andythenorth> "days since treetypes" πŸ™‚
12:22:19 <peter1138> Hmm, doesn't appear to work :o
12:23:16 <peter1138> Oh, probably the height thing!
12:50:08 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1556287303540936774/image.png?backend=b2&ex=6ac39cff&is=6ac24b7f&hm=3e4f6bba3dc219175eb16d3a2cfda2cf1d9e40a4085b0f1377cb2671cfa1c124&
12:50:08 <peter1138[d]> Something might've gone wrong :o
12:50:48 *** notluke2578 has joined #openttd
12:50:48 <notluke2578> the landscape is just eepy
13:01:52 <peter1138> It was the NewGRF :)
13:43:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15923: Feature: [NewGRF] Treeeees https://github.com/OpenTTD/OpenTTD/pull/15923
13:57:07 <peter1138> Probably should do tree classes too.
13:57:24 <peter1138> To prepare for a Picker UI.
14:19:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #16061: Codefix: include stdafx.h before any other includes https://github.com/OpenTTD/OpenTTD/pull/16061
14:20:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #16061: Codefix: include stdafx.h before any other includes https://github.com/OpenTTD/OpenTTD/pull/16061#pullrequestreview-5406631762
14:20:43 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #16061: Codefix: include stdafx.h before any other includes https://github.com/OpenTTD/OpenTTD/pull/16061#pullrequestreview-5406632466
14:20:54 <LordAro> damn.
14:28:49 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060#pullrequestreview-5406657342
14:31:11 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060#pullrequestreview-5406665014
15:07:02 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #16061: Codefix: include stdafx.h before any other includes https://github.com/OpenTTD/OpenTTD/pull/16061
15:35:03 <peter1138> https://www.tt-forums.net/viewtopic.php?t=92915
15:35:05 <peter1138> That's a word salad.
15:35:10 <peter1138> "ECS pathfinder"?
15:38:01 * peter1138 wonders how to classify the default trees. Or just go with "Default" like stations :)
15:38:26 <peter1138> Would be nice to split rainforest and normal trees though.
15:44:55 <talltyler> Split desert trees (cacti) too?
15:45:00 <peter1138> Sure.
15:45:22 <peter1138> Hmm, setting classes for built-in stuff is a bit of a pain.
15:46:08 <talltyler> I guess this begs the question if you intend NewGRF authors to follow a similar pattern and split trees by where they can be planted. It does seem to follow the house picker showing town zones, but worth thinking about if you haven't.
15:47:24 <peter1138> No, classes are only for categorising within the picker UI. There are landscape and tropical zone flags to actually handle where things can be placed.
15:48:19 <peter1138> For houses I used town zones because there are no house classes (actually there is but it's entirely something different from the 4 byte labels)
15:48:46 <peter1138> We could add/have added house classes and use those instead.
15:50:54 <peter1138> Ah, we do wrangle classes for airports in a way that's workable for trees too.
15:57:05 <peter1138[d]> I haven't done flags for snow compatibility because only the arctic trees have snow versions, and all of them have snow versions.
15:57:22 <peter1138[d]> Variable snow-line would make things a bit awkward too.
16:23:51 <peter1138> Hmm, probably ought to add a badges property :p
16:32:12 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #16062: Fix: Don't show error when double-clicking depot or waypoint order https://github.com/OpenTTD/OpenTTD/pull/16062
16:50:49 *** babel_ttt has joined #openttd
16:50:49 <babel_ttt> https://cdn.discordapp.com/attachments/1008473233844097104/1556347873254187159/1791132577976.png?backend=b2&ex=6ac3d568&is=6ac283e8&hm=e988817125541a23edfff4c64e6756445ef0c49c3e17885a3121a6fab321e6f3&
16:50:49 <babel_ttt> Is there any way to solve this problem?
16:51:41 <peter1138> Draw larger tunnel entrances.
16:52:37 <peter1138> And shorter catenary poles.
16:53:13 *** tokai has joined #openttd
16:53:13 *** ChanServ sets mode: +v tokai
16:54:36 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1556348825810243705/image.png?backend=b2&ex=6ac3d64b&is=6ac284cb&hm=c0aff4048f511aa81ecbe09c636eaaa8651636e779c0b0a58f9fb600b9b03bb7&
16:55:41 <peter1138> Original tunnels use tricks to make the tunnel entrance look larger. The catenary poles are also quite a bit shorter.
17:11:22 <andythenorth> oops ` nmlc info: Train items: 11282/65420`
17:11:45 <andythenorth> adding a1 livery to each wagon in Horse grows that number fast πŸ˜›
17:56:04 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1556364297825685554/image.png?backend=b2&ex=6ac3e4b4&is=6ac29334&hm=13dd8850a95e25ade96899bfe13bdb1bfb5567290efffc6e6836c84fa74dc1e7&
17:56:04 <peter1138[d]> Something something something trees
17:56:34 <peter1138[d]> I added trees and didn't set a class, so they default to the first class, which is "Temperate".
17:56:43 <peter1138[d]> So probably "Default" is a better bet.
17:57:44 *** Flygon has quit IRC (Read error: Connection reset by peer)
17:58:47 <talltyler> Yeah, probably. Also Cactus should be Desert, maybe?
17:59:45 *** gelignite has joined #openttd
18:15:58 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #16062: Fix: Don't show error when double-clicking depot or waypoint order https://github.com/OpenTTD/OpenTTD/pull/16062#pullrequestreview-5407482114
18:21:02 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #16062: Fix: Don't show error when double-clicking depot or waypoint order https://github.com/OpenTTD/OpenTTD/pull/16062
18:22:01 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #16058: Change: [NewGRF] Implement airport tile 0xFF clearance check https://github.com/OpenTTD/OpenTTD/pull/16058#pullrequestreview-5407508131
18:26:22 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #16063: Feature: Reverse trains at waypoints https://github.com/OpenTTD/OpenTTD/pull/16063
18:33:03 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #16063: Feature: Reverse trains at waypoints https://github.com/OpenTTD/OpenTTD/pull/16063
18:35:30 <_zephyris> Looks nice. "Default" and "desert" work well
18:39:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #16063: Feature: Reverse trains at waypoints https://github.com/OpenTTD/OpenTTD/pull/16063#issuecomment-5983153993
18:44:57 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15971: Change: Convert lighthouse and transmitter build rules to object flags and properties https://github.com/OpenTTD/OpenTTD/pull/15971#pullrequestreview-5407526034
19:47:52 *** gelignite has quit IRC ()
19:50:33 <peter1138> Hmm, playing music from the ROLAND.CAT file, it's... a bit different.
20:10:51 <peter1138> Ah, maybe it's Roland MT-32.
20:13:56 <peter1138> Yeah, pretty sure it is.
20:35:06 <peter1138> Hmm, I guess ADLIB.CAT is a different format internally.
21:07:12 *** jfs_ has quit IRC (Ping timeout: 480 seconds)
21:34:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060#pullrequestreview-5406657342
21:34:28 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060
21:37:01 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:15:50 <DorpsGek> [OpenTTD/OpenTTD] perryprog approved pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060#pullrequestreview-5408504798
22:50:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060#issuecomment-5985307821
23:04:34 <DorpsGek> [OpenTTD/OpenTTD] perryprog commented on pull request #16060: Fix #16053: Crash due to undefined behaviour when leaving screenaver mode https://github.com/OpenTTD/OpenTTD/pull/16060#issuecomment-5985413858