IRC Logs | #openttd | 2025-04-28

00:41:46 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)

00:47:56 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

00:58:45 *** WormnestAndroid has quit IRC (Remote host closed the connection)

02:29:25 *** WormnestAndroid has quit IRC (Remote host closed the connection)

02:33:30 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)

03:06:35 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)

03:19:09 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

03:19:21 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

03:19:25 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

04:46:37 <DorpsGek> - Update: Translations from eints (by translators)

05:08:43 *** keikoz has quit IRC (Ping timeout: 480 seconds)

06:00:27 <DorpsGek> [OpenTTD/OpenTTD] zephyris merged pull request #14127: Fix: List Ctrl+Click behaviour for vehicle details tooltip https://github.com/OpenTTD/OpenTTD/pull/14127

06:00:30 <DorpsGek> [OpenTTD/OpenTTD] zephyris closed issue #14128: [Bug]: Missing tooltip explanation for ctrl-click vehicle details https://github.com/OpenTTD/OpenTTD/issues/14128

06:07:15 <DorpsGek> [OpenTTD/OpenTTD] WenSimEHRP commented on pull request #13993: Add: Player configuration of badge display. https://github.com/OpenTTD/OpenTTD/pull/13993#issuecomment-2834073003

07:04:24 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14137: Codechange: Use EnumBitSet for HouseZones. https://github.com/OpenTTD/OpenTTD/pull/14137

07:43:00 <DorpsGek> - Add: summary for week 17 of 2025 (by OpenTTD Survey)

07:47:54 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris commented on pull request #7: Fix: Sub-pixel shift small font slashes https://github.com/OpenTTD/OpenTTD-TTF/pull/7#issuecomment-2834277834

07:49:28 <_zephyris> Would make my hebrew efforts easier, because of the silly way I've set up the repo

07:56:51 <DorpsGek> [OpenTTD/OpenTTD-TTF] PeterN approved pull request #7: Fix: Sub-pixel shift small font slashes https://github.com/OpenTTD/OpenTTD-TTF/pull/7#pullrequestreview-2798323907

07:57:45 <peter1138[d]> Picture helps, though it was hidden in the 'diff' anyway

07:57:47 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris merged pull request #7: Fix: Sub-pixel shift small font slashes https://github.com/OpenTTD/OpenTTD-TTF/pull/7

07:57:52 <DorpsGek> [OpenTTD/OpenGFX2] zephyris closed issue #147: [Bug]: 1x zoom small pixel font is losing pixels in forward and backslash https://github.com/OpenTTD/OpenGFX2/issues/147

08:01:42 <_zephyris> Yup, thought the picture would be good 🙂

08:02:05 <_zephyris> I believe I have fully functional Hebew, which is quite satisfying

08:28:29 <peter1138> As a sprite font, or TTF? (I thought it was 'basically' done.)

08:58:38 <xarick> question, does the game ship with one industry per type per town ?

09:01:37 <peter1138> You can play 4kx4k maps, but you should expect that world generation will take longer because it has to do more.

09:06:56 <kuhnovic> And, surprise, it takes a lot longer in debug.

09:35:26 <peter1138> LordAro, urgh, how do I deal with .resx files? :S

09:36:53 <peter1138> I don't like using Google these days :(

09:39:32 <DorpsGek> [OpenTTD/website] auge8472 commented on pull request #350: Clarify the navigation bar https://github.com/OpenTTD/website/pull/350#issuecomment-2834621019

09:47:16 <pickpacket> is there a way to limit the length of a new train added in a newgrf? I.e. if I make a newgrf with a new train engine, but I want to set a maximum tile length of any train using it. Is that possible?

09:47:25 <DorpsGek> [OpenTTD/website] auge8472 requested changes for pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#pullrequestreview-2798728519

09:58:04 *** WormnestAndroid has quit IRC (Remote host closed the connection)

10:12:32 *** WormnestAndroid has quit IRC (Remote host closed the connection)

10:34:48 <frosch123> pickpacket: The front engine can make such restrictions, secondary engines have no say

10:37:53 <pickpacket> frosch123: do seconday engines count toward the length?

