IRC logs for #openttd on OFTC at 2026-07-26
#openttd / 2026 / 07 
            

00:26:52  *** WormnestAndroid has quit IRC (Remote host closed the connection)



00:26:58  *** WormnestAndroid has joined #openttd



01:47:18  *** Wormnest has quit IRC (Quit: Leaving)



02:40:27  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)



02:41:25  *** MinchinWeb[m] has joined #openttd



03:31:26  *** Phileman has joined #openttd



03:34:54  *** Philemon has quit IRC (Ping timeout: 480 seconds)



03:57:07  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)



03:58:35  *** MinchinWeb[m] has joined #openttd



03:59:18  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)



04:00:15  *** MinchinWeb[m] has joined #openttd



04:15:34  <DorpsGek> [OpenTTD/OpenTTD] SomewhatAn0nym0us opened issue #15912: [Bug]: Cursor snaps to bottom right of screen https://github.com/OpenTTD/OpenTTD/issues/15912



04:23:04  *** Flygon has quit IRC (Read error: Connection reset by peer)



05:02:06  *** tokai|noir has joined #openttd



05:02:06  *** ChanServ sets mode: +v tokai|noir



05:09:19  *** tokai has quit IRC (Ping timeout: 480 seconds)



05:17:22  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15913: Codechange: make AddressFunction more const, so less casting is required https://github.com/OpenTTD/OpenTTD/pull/15913



05:22:39  *** MinchinWeb[m] has quit IRC (Remote host closed the connection)



05:22:53  *** MinchinWeb[m] has joined #openttd



06:35:43  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15897: Codechange: reduce SaveLoad macros, improve type safety https://github.com/OpenTTD/OpenTTD/pull/15897



07:04:18  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15913: Codechange: make AddressFunction more const, so less casting is required https://github.com/OpenTTD/OpenTTD/pull/15913#issuecomment-5082467346



07:10:29  <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/2a33c65404bfa9810c63e30ea89b6df45358360e



07:10:30  <DorpsGek>   - Update: Translations from eints (by translators)



07:14:48  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15913: Codechange: make AddressFunction more const, so less casting is required https://github.com/OpenTTD/OpenTTD/pull/15913#issuecomment-5082501023



07:27:26  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15911: Change: Improve duplicate detection for company/president name changes https://github.com/OpenTTD/OpenTTD/pull/15911



07:30:06  *** Wolf01 has joined #openttd



07:53:17  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15911: Change: Improve duplicate detection for company/president name changes https://github.com/OpenTTD/OpenTTD/pull/15911#pullrequestreview-4781234110



07:55:55  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15911: Change: Improve duplicate detection for company/president name changes https://github.com/OpenTTD/OpenTTD/pull/15911



08:05:05  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15913: Codechange: make AddressFunction more const, so less casting is required https://github.com/OpenTTD/OpenTTD/pull/15913#issuecomment-5082649222



08:05:30  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15911: Change: Improve duplicate detection for company/president name changes https://github.com/OpenTTD/OpenTTD/pull/15911#pullrequestreview-4781251074



08:09:53  <peter1138> Rubidium, poolitem index is already const :)



08:18:54  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #15913: Codechange: make AddressFunction more const, so less casting is required https://github.com/OpenTTD/OpenTTD/pull/15913#pullrequestreview-4781268915



08:33:38  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15913: Codechange: make AddressFunction more const, so less casting is required https://github.com/OpenTTD/OpenTTD/pull/15913



08:33:59  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #15914: Codechange: Reduce scope of script_event_types include https://github.com/OpenTTD/OpenTTD/pull/15914



08:34:25  *** hpiirai has joined #openttd



08:34:25  <hpiirai> https://huggingface.co/spaces/HuggingFaceCode/in-the-stack “The Stack v3 is a 15.9 TB dataset of source code across 713 programming languages from 173M repositories, crawled from GitHub in 2025.” including openttd, of course



08:35:10  <andythenorth_> I'm there!



08:35:11  <andythenorth_> fame



08:35:15  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15914: Codechange: Reduce scope of script_event_types include https://github.com/OpenTTD/OpenTTD/pull/15914



08:35:58  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15910: Codechange: do not use SlCopy to skip bytes that should not be read any more https://github.com/OpenTTD/OpenTTD/pull/15910



