IRC logs for #openttd on OFTC at 2023-04-20
            
00:52:12 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
00:53:00 *** WormnestAndroid has joined #openttd
01:09:14 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
01:24:36 *** WormnestAndroid has joined #openttd
01:43:03 *** WormnestAndroid has quit IRC (Remote host closed the connection)
01:43:16 *** WormnestAndroid has joined #openttd
02:03:34 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
02:30:53 *** WormnestAndroid has joined #openttd
02:38:59 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
02:42:23 *** D-HUND has joined #openttd
02:45:46 *** debdog has quit IRC (Ping timeout: 480 seconds)
02:47:30 <DorpsGek> [OpenTTD/OpenTTD] maaaaaaaaaad opened issue #10681: [Crash]: https://github.com/OpenTTD/OpenTTD/issues/10681
02:50:07 *** WormnestAndroid has joined #openttd
03:00:05 *** herms has quit IRC (Quit: bye)
03:02:59 *** D-HUND is now known as debdog
03:03:59 *** herms has joined #openttd
03:17:30 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
03:17:33 *** WormnestAndroid has joined #openttd
03:21:45 *** WormnestAndroid has quit IRC (Read error: No route to host)
03:27:24 *** WormnestAndroid has joined #openttd
03:32:46 *** Wormnest has quit IRC (Quit: Leaving)
03:47:06 *** keikoz has joined #openttd
04:44:56 *** Flygon has quit IRC (Remote host closed the connection)
05:43:22 <petern> MacOS stack traces useless again?
05:55:31 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1515754075
06:07:53 <petern> #10681 requires downloading 772MiB of NewGRFs... :/
06:11:40 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1098491156289036299/image.png
06:11:40 <petern> Hmm, something broke, but I wasn't watching it.
06:11:51 <petern> It had started downloading and then went back to 0.
06:16:19 <petern> Rubidium, how do I debug tile accessors these days? Since the Tile/TileIndex change I can't just put in numbers.
06:20:57 <petern> Hmm, I wonder if it was still downloading but overflowed -- I did change how the progress bar is drawn with interface scaling.
06:24:19 <petern> Oh great, grfmsg got sorted, I was wondering if I should macroize it somehow but it's been done 😄
06:46:43 <petern> Is it possible we'll do a 13.2 or are we too far gone for that?
06:50:28 <andythenorth> did we break anything that causes 14.0 to happen?
06:53:45 <Eddi|zuHause> didn't we do 13.1 like yesterday?
06:53:51 <LordAro> petern: macos traces from old versions definitely still useless
06:54:11 <petern> It's 13.1, despite the report saying 11.3.
06:54:53 <LordAro> ah
06:55:04 <petern> (And there wasn't a 11.3 :))
06:55:10 <LordAro> oh yeah, lol
06:55:21 <LordAro> might have been 1.1.1 :p
06:55:26 <petern> I wouldn't remember that either.
06:55:29 <petern> Also...
06:55:41 <petern> I got woken up by leg cramping this morning 😦
06:55:50 <LordAro> :(
06:56:08 <LordAro> my knee did better than expected yesterday
06:56:15 <LordAro> paying for it today though
06:56:23 *** sla_ro|master has joined #openttd
06:57:09 <andythenorth> I should get up
06:57:12 <andythenorth> have to go places
06:57:39 <andythenorth> should stop making variants for Horse
06:59:31 <Rubidium_> petern: with a unit test? Though, can't you do GetTileSomethingSomething(Tile(1234))?
06:59:58 <petern> Unit tests are not debugging 🙂
07:00:09 <petern> And no.
07:01:54 <petern> > GetTileType(t)
07:01:54 <petern> > Passing class, struct, or union types by value to a function evaluation is not supported.
07:02:02 <petern> > GetTileType(1876536)
07:02:02 <petern> > Passing class, struct, or union types by value to a function evaluation is not supported.
07:02:02 <petern> > GetTileType(Tile(1876536))
07:02:02 <petern> > expression preceding parentheses of apparent call must have (pointer-to-) function type
07:02:25 <petern> > Tile(1876536)
07:02:25 <petern> > Implicit constructor call not supported.
07:02:55 <Rubidium_> hmm, are there many accessors you need/want to manually call from the debugger? Otherwise just make a void MySomething(int tile) { GetTileSomethingSometing(tile); } for debugging
07:02:58 <petern> Hmm, maybe I need a debug build 🙂
07:03:51 <petern> Depends, I'm just trying to look at data for the crash save in #10681, so it could be any old thing.
07:06:42 <petern> Urgh, this NewGRF has tons of 4x 32bpp graphics, debug build is slow to load 😄
07:14:37 <Eddi|zuHause> the 700MB must come from somewhere :p
07:20:28 <petern> Anyway, seems a road stop points to a plain grass tile.
07:30:29 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #10681: [Crash]: Segmentation fault https://github.com/OpenTTD/OpenTTD/issues/10681
08:11:25 *** sittinbythefire has joined #openttd
08:11:25 <sittinbythefire> Would anyone happen to have an updated NML syntax highlighter for Notepad++? The one on the Devzone is pretty outdated and I can't access the (slightly) more current one I have on my laptop at the moment.
08:13:13 <Eddi|zuHause> the lone signal https://i.ibb.co/pf4ZbBp/Screenshot-20230420-091829.png
08:17:34 <pickpacket> Eddi|zuHause: is this a game or reality?
08:17:47 <Eddi|zuHause> that is transport fever 2
08:18:00 <pickpacket> graphics look pretty sweet
08:18:15 <pickpacket> just the fact that I had to ask speaks for that I guess :D
08:18:47 <Eddi|zuHause> that's low-medium settings on the graphics :p
08:18:57 <pickpacket> wow
08:25:33 <TrueBrain> also, maybe check in with your eye-doctor, you might need glasses 😛
08:32:09 <pickpacket> TrueBrain: I have glasses 😛
08:41:42 *** sla_ro|master has quit IRC ()
08:44:21 *** sla_ro|master has joined #openttd
08:50:19 *** sla_ro|master has quit IRC ()
08:51:54 *** sla_ro|master has joined #openttd
08:52:42 <petern> > ALL my projects in Bananas are all OBSOLETE
08:52:43 <petern> Hmm well
08:54:17 <petern> Good job, have to trawl the forums finding a specific page...
08:55:09 <LordAro> sounds like we should mark them all as savegame-only
08:55:20 <LordAro> if that's what they feel
08:55:44 <petern> Well, the issue is the one I need is in a savegame, and isn't on bananas
08:55:53 <petern> Or at least, not the correct version.
08:56:22 <petern> https://bananas.openttd.org/package/newgrf/47474750
08:58:26 <petern> Mind you, a missing object NewGRF shouldn't cause much issue.
08:59:14 <LordAro> seems like it's something to do with the new versions not being ready?
08:59:26 <LordAro> rather than any particular idealogical disagreement
09:45:38 <petern> I wasn't suggesting any motive.
09:49:27 <pickpacket> it's all because of the new world order. The lizard people have hacked bananas
09:50:34 <FLHerne> sittinbythefire: belatedly: https://www.flherne.uk/files/nml_notepadpp.xml
10:12:05 <LordAro> FLHerne: maybe worth putting in the nml repo?
10:12:52 <FLHerne> LordAro: it's generated by the gen_editor script in the nml repo
10:13:45 <FLHerne> but yeah, including the actual generated files in some download would be nice
10:13:57 <FLHerne> I can't find them anywhere
10:13:58 <LordAro> oh i see
10:53:57 <glx[d]> petern: Well the trace is at least readable now, but it's too small to be useful in #10681
10:57:01 <petern> Quite.
11:05:12 <petern> Hmm, apparently it's 20°C in here, but it feels freezing :/
11:12:37 <Eddi|zuHause> check your blood pressure?
11:13:14 * petern has no blood
11:19:49 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516152795
11:22:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516155428
11:46:28 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
11:46:55 *** WormnestAndroid has joined #openttd
11:48:09 <petern> Hmm, it wasn't an overflow.
11:51:55 <petern> I think the timeout flow control is breaking.
12:01:04 <petern> Or something else is. I changed the timeout and it stopped quickly, but now it just works. Huh.
12:13:44 <Kuhnovic> I recently learned that there is a setting to scale plane speeds so they aren't too overpowered. Maybe it makes sense to do the same for ships but the other way around, because they can feel very sluggish. What do you guys think?
12:16:22 <pickpacket> Kuhnovic: I feel like ships are a bit neglected in the base game, but there are several NewGRFs that do a lot more with them. Those NewGRFs make that kind of setting superfluous imho
12:16:50 <petern> Actually it's the wrong way around, the original and default is 1/4, but of course people want planes to actually go fast, so the setting scales plane speeds so they are overpowered...
12:18:45 <petern> Making ships go faster would be weird.
12:18:59 <Kuhnovic> pickpacket: I agree with you that there are quite nice Ship NewGRFs out there, but most of them don't provide any faster ships, just a bigger variety
12:19:39 <petern> Hmm, is a savegame load failure critical?
12:19:47 <petern> Or rather, ever not critical?
12:20:48 <Kuhnovic> Crash-the-game-critical or abort-the-loading-process-critical 😛 ?
12:21:23 <Rubidium_> "a" savegame load failure not necessarily, though "intro game" load crash is probably fairly critical
12:22:42 <dP> I just happened to to confirm #9817 can cause trains to skip servicing, can someone finally reopen it or should I make a new bug?
12:22:51 <dP> https://cdn.discordapp.com/attachments/1008473233844097104/1098584564437368902/Screenshot_from_2023-04-20_16-20-45.png
12:22:58 <Rubidium_> I was almost thinking you were reviewing a PR of me, but... I haven't pushed that yet...
12:24:00 <DorpsGek> [OpenTTD/OpenTTD] PeterN reopened issue #9817: [Bug]: Trains will not enter depots in front of them https://github.com/OpenTTD/OpenTTD/issues/9817
12:24:15 <FLHerne> Ships already have the unique strength of near-infinite capacity, plus lower infrastructure costs
12:24:35 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #9817: [Bug]: Trains will not enter depots in front of them https://github.com/OpenTTD/OpenTTD/issues/9817
12:26:02 <Rubidium_> but... (automatically) changing the reservation while driving is not really realistic! :D Here doing that means revoking the signal and leaving everything in place for I think 3 minutes before the switches may be changed again, to prevent any kind of weird derailments due to the revocation of the signal arriving too late
12:26:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10682: Change: Failure to load a savegame is critical. https://github.com/OpenTTD/OpenTTD/pull/10682
12:27:01 <Rubidium_> but then again, there are basically always signals just before switches
12:27:03 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on issue #9817: [Bug]: Trains will not enter depots in front of them https://github.com/OpenTTD/OpenTTD/issues/9817
12:31:36 <petern> it's either a pathfinding (+reservation) conflict issue, or a timing issue, or most likely both.
12:43:56 <dP> you mean skipping service? there is nothing timing about it, it just can't find depot because of existing reservation
12:44:07 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #10682: Change: Failure to load a savegame is critical. https://github.com/OpenTTD/OpenTTD/pull/10682#pullrequestreview-1393905497
12:45:00 <dP> and it was too far from the depot when it reserved that
12:58:02 *** lobstarooo_ has joined #openttd
12:59:18 *** lobstarooo has joined #openttd
12:59:44 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#pullrequestreview-1393934442
13:02:15 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516289171
13:06:04 *** lobstarooo_ has quit IRC (Ping timeout: 480 seconds)
13:11:52 *** lobstarooo has quit IRC (Ping timeout: 480 seconds)
13:24:12 <LordAro> T-4min til maybe-space
13:38:09 <petern> dP: No I mean working out if we will need to find a depot earlier.
13:39:06 <Eddi|zuHause> i read "space" and thought i was in a different channel :p
13:39:36 <LordAro> rocket went bang
13:50:35 <petern> "Shame"
13:53:59 <petern> So they're just happy it launched. We had rocket launches 60-odd years ago...
14:09:59 *** Flygon has joined #openttd
14:14:47 <Eddi|zuHause> but not THIS rocket. it's a special rocket. or something
14:16:06 <LordAro> it is a very big rocket
14:16:16 <LordAro> bigger than Saturn V in several ways
14:27:41 <DorpsGek> [OpenTTD/OpenTTD] maaaaaaaaaad commented on issue #10681: [Crash]: Segmentation fault https://github.com/OpenTTD/OpenTTD/issues/10681
14:31:37 <petern> Hmm, using HTTP POST to retrieve data feels wrong :/
14:34:17 <pickpacket> petern: uhm. Yeah. That sounds wrong 🤔
14:35:08 <petern> Javascript's fetch() API disallows HTTP GET with a body, so you end up hitting query string length limits.
14:37:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516446683
14:50:25 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10679: Codechange: replace printf-like functions with fmt in Squirrel https://github.com/OpenTTD/OpenTTD/pull/10679
15:10:56 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10675: Codechange: use string/fmt instead of printf for ShowInfo(F) https://github.com/OpenTTD/OpenTTD/pull/10675
15:15:34 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10683: Fix: fmt's {:#04X} yields '0X00', not '0x0000' https://github.com/OpenTTD/OpenTTD/pull/10683
15:30:24 <glx[d]> It includes 0X in the 4 ?
15:31:19 <Rubidium_> yes...
15:31:37 <LordAro> that's dumb
15:33:26 <dP> probably for some consistency
15:33:29 <dP> total width of the output
15:37:53 <LordAro> >>> '{:#04x}'.format(0)
15:37:53 <LordAro> '0x00'
15:37:58 <LordAro> consistency with python i suppose
15:48:17 <dP> stuff like this in general I'd guess `'{:#7,}'.format(-1234)`
15:48:56 <dP> don't want some unexpected punctuation screwing your formatting
15:49:58 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516567089
15:50:40 <LordAro> probably so
15:51:25 <LordAro> fun Ada fact: the Image function for integers ("to string") always includes a leading space for non-negative numbers, so it's consistently aligned for negative numbres
15:51:32 <LordAro> this is helpful precisely never
16:04:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #10682: Change: Failure to load a savegame is critical. https://github.com/OpenTTD/OpenTTD/pull/10682
16:04:29 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516590051
16:06:35 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516593440
16:11:09 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516600057
16:12:10 *** WormnestAndroid has quit IRC (Remote host closed the connection)
16:12:11 *** WormnestAndroid has joined #openttd
16:45:21 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10684: Codechange: replace error/usererror printf variant with fmt variant and rename https://github.com/OpenTTD/OpenTTD/pull/10684
17:08:31 <petern> > commit 74444b3d347f67fb8ab430588a485e083ccd7d51 (more-scale-by-interface)
17:08:31 <petern> > Author: Peter Nelson <peter1138@openttd.org>
17:08:31 <petern> > Date: Wed Nov 16 17:15:22 2022 +0000
17:08:31 <petern> >
17:08:31 <petern> > Change: Centre company face in scaled widget.
17:08:38 <petern> I knew I had a commit for that...
17:54:45 *** nielsm has joined #openttd
18:25:30 *** Wolf01 has joined #openttd
18:34:51 *** frosch has quit IRC (Quit: User went offline on Discord a while ago)
18:42:36 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/b56be839378e93ee6bb3b54c44cc83aa9b4d1daf
18:42:37 <DorpsGek> - Update: Translations from eints (by translators)
18:46:22 <Eddi|zuHause> but what about the asymmetric earrings? :p
18:48:47 <petern> I never understood why that bugged somebody...
18:51:08 *** Vadtec has quit IRC (Quit: Vadtec was here....)
18:58:21 <Eddi|zuHause> i never really cared about the faces anyway
18:58:58 <Eddi|zuHause> i don't think i even knew there were earrings before that "bug" came up
19:12:14 <LordAro> i liked the gold-rimmed glasses, didn't care about anything else
19:15:57 <petern> Hmm, I wonder why we memset here, given it was emplaced, and is going to filled anyway. <https://github.com/OpenTTD/OpenTTD/blob/master/src/newgrf.cpp#L1968>
19:17:14 <petern> I wonder why it uses a temporary vector too 😄
19:17:24 <petern> Probably a left-over from malloc/free code flow.
19:17:51 <LordAro> mm
19:19:33 <petern> Oh it's cloning into not-a-vector.
19:22:08 <LordAro> maybe it shouldn't
19:23:00 <petern> Yeah.
19:24:04 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10685: Codechange: Make GRF temporary engine data a vector. https://github.com/OpenTTD/OpenTTD/pull/10685
19:25:49 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #10683: Fix: fmt's {:#04X} yields '0X00', not '0x0000' https://github.com/OpenTTD/OpenTTD/pull/10683#pullrequestreview-1394627448
19:29:33 *** gelignite has joined #openttd
19:32:33 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #10685: Codechange: Make GRF temporary engine data a vector. https://github.com/OpenTTD/OpenTTD/pull/10685#pullrequestreview-1394640108
19:35:55 <DorpsGek> [OpenTTD/OpenTTD] carlossss111 commented on pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#issuecomment-1516849347
19:38:53 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #10685: Codechange: Make GRF temporary engine data a vector. https://github.com/OpenTTD/OpenTTD/pull/10685
19:39:20 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1098694410914504865/image.png
19:39:20 <petern> o_O
19:39:32 <LordAro> oh no
19:40:02 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10683: Fix: fmt's {:#04X} yields '0X00', not '0x0000' https://github.com/OpenTTD/OpenTTD/pull/10683
20:15:55 *** HerzogDeXtEr has joined #openttd
20:26:32 <DorpsGek> [OpenTTD/OpenTTD] carlossss111 dismissed a review for pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680#pullrequestreview-1393934442
20:26:35 <DorpsGek> [OpenTTD/OpenTTD] carlossss111 updated pull request #10680: Fix #10600: 'Replace Vehicles' shows numbers >999 https://github.com/OpenTTD/OpenTTD/pull/10680
20:29:11 *** gelignite has quit IRC (Quit: Stay safe!)
20:50:35 *** Wormnest has joined #openttd
21:00:21 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:21:43 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10684: Codechange: replace error/usererror printf variant with fmt variant and rename https://github.com/OpenTTD/OpenTTD/pull/10684
21:28:27 <DorpsGek> [OpenTTD/team] densxd opened issue #416: [tr_TR] Translator access request https://github.com/OpenTTD/team/issues/416
21:50:44 <sittinbythefire> FLHerne: very belatedly, thank you 🙂
22:05:58 <TrueBrain> petern: Yeah, a few checks aren't mandatory .. meant so you don't have to wait for the slow mingw. But that was before auto merge was used .. so I will make those mandatory soon too :p
22:06:49 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:17:41 *** sla_ro|master has quit IRC ()
22:23:54 *** Eddi|zuHause has quit IRC (Ping timeout: 480 seconds)
22:35:31 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:54:21 *** nebulabc has quit IRC (Quit: must've rage quit ¯\_(ツ)_/¯)
22:54:29 *** nebulabc has joined #openttd
23:35:04 <DorpsGek> [OpenTTD/team] glx22 commented on issue #416: [tr_TR] Translator access request https://github.com/OpenTTD/team/issues/416