10:38:42 <frosch123> That's up to the front engine to decide

10:40:31 <frosch123> Though the "stop" part is a bit weird, as usual for grf

10:49:59 <DorpsGek> [OpenTTD/website] glx22 commented on pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#issuecomment-2834845573

10:56:22 <peter1138> Being unable to stop a vehicle is a bit of weird thing to implement :)

11:00:00 <andythenorth> there's no var for total train length, that I could find

11:00:05 <pickpacket> "I can't stop this train!!! It's about to crash!!!"

11:02:20 <andythenorth> "trains can't be stopped unless they have a brake van"

11:12:11 <reldred> it just explodes the moment it leaves the depot

11:14:50 <andythenorth> I used to make unwinnable flash games

11:15:38 <_zephyris> peter1138[d]: I'd done the normal font, serif and small needed to be made and fixed

11:44:20 <DorpsGek> [OpenTTD/OpenGFX2] frosch123 commented on pull request #187: Fix: Consolidate string definitions to allow NewGRF translation https://github.com/OpenTTD/OpenGFX2/pull/187#issuecomment-2834977127

11:44:49 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14119: Codechange: Simplify usage of GRFFileProps by adding some common helper methods. https://github.com/OpenTTD/OpenTTD/pull/14119

11:59:57 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14138: Fix: [NewGRF] Computed VarAction2 callback results were not always properly masked to 15 bit results. https://github.com/OpenTTD/OpenTTD/pull/14138

12:07:49 <DorpsGek> [OpenTTD/website] J-PIE-314 commented on pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#issuecomment-2835031320

12:14:40 <DorpsGek> [OpenTTD/website] J-PIE-314 updated pull request #350: Clarify the navigation bar https://github.com/OpenTTD/website/pull/350

12:16:26 <DorpsGek> [OpenTTD/website] J-PIE-314 updated pull request #350: Clarify the navigation bar https://github.com/OpenTTD/website/pull/350

12:18:19 <andythenorth> just until grid frequency is restored

12:18:37 <digitalfox> The military are being called and put on alert

12:19:10 <andythenorth> civil defense measures, probably a protocol

12:23:15 <digitalfox> There is no idea when it will be restored

12:23:51 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139

12:24:15 *** Flygon has quit IRC (Read error: Connection reset by peer)

12:45:16 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14138: Fix: [NewGRF] Computed VarAction2 callback results were not always properly masked to 15 bit results. https://github.com/OpenTTD/OpenTTD/pull/14138#issuecomment-2835126641

12:52:18 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14138: Fix: [NewGRF] Computed VarAction2 callback results were not always properly masked to 15 bit results. https://github.com/OpenTTD/OpenTTD/pull/14138#issuecomment-2835144502

13:39:13 <DorpsGek> [OpenTTD/OpenGFX2] 2TallTyler approved pull request #187: Fix: Consolidate string definitions to allow NewGRF translation https://github.com/OpenTTD/OpenGFX2/pull/187#pullrequestreview-2799339908

13:41:39 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14137: Codechange: Use EnumBitSet for HouseZones. https://github.com/OpenTTD/OpenTTD/pull/14137#pullrequestreview-2799331015

13:49:32 <peter1138> I guess it probably needs a rebuild, but I'm bad at rebuilding wheels.

13:49:41 <peter1138> If only there was some way to get better...

13:52:30 <peter1138> Pfft. My local guy is busy, so I decided to make it worse first before I go crawling back to him.

13:56:17 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14134: Codechange: replace more std::string_view in network code https://github.com/OpenTTD/OpenTTD/pull/14134#pullrequestreview-2799383850

14:18:11 <peter1138> Hmm, so drag and drop in a fake drop dowm menu?

14:31:14 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #14134: Codechange: replace more std::string_view in network code https://github.com/OpenTTD/OpenTTD/pull/14134#pullrequestreview-2799522832

14:31:27 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14134: Codechange: replace more std::string_view in network code https://github.com/OpenTTD/OpenTTD/pull/14134

14:32:39 <kuhnovic> peter1138: I tend to go for the "find the peak of the wobble, pluck it like a guitar string, tighten until it sounds somewhat like the spokes around it" method. That's on a city bike mind you.