08:36:41  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #15910: Codechange: do not use SlCopy to skip bytes that should not be read any more https://github.com/OpenTTD/OpenTTD/pull/15910#pullrequestreview-4781292603



08:45:04  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15914: Codechange: Reduce scope of script_event_types include https://github.com/OpenTTD/OpenTTD/pull/15914#pullrequestreview-4781304585



08:50:16  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 closed pull request #15868: Codechange: improve compile time type sanity checks for the saveload variables https://github.com/OpenTTD/OpenTTD/pull/15868



08:51:38  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 closed pull request #15884: Codechange: store all Labels in order, i.e. reverse the GrfID labels upon loading old saves https://github.com/OpenTTD/OpenTTD/pull/15884



08:51:41  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15884: Codechange: store all Labels in order, i.e. reverse the GrfID labels upon loading old saves https://github.com/OpenTTD/OpenTTD/pull/15884#issuecomment-5082789285



08:54:29  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15888: Codechange: store all Labels in 'GrfID' order https://github.com/OpenTTD/OpenTTD/pull/15888



09:02:53  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15915: Codefix: use ubuntu-latest over ubuntu-24.04 https://github.com/OpenTTD/OpenTTD/pull/15915



09:12:31  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15857: Codechange: Deduplicate foundation sprite code https://github.com/OpenTTD/OpenTTD/pull/15857



09:16:19  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15904: Fix #15902: NewGRF GRF ID checks failing https://github.com/OpenTTD/OpenTTD/pull/15904



09:16:22  <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #15902: [Bug]: NewGRF check if active (or Version) fails https://github.com/OpenTTD/OpenTTD/issues/15902



09:17:00  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15911: Change: Improve duplicate detection for company/president name changes https://github.com/OpenTTD/OpenTTD/pull/15911



09:17:13  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15914: Codechange: Reduce scope of script_event_types include https://github.com/OpenTTD/OpenTTD/pull/15914



09:26:06  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15910: Codechange: do not use SlCopy to skip bytes that should not be read any more https://github.com/OpenTTD/OpenTTD/pull/15910



09:27:16  *** sgtStrom has joined #openttd



09:28:09  *** sgtStrom has quit IRC (Remote host closed the connection)



09:48:41  <peter1138> Okay, so I got the smallmap button layout stuff working by introducing a widget wrapper. This allows an instance of a widget to appear multiple times in the hierarchy.



09:49:18  <peter1138> That then allows the one column and the two column layouts to be regular widget layouts that wrap the real buttons.



09:53:43  <peter1138> However I can't help feeling it's a bit wrong.



10:10:32  <Rubidium> would it maybe make sense to make a generalisation of NWID_HORIZONTAL and NWID_VERTICAL, which choses the best space consumption given some 'external' limits? Might also be way more difficult than writing this idea down, as you need to somehow need to know the size of your neighbours before being able to choose the best layout. Though, you might have 'fixed' that question already in your branch



10:15:39  <peter1138> Yes, minimum widget size is a problem, I was able to work around it because the NWidgetSmallmap already exists.



10:19:00  <peter1138> In this case there is also the additional complexity that it's actually two vertical containers with a separator, so both would need to know the available size and coordinate.



10:26:08  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15916: Codechange: add Bool and Label aliases to VarFileType to denote intent https://github.com/OpenTTD/OpenTTD/pull/15916



11:11:38  <peter1138> Hm.



11:12:50  * peter1138 has an idea.



11:12:54  <peter1138> (Oh no.)



11:14:19  <peter1138> During AssignSizePosition, we know what size is going to be used. We can ask the widget what size to use based on that size.



11:15:03  <peter1138> By default that would be the same size. If the size is tall enough, we can return a narrower width.



11:15:44  * peter1138 tests.



11:21:42  <peter1138> Might not work with nesting though.



11:28:57  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #15915: Codefix: use ubuntu-latest over ubuntu-24.04 https://github.com/OpenTTD/OpenTTD/pull/15915#pullrequestreview-4781562292



11:29:46  *** squirejames has quit IRC (Quit: User went offline on Discord a while ago)



11:30:40  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15915: Codefix: use ubuntu-latest over ubuntu-24.04 https://github.com/OpenTTD/OpenTTD/pull/15915



11:31:25  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15779: Change: migrate from SDL2 to SDL3 https://github.com/OpenTTD/OpenTTD/pull/15779