14:43:54 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14134: Codechange: replace more std::string_view in network code https://github.com/OpenTTD/OpenTTD/pull/14134#pullrequestreview-2799569059

14:57:19 <andythenorth> on an old stamped out Raleigh wheel?

14:57:40 <andythenorth> the seam de-welds and the wheel unfolds

15:00:07 <DorpsGek> [OpenTTD/website] J-PIE-314 commented on pull request #350: Clarify the navigation bar https://github.com/OpenTTD/website/pull/350#issuecomment-2835545956

15:02:43 *** tokai has quit IRC (Ping timeout: 480 seconds)

15:27:44 <frosch123> peter1138: Drag and drop is weird in the dropdown. If dragging at all, I would expect it on the column headers.

15:27:44 <frosch123> But imo the buttons are fine. Though I prefer up/down, since I understand them as moving stuff in the dropdown, and only by proxy the columns

15:30:30 <andythenorth> what about "just" dragging the badge columns around 😛

15:53:23 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14134: Codechange: replace more std::string_view in network code https://github.com/OpenTTD/OpenTTD/pull/14134

16:27:51 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14137: Codechange: Use EnumBitSet for HouseZones. https://github.com/OpenTTD/OpenTTD/pull/14137

16:31:01 *** Smedles_ has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)

16:31:18 <DorpsGek> [OpenTTD/nml] frosch123 updated pull request #375: Add: Constants for (new) station and roadstop animation and random triggers. https://github.com/OpenTTD/nml/pull/375

16:37:55 <peter1138> OpenGFX 1 arrow sizes are oddly... large.

16:40:40 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14137: Codechange: Use EnumBitSet for HouseZones. https://github.com/OpenTTD/OpenTTD/pull/14137#pullrequestreview-2799978028

16:52:35 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14140: Fix: Out-of-bounds read, if NewGRF stations provided no spritesets. https://github.com/OpenTTD/OpenTTD/pull/14140

16:53:37 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14140: Fix: Out-of-bounds read, if NewGRF stations provided no spritesets. https://github.com/OpenTTD/OpenTTD/pull/14140#pullrequestreview-2800021842

16:53:57 <DorpsGek> [OpenTTD/OpenTTD] frosch123 dismissed a review for pull request #14140: Fix: Out-of-bounds read, if NewGRF stations provided no spritesets. https://github.com/OpenTTD/OpenTTD/pull/14140#pullrequestreview-2800021842

16:54:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14140: Fix: Out-of-bounds read, if NewGRF stations provided no spritesets. https://github.com/OpenTTD/OpenTTD/pull/14140

16:55:09 <frosch123> Too much copy paste when splitting patches

16:56:40 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14140: Fix: Out-of-bounds read, if NewGRF stations provided no spritesets. https://github.com/OpenTTD/OpenTTD/pull/14140#pullrequestreview-2800028910

16:59:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141

16:59:16 <DorpsGek> [OpenTTD/OpenGFX2] zephyris merged pull request #187: Fix: Consolidate string definitions to allow NewGRF translation https://github.com/OpenTTD/OpenGFX2/pull/187

17:09:49 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14142: Fix: [NewGRF] AI station construction callback did not work for stations with ID >= 0x100. https://github.com/OpenTTD/OpenTTD/pull/14142

17:11:02 <peter1138> File that one under "things that are unlikely to ever be used" :-)

17:12:31 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14142: Fix: [NewGRF] AI station construction callback did not work for stations with ID >= 0x100. https://github.com/OpenTTD/OpenTTD/pull/14142#pullrequestreview-2800068970

17:23:24 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141

17:24:04 <peter1138> Hmm. Making the settings buttons wider to fit the up/down arrow sprites makes them feel overly large.

17:34:38 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14143: Change: [NewGRF] If Action123 does not resolve in a valid SpriteSet, prefer drawing the default sprite instead of an invalid sprite. https://github.com/OpenTTD/OpenTTD/pull/14143

17:39:54 *** Wormnest has quit IRC (Ping timeout: 480 seconds)

17:45:32 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141#pullrequestreview-2800138589

17:46:49 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)

17:51:04 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141#issuecomment-2836032646

17:55:31 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on issue #14098: [Bug]: Failure to play music with long MIDI tracks in Base Music Set https://github.com/OpenTTD/OpenTTD/issues/14098

17:58:30 <jfs> nielsmhviaGitHub: I don't have a dev environment set up at this time, but I'd think the fix is easy to implement, just tedious to test

18:15:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14144: Fix #14098: Use 64-bit type for MIDI realtime information. https://github.com/OpenTTD/OpenTTD/pull/14144

18:21:49 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141

18:39:24 <peter1138> Initial minimum size doesn't take resize_x/resize_y into account.

18:39:47 <peter1138> This leads to widgets being not-a-multiple-of-resize.

18:39:49 <DorpsGek> [OpenTTD/website] auge8472 commented on pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#issuecomment-2836156731

18:41:06 <jfs> peter1138: Perhaps also change the two conversion constants at line 37-38 of dmusic.cpp to REFERENCE_TIME

18:42:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14144: Fix #14098: Use 64-bit type for MIDI realtime information. https://github.com/OpenTTD/OpenTTD/pull/14144

18:46:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141

18:49:23 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14140: Fix: Out-of-bounds read, if NewGRF stations provided no spritesets. https://github.com/OpenTTD/OpenTTD/pull/14140

18:50:16 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14142: Fix: [NewGRF] AI station construction callback did not work for stations with ID >= 0x100. https://github.com/OpenTTD/OpenTTD/pull/14142

18:52:43 <DorpsGek> [OpenTTD/website] auge8472 commented on pull request #351: Make entire header logo clickable https://github.com/OpenTTD/website/pull/351#issuecomment-2836194286

18:54:44 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141#pullrequestreview-2800391281

18:54:47 <DorpsGek> [OpenTTD/website] J-PIE-314 commented on pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#issuecomment-2836201293

19:01:18 <DorpsGek> [OpenTTD/website] J-PIE-314 dismissed a review for pull request #351: Make entire header logo clickable https://github.com/OpenTTD/website/pull/351#pullrequestreview-2797745740

19:01:21 <DorpsGek> [OpenTTD/website] J-PIE-314 updated pull request #351: Make entire header logo clickable https://github.com/OpenTTD/website/pull/351

19:01:33 <DorpsGek> [OpenTTD/website] auge8472 commented on pull request #352: Fix typo (can->can't) https://github.com/OpenTTD/website/pull/352#issuecomment-2836222742

19:02:17 <DorpsGek> [OpenTTD/website] auge8472 merged pull request #352: Fix typo (can->can't) https://github.com/OpenTTD/website/pull/352

19:02:22 <DorpsGek> [OpenTTD/website] auge8472 closed issue #349: Typo can/can't https://github.com/OpenTTD/website/issues/349

19:05:53 <DorpsGek> [OpenTTD/website] auge8472 approved pull request #351: Make entire header logo clickable https://github.com/OpenTTD/website/pull/351#pullrequestreview-2800421384

19:09:58 <DorpsGek> [OpenTTD/website] auge8472 approved pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#pullrequestreview-2800429647

19:12:52 <DorpsGek> [OpenTTD/website] auge8472 commented on pull request #348: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348#issuecomment-2836248185

19:14:41 <DorpsGek> [OpenTTD/website] 2TallTyler merged pull request #351: Change: Make entire header logo clickable https://github.com/OpenTTD/website/pull/351

19:15:15 <DorpsGek> [OpenTTD/website] 2TallTyler merged pull request #348: Change: Display a larger resolution of the featured screenshot when clicked https://github.com/OpenTTD/website/pull/348

19:15:59 <DorpsGek> [OpenTTD/website] auge8472 commented on pull request #352: Fix typo (can->can't) https://github.com/OpenTTD/website/pull/352#issuecomment-2836259257

19:16:39 <DorpsGek> [OpenTTD/website] 2TallTyler commented on pull request #352: Fix typo (can->can't) https://github.com/OpenTTD/website/pull/352#issuecomment-2836262170

19:18:07 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14145: Fix: Calculating maximum minimum widget size ignored resize steps. https://github.com/OpenTTD/OpenTTD/pull/14145