11:33:55  <peter1138> :(



11:58:23  <peter1138> Okay, so...



11:59:41  <peter1138> Now it's something a bit more like the custom toolbar container.



12:09:02  <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885#pullrequestreview-4739593180



12:09:05  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885



12:10:47  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885#issuecomment-5083406251



12:13:18  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885



12:13:36  <peter1138> Such noise.



12:14:20  <peter1138> Propogating GetPreferredSizeForSize() is probably a bit complicated, so I didn't bother.



12:16:15  <peter1138> With a better definition, it might be possible to make the custom container more generic.



12:16:18  <peter1138> But, uh...



12:37:48  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15897: Codechange: prefer functions over SaveLoad macros https://github.com/OpenTTD/OpenTTD/pull/15897



12:38:41  <DorpsGek> [OpenTTD/OpenTTD] arikover opened issue #15917: [Bug]: Only one Rail/Road type of each category available https://github.com/OpenTTD/OpenTTD/issues/15917



12:45:19  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885#pullrequestreview-4781757067



12:47:17  <peter1138> Oh, I messed up a merge and the comit title is wrong :(



12:47:19  <peter1138> +m



12:48:12  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885



13:08:22  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #15917: [Bug]: Only one Rail/Road type of each category available https://github.com/OpenTTD/OpenTTD/issues/15917



13:22:13  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15494: Change: account for driving cabs in driving backwards conditional order https://github.com/OpenTTD/OpenTTD/pull/15494



13:22:29  <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885#pullrequestreview-4781822539



13:23:44  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15485: Change: [NewGRF] xor reversed and driving backwards flags for modflags bit 8 https://github.com/OpenTTD/OpenTTD/pull/15485#issuecomment-5083658423



13:24:56  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885



13:27:37  *** Flygon has joined #openttd



13:28:10  *** Wolf01 has quit IRC (Ping timeout: 480 seconds)



13:56:03  <DorpsGek> [OpenTTD/OpenTTD-git-hooks] certator closed pull request #23: Codechange: fix indentation https://github.com/OpenTTD/OpenTTD-git-hooks/pull/23



14:06:52  <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885#pullrequestreview-4781907178



14:09:51  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885#pullrequestreview-4781911625



14:20:45  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15885: Change: Move small map buttons to side of window https://github.com/OpenTTD/OpenTTD/pull/15885



14:37:19  <peter1138> Might make sense to reverse the order that (un)flatten use, and then byteswap only on the rare cases that it's the other way around.



14:37:32  <peter1138> Basically `ci.unqiue_id`, as far as I can tell.



14:40:12  *** Wolf01 has joined #openttd



14:41:35  <Rubidium> makes a lot of sense



14:47:37  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #15918: Fix: Change the order that (Un)FlattenNewGRFLabel convert to/from uint32_t https://github.com/OpenTTD/OpenTTD/pull/15918



14:48:01  <peter1138> ^^



14:51:19  <peter1138> Not entirely sure everything is covered yet.



14:51:41  <peter1138> In particular GrfID<->script calls



14:51:52  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)



14:52:04  *** MinchinWeb[m] has joined #openttd



14:55:30  *** andythenorth_ has quit IRC (Quit: Connection closed for inactivity)



15:02:45  <Rubidium> sadly the script stuff needs byteswapping now compared to beta2



15:15:00  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15918: Fix: Change the order that (Un)FlattenNewGRFLabel convert to/from uint32_t https://github.com/OpenTTD/OpenTTD/pull/15918#issuecomment-5084084770



15:20:49  <peter1138> Hmm, std::byteswap on SQInteger seems unhappy.



15:21:33  <peter1138> Ah, 64 bit.



15:35:55  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15918: Fix: Change the order that (Un)FlattenNewGRFLabel convert to/from uint32_t https://github.com/OpenTTD/OpenTTD/pull/15918



15:35:58  <peter1138> Possibly



15:49:52  *** Wormnest has joined #openttd



16:16:39  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15918: Fix: Change the order that (Un)FlattenNewGRFLabel convert to/from uint32_t https://github.com/OpenTTD/OpenTTD/pull/15918#pullrequestreview-4782124814



16:31:15  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15907: Fix #15905: dual-headed engines show correct capacity when sorting purchase list https://github.com/OpenTTD/OpenTTD/pull/15907#issuecomment-5084369743



16:47:30  *** andythenorth_ has joined #openttd



16:47:42  <andythenorth_> oops



16:47:53  <andythenorth_> this irc client doesn't auto-reconnect :P



16:56:50  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #15903: Fix: potential off-by-one out of bounds read during saving https://github.com/OpenTTD/OpenTTD/pull/15903#pullrequestreview-4782187950



17:17:22  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15903: Fix: potential off-by-one out of bounds read during saving https://github.com/OpenTTD/OpenTTD/pull/15903



17:48:07  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15919: Codechange: use ContiguousContainer for SlCopy and extract VarMemType https://github.com/OpenTTD/OpenTTD/pull/15919



17:52:13  <andythenorth_> hmm does a single unit train drive backwards?



17:52:17  <andythenorth_> _assumes so_



17:55:22  <peter1138> Yes, at full speed no less.



17:55:30  <peter1138> Might be considered a bug for steam engines.



17:55:46  <peter1138> (Or indeed other engines)



17:57:36  <andythenorth_> _generating tail lights_



17:57:56  <andythenorth_> it's not trivial, figuring out backwards, flipped, and which angles need lights :P



18:00:30  <talltyler> Non-articulated steam engines should be able to go full speed, tender engines should not (in some places)



18:00:48  <peter1138> Fair enough.



18:04:39  <belajalilija> i think by default tender engines should be able to go full speed



18:05:51  <peter1138> All tender steam engines are NewGRF, so it makes sense for NewGRFs to set flags as appropriate.



18:13:19  <peter1138> Hmm, I need to figure out how to define a NewGRF tree tile.



18:13:34  <peter1138> (Because trees are not as simple as "here's a sprite")



18:18:30  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)