19:20:49 <peter1138> This issue has bugged/plagued me for years and I finally did something about it :p

19:24:36 <peter1138> Nah, my main quest is long lost from a scroll back buffer somewhere.

19:24:43 <peter1138> Maybe it's in the IRC logs from 200x...

19:25:50 <peter1138> I should test this some more, it works in this case but I have a feeling it might not in all cases :s

19:27:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14137: Codechange: Use EnumBitSet for HouseZones. https://github.com/OpenTTD/OpenTTD/pull/14137

19:29:23 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141#pullrequestreview-2800481189

19:29:41 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)

19:29:46 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14145: Fix: Calculating maximum minimum widget size ignored resize steps. https://github.com/OpenTTD/OpenTTD/pull/14145#issuecomment-2836294592

19:30:56 <peter1138> Requires the resize widget to be last in the list.

19:31:16 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14141: Codechange: replace char*s in file related functions https://github.com/OpenTTD/OpenTTD/pull/14141

19:41:37 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14146: Codechange: replace char* from settingsgen https://github.com/OpenTTD/OpenTTD/pull/14146

19:48:26 <peter1138> Ah, vertical/horizontal containers already do something, but not the right thing. Hmm.

19:48:43 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14143: Change: [NewGRF] If Action123 does not resolve in a valid SpriteSet, prefer drawing the default sprite instead of an invalid sprite. https://github.com/OpenTTD/OpenTTD/pull/14143#pullrequestreview-2800531655

19:50:58 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139#pullrequestreview-2800539837

19:52:54 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14147: Codechange: Simplify usage of ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14147

19:54:03 <Rubidium> frosch123: regarding the string view literals. If/when I remove the char* variant of StringParameter, there's a 'fios.title = GetString(STR_SAVELOAD_PARENT_DIRECTORY, "..");

19:54:30 <Rubidium> in fios.cpp which will fail to build because it can't choose between std::string& and std::string_view

19:55:15 <Rubidium> I can ofcourse wrap the ".." in std::string_view, but isn't that what the `sv` literal is supposed to do?

19:56:19 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14147: Codechange: Simplify usage of ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14147#pullrequestreview-2800562255

19:57:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14148: Codechange: Do not use a mutable global to return calculated VarAction2 results. https://github.com/OpenTTD/OpenTTD/pull/14148

20:03:21 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139#pullrequestreview-2800582489

20:03:30 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14149: Add: [NewGRF] Add explicit (Var)Action2 results for "callback failed" and "calculated result". https://github.com/OpenTTD/OpenTTD/pull/14149

20:06:38 <frosch123> Rubidium: Valid case. But when I tried I failed to only import "sv" into the global namespace. "using sv = std::literals::sv;" failed for me

20:19:45 <DorpsGek> [OpenTTD/OpenTTD] frosch123 dismissed a review for pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139#pullrequestreview-2800539837

20:19:48 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139

20:20:58 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14143: Change: [NewGRF] If Action123 does not resolve in a valid SpriteSet, prefer drawing the default sprite instead of an invalid sprite. https://github.com/OpenTTD/OpenTTD/pull/14143

20:21:22 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139#pullrequestreview-2800638764

20:22:20 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14150: Change: Use enum/bitset to track which script configuration items are defined. https://github.com/OpenTTD/OpenTTD/pull/14150

20:33:55 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14146: Codechange: replace char* from settingsgen https://github.com/OpenTTD/OpenTTD/pull/14146#pullrequestreview-2800645013

20:40:34 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14146: Codechange: replace char* from settingsgen https://github.com/OpenTTD/OpenTTD/pull/14146

20:41:16 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14146: Codechange: replace char* from settingsgen https://github.com/OpenTTD/OpenTTD/pull/14146

20:42:15 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14150: Change: Use enum/bitset to track which script configuration items are defined. https://github.com/OpenTTD/OpenTTD/pull/14150#pullrequestreview-2800759780

20:45:45 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14146: Codechange: replace char* from settingsgen https://github.com/OpenTTD/OpenTTD/pull/14146#pullrequestreview-2800784618