18:18:48  *** MinchinWeb[m] has joined #openttd



18:24:24  <peter1138> How do you do this with NewGRF? https://www.youtube.com/watch?v=QXlPAYAVXbc



18:29:43  <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #15919: Codechange: use ContiguousContainer for SlCopy and extract VarMemType https://github.com/OpenTTD/OpenTTD/pull/15919#pullrequestreview-4782333029



18:39:22  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15919: Codechange: use ContiguousContainer for SlCopy and extract VarMemType https://github.com/OpenTTD/OpenTTD/pull/15919



18:41:29  <andythenorth_> peter1138: we don't :)



18:58:23  *** Wormnest has quit IRC (Ping timeout: 480 seconds)



18:58:29  <will_marshall_> Tender engines should clearly be able to go just as fast in reverse because 100% of the time the view backwards around the tender is just as good as the view over the front 🙂



19:02:26  *** aibo1 has joined #openttd



19:07:53  *** aibo1 has quit IRC (Quit: WeeChat 4.9.4)



19:07:56  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #15920: Codechange: Use static_cast instead of C-cast in map accessors https://github.com/OpenTTD/OpenTTD/pull/15920



19:09:01  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15919: Codechange: use ContiguousContainer for SlCopy and extract VarMemType https://github.com/OpenTTD/OpenTTD/pull/15919



19:13:58  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15920: Codechange: Use static_cast instead of C-cast in map accessors https://github.com/OpenTTD/OpenTTD/pull/15920#pullrequestreview-4782408124



19:22:52  *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)



19:24:30  *** WormnestAndroid has joined #openttd



19:47:13  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15920: Codechange: Use static_cast instead of C-cast in map accessors https://github.com/OpenTTD/OpenTTD/pull/15920



20:06:26  *** gelignite has joined #openttd



20:49:34  *** gelignite has quit IRC ()



21:11:47  *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)



21:31:39  <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #15906: Fix #11127: use callback for dual-headed engine purchase capacity https://github.com/OpenTTD/OpenTTD/pull/15906#pullrequestreview-4782522494



21:48:44  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #15918: Fix: Change the order that (Un)FlattenNewGRFLabel convert to/from uint32_t https://github.com/OpenTTD/OpenTTD/pull/15918



21:48:47  <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #15917: [Bug]: Only one Rail/Road type of each category available https://github.com/OpenTTD/OpenTTD/issues/15917



22:44:43  *** WormnestAndroid has quit IRC (Remote host closed the connection)



22:45:43  *** WormnestAndroid has joined #openttd



23:00:22  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)



23:00:55  *** MinchinWeb[m] has joined #openttd



23:24:04  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)



23:24:21  *** MinchinWeb[m] has joined #openttd