20:51:12 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14151: Codechange: provide 'sv' (string_view) literals globally https://github.com/OpenTTD/OpenTTD/pull/14151

20:54:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14152: Codechange: Pass script instance/engine/info by reference instead of pointer. https://github.com/OpenTTD/OpenTTD/pull/14152

20:56:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14151: Codechange: provide 'sv' (string_view) literals globally https://github.com/OpenTTD/OpenTTD/pull/14151#issuecomment-2836587110

20:56:57 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14153: Codechange: store revision information in C++ style strings https://github.com/OpenTTD/OpenTTD/pull/14153

20:58:35 *** nielsm has quit IRC (Ping timeout: 480 seconds)

21:05:01 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #14152: Codechange: Pass script instance/engine/info by reference instead of pointer. https://github.com/OpenTTD/OpenTTD/pull/14152#pullrequestreview-2800923556

21:08:21 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14138: Fix: [NewGRF] Computed VarAction2 callback results were not always properly masked to 15 bit results. https://github.com/OpenTTD/OpenTTD/pull/14138#pullrequestreview-2800941226

21:18:12 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14153: Codechange: store revision information in C++ style strings https://github.com/OpenTTD/OpenTTD/pull/14153#pullrequestreview-2800979497

21:20:47 <DorpsGek> [OpenTTD/OpenGFX2] J-PIE-314 opened issue #193: Tree trunks look like they have a horizontal line across them (1x zoom) https://github.com/OpenTTD/OpenGFX2/issues/193

21:24:46 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14139: Fix: [NewGRF] The result of Action123 evaluation affected rerandomisation in a weird corner case. https://github.com/OpenTTD/OpenTTD/pull/14139

21:25:35 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14138: Fix: [NewGRF] Computed VarAction2 callback results were not always properly masked to 15 bit results. https://github.com/OpenTTD/OpenTTD/pull/14138

21:26:12 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14151: Codechange: provide 'sv' (string_view) literals globally https://github.com/OpenTTD/OpenTTD/pull/14151#pullrequestreview-2801006107

21:26:48 *** akimoto has quit IRC (Remote host closed the connection)

21:27:50 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14146: Codechange: replace char* from settingsgen https://github.com/OpenTTD/OpenTTD/pull/14146

21:30:43 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14121: Add: [NewGRF] Add purchase list Action3 CID for houses, industries, industry tiles, airports and airport tiles. https://github.com/OpenTTD/OpenTTD/pull/14121

21:31:23 *** keikoz has quit IRC (Ping timeout: 480 seconds)

21:36:25 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)

21:38:14 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14147: Codechange: Simplify usage of ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14147

21:39:37 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14151: Codechange: provide 'sv' (string_view) literals globally https://github.com/OpenTTD/OpenTTD/pull/14151

21:48:36 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14153: Codechange: store revision information in C++ style strings https://github.com/OpenTTD/OpenTTD/pull/14153

22:02:32 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14154: Codechange: Use result of .find() instead of looking up multiple times. https://github.com/OpenTTD/OpenTTD/pull/14154

22:08:01 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14150: Change: Use enum/bitset to track which script configuration items are defined. https://github.com/OpenTTD/OpenTTD/pull/14150

22:08:23 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14152: Codechange: Pass script instance/engine/info by reference instead of pointer. https://github.com/OpenTTD/OpenTTD/pull/14152

22:28:49 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14154: Codechange: Use result of .find() instead of looking up multiple times. https://github.com/OpenTTD/OpenTTD/pull/14154#pullrequestreview-2801167925

22:38:12 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #373: Add: Road/tram/railtype variable 0x45. https://github.com/OpenTTD/nml/pull/373#pullrequestreview-2801200754

22:39:29 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #375: Add: Constants for (new) station and roadstop animation and random triggers. https://github.com/OpenTTD/nml/pull/375#pullrequestreview-2801204667

22:56:48 *** debdog has quit IRC (Ping timeout: 480 seconds)

23:06:01 <goddess_ishtar> do any of you guys use an IDE for OpenTTD development?

23:20:23 <_glx_> hmm for some weird reason workflows didn't trigger for nml#375

23:44:15 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)