IRC logs for #openttd on OFTC at 2022-11-14
            
00:08:03 *** Soni has quit IRC (Ping timeout: 480 seconds)
01:23:32 *** Wormnest_ has quit IRC (Ping timeout: 480 seconds)
01:29:29 *** Soni has joined #openttd
01:35:58 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
01:50:09 *** WormnestAndroid has joined #openttd
02:01:47 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #10168: Change: Requested changes to the Finance GUI https://github.com/OpenTTD/OpenTTD/pull/10168
02:01:50 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #10160: [Bug]: Finance window - The title "Net profit" is most probably wrong (2/6) https://github.com/OpenTTD/OpenTTD/issues/10160
02:01:53 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #10163: [Detail]: Finance window - The location of total of revenues or expense seems not entirely legible and confusing (4/6) https://github.com/OpenTTD/OpenTTD/issues/10163
02:01:56 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #10165: [Detail]: Finance Window - The financial summary can be confusing (5/6) https://github.com/OpenTTD/OpenTTD/issues/10165
02:43:19 *** Wormnest_ has joined #openttd
02:43:26 *** Wormnest_ has quit IRC (Remote host closed the connection)
02:46:44 *** tokai has joined #openttd
02:46:44 *** ChanServ sets mode: +v tokai
02:53:16 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
03:34:52 *** Flygon has joined #openttd
03:47:43 *** D-HUND has joined #openttd
03:51:06 *** debdog has quit IRC (Ping timeout: 480 seconds)
04:24:30 *** felix has quit IRC ()
04:24:45 *** felix has joined #openttd
04:26:58 *** supermop_Home has quit IRC (Ping timeout: 480 seconds)
05:16:23 <DorpsGek> [OpenTTD/OpenTTD] ldpl updated pull request #10001: Feature: [GS] Scriptable league tables https://github.com/OpenTTD/OpenTTD/pull/10001
05:16:47 <DorpsGek> [OpenTTD/OpenTTD] ldpl commented on pull request #10001: Feature: [GS] Scriptable league tables https://github.com/OpenTTD/OpenTTD/pull/10001#pullrequestreview-1178459358
05:51:31 <DorpsGek> [OpenTTD/OpenTTD] Arastais updated pull request #10104: Update: [Dev] Improve .editorconfig with C++ formatting rules based on OpenTTD code style https://github.com/OpenTTD/OpenTTD/pull/10104
05:52:00 <DorpsGek> [OpenTTD/OpenTTD] Arastais commented on pull request #10104: Update: [Dev] Improve .editorconfig with C++ formatting rules based on OpenTTD code style https://github.com/OpenTTD/OpenTTD/pull/10104#issuecomment-1313130728
05:58:38 *** nielsm has joined #openttd
06:02:41 <DorpsGek> [OpenTTD/OpenTTD] Arastais updated pull request #10104: Update: [Dev] Improve .editorconfig with C++ formatting rules based on OpenTTD code style https://github.com/OpenTTD/OpenTTD/pull/10104
06:06:44 <DorpsGek> [OpenTTD/OpenTTD] Arastais updated pull request #10104: Update: [Dev] Improve .editorconfig with C++ formatting rules based on OpenTTD code style https://github.com/OpenTTD/OpenTTD/pull/10104
06:09:39 *** wallabra has quit IRC (Quit: ZNC 1.8.2 - https://znc.in)
06:19:45 *** wallabra has joined #openttd
06:21:12 *** DDR has quit IRC (Ping timeout: 480 seconds)
06:43:14 *** nielsm has quit IRC (Ping timeout: 480 seconds)
07:25:19 <DorpsGek> [OpenTTD/OpenTTD] chiizujin opened issue #10171: [Bug]: Server desync when client's player_face is non-zero https://github.com/OpenTTD/OpenTTD/issues/10171
07:28:14 *** sla_ro|master has joined #openttd
08:29:07 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #10171: [Bug]: Server desync when client's player_face is non-zero https://github.com/OpenTTD/OpenTTD/issues/10171
08:29:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #10171: [Bug]: Server desync when client's player_face is non-zero https://github.com/OpenTTD/OpenTTD/issues/10171
08:54:33 *** WormnestAndroid has quit IRC (Remote host closed the connection)
10:06:52 <petern> Hmm, I should set up an icon/image for my misskey instance.
11:18:20 *** sla_ro|master has quit IRC ()
11:28:00 *** sla_ro|master has joined #openttd
11:38:16 *** leanishbrish has quit IRC (Quit: User went offline on Discord a while ago)
11:39:54 *** sla_ro|master has quit IRC ()
12:04:08 <petern> Does our INI settings system support floats?
12:05:35 <petern> Previously you could just leave font-zoom on 1x and set your gui zoom and 2x and font size to say 15px, now as it's intrinsically linked you can get 7px to get 14px, or 8px to get 16px... but you can't get 15px.
12:06:18 <petern> It is a bit weird how the system font APIs seem to only allow whole-numbers too.
12:22:09 *** JustANortherner has quit IRC (Quit: User went offline on Discord a while ago)
12:25:35 *** WormnestAndroid has joined #openttd
12:29:22 <Eddi|zuHause> i think you need to add a float parser in settings.cpp
12:30:56 <Eddi|zuHause> probably easier to just do a fixed-point integer
12:31:17 <peter1138> Problem with that is the setting already exists as an integer.
12:31:48 *** discord_user_89adaf5 has joined #openttd
12:31:48 <discord_user_89adaf5> What does irc mean
12:32:27 <Eddi|zuHause> IRC stands for Internet Relay Chat. it's what your grandparents used to talk before the war...
12:32:49 <andythenorth> that's increasingly worryingly true
12:33:15 <andythenorth> allowing that every 12 year old seems to have discord, or is campaigning to have it
12:33:32 <andythenorth> and irc was already 'old news' when I joined the web
12:33:55 <Eddi|zuHause> some people who used IRC in the late 80s are bound to be grandparents by now
12:34:07 <andythenorth> and there have been plenty of wars
12:34:56 <Eddi|zuHause> i think the first time i used IRC was arond 2001
12:35:06 <Eddi|zuHause> *around
12:36:30 <petern> Hey look, I managed to type in IRC by mistake 🙂
12:36:55 <Eddi|zuHause> i think i do have a discord account, actually
12:38:19 <discord_user_89adaf5> How do u talk here without discord
12:39:36 <petern> Via the IRC bridge...
12:52:39 <Eddi|zuHause> echoes from the past
13:31:26 <LordAro> beep boop
13:35:29 *** sla_ro|master has joined #openttd
14:01:54 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1041714598367207494/image.png
14:01:54 <andythenorth> chunky tankers with chunky bevels?
14:02:17 <LordAro> tiny tankers
14:02:53 <andythenorth> chibi bevels?
14:34:33 <petern> That is annoyingly not quite 1:1 on here, so it's blurry :/
14:35:47 <petern> Hmm, does uniscribe get font widths directly from the font, or from what we say?
14:37:37 <petern> Hmm, seems to be from GetGlyphWidth, so...
14:38:03 <petern> Oh maybe not, that's used for sprite overrides.
14:40:14 <petern> Makes sense otherwise kerning wouldn't work.
14:47:28 *** nielsm has joined #openttd
15:52:35 *** Smedles has joined #openttd
15:52:41 *** Smedles_ has quit IRC (Ping timeout: 480 seconds)
16:32:47 <andythenorth> hmm
16:33:00 <andythenorth> can 'chunky' be be the new New and Not
16:33:02 <andythenorth> ?
16:44:04 *** peter1138 has quit IRC (Quit: leaving)
16:44:28 *** peter1138 has joined #openttd
16:44:28 *** ChanServ sets mode: +o peter1138
16:44:58 <petern> ChunkyVariants?
16:50:42 <andythenorth> maybe
16:52:10 <petern> Interestingly, it's possible to use the monospace font before the monospace font cache is initialised, in which case it'll use the sprite font instead of configured truetype font.
16:57:22 <petern> Okay it's not that interesting.
16:59:09 *** D-HUND is now known as debdog
16:59:18 <andythenorth> I enjoyed it
17:19:08 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on issue #9985: [Bug]: Max zoom in level setting affects interface size https://github.com/OpenTTD/OpenTTD/issues/9985
17:21:26 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on issue #9876: [Crash]: OpenTTD crashed on MacOS with 32bpp baseset due to touchbar https://github.com/OpenTTD/OpenTTD/issues/9876
17:21:29 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #9876: [Crash]: OpenTTD crashed on MacOS with 32bpp baseset due to touchbar https://github.com/OpenTTD/OpenTTD/issues/9876
17:25:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #9985: [Bug]: Max zoom in level setting affects interface size https://github.com/OpenTTD/OpenTTD/issues/9985
17:27:48 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on issue #10008: [Bug]: Ctrl modifier not detected https://github.com/OpenTTD/OpenTTD/issues/10008
17:36:11 <TallTyler> I was hoping that chunky bevels would solve this, but no 😦
17:36:11 <TallTyler> https://github.com/OpenTTD/OpenTTD/issues/8770
17:36:28 <TallTyler> (Not because it bothers me, I just want to get the issue count down 😛 )
17:37:22 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on issue #9985: [Bug]: Max zoom in level setting affects interface size https://github.com/OpenTTD/OpenTTD/issues/9985
17:37:25 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #9985: [Bug]: Max zoom in level setting affects interface size https://github.com/OpenTTD/OpenTTD/issues/9985
17:39:10 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on issue #9815: [Bug]: Scroll down main settings window issue on smaller resolutions https://github.com/OpenTTD/OpenTTD/issues/9815
17:39:13 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #9815: [Bug]: Scroll down main settings window issue on smaller resolutions https://github.com/OpenTTD/OpenTTD/issues/9815
17:41:48 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #8770: Start/stop icon in vehicle status bar is off-center vertically https://github.com/OpenTTD/OpenTTD/issues/8770
17:52:59 <petern> TallTyler: A good test to find things that are still done by absolute sprite size rather than applying scaling is to use 1.75x or even 3x GUI scaling. The things that don't expand become more obvious then. Sometimes it's appropriate, other times not.
17:54:15 <petern> Not that I'm suggesting you need to 🙂
17:55:34 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #10149: Doc: Prepare for 13.0-beta2 release https://github.com/OpenTTD/OpenTTD/pull/10149
17:56:02 <TallTyler> I'm sure there are other people eager to try and break things 🙂
17:56:22 <TallTyler> James, Zorg, Samu, your time has come 😛
17:56:56 <LordAro> TallTyler: traditionally (well, since git) the changelog has been in reverse PR# order (after feature/change/fix)
17:56:59 <TallTyler> I mean that seriously though, it's good to have people with a variety of skills and interests
17:57:18 <DorpsGek> [OpenTTD/OpenTTD] James103 commented on pull request #10149: Doc: Prepare for 13.0-beta2 release https://github.com/OpenTTD/OpenTTD/pull/10149#pullrequestreview-1179588728
17:57:27 <petern> Speaking of 🙂
17:57:38 <TallTyler> Oh, I've been ordering by "excitement level" 😛
17:57:55 <petern> Excitement level is valid IMHO.
17:57:55 <LordAro> yeah, doesn't factor in i'm afraid :p
17:58:01 <petern> Ah well.
17:58:01 <LordAro> save it for the news post
17:58:10 <petern> Truedat
17:58:30 <LordAro> is that TrueBrain's new data format?
17:58:31 <TallTyler> Well that's an easy change to make then
17:58:41 <petern> 😄
17:59:21 <petern> Oops, redefinition of default argument...
17:59:27 <LordAro> seems james didn't notice the )) in the line above the missing )
17:59:39 <LordAro> presumably the extra one was supposed to go on the line below
18:01:30 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #10149: Doc: Prepare for 13.0-beta2 release https://github.com/OpenTTD/OpenTTD/pull/10149
18:02:01 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10149: Doc: Prepare for 13.0-beta2 release https://github.com/OpenTTD/OpenTTD/pull/10149#issuecomment-1314163436
18:02:35 <LordAro> weren't we leaving #10110 out of the changelog? :p
18:03:39 <TallTyler> Arastais: Would you be insulted if we left that out as an easter egg?
18:03:58 <dwfreed> heh
18:04:01 <TallTyler> We could always leave it in the changelog and make a teasing reference in the news post
18:06:34 *** wallabra has quit IRC (Quit: ZNC 1.8.2 - https://znc.in)
18:07:55 *** wallabra has joined #openttd
18:09:19 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1041776861014917200/image.png
18:09:19 <petern> Is that... too much?
18:09:46 <petern> Maybe left-align it.
18:10:05 <petern> It's centred within a scaled bounding box currently
18:10:37 <TallTyler> Too much what? It looks fine to me
18:10:40 <petern> What other icons are shown there?
18:10:48 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
18:10:59 <TallTyler> Stop, and I think the lost icon?
18:11:29 *** WormnestAndroid has joined #openttd
18:11:41 <LordAro> is the "no power" icon the same as the lost icon?
18:11:45 <petern> Oops, I found a crash.
18:12:03 <LordAro> :o
18:13:06 <Arastais> TallTyler: nah, completely up to you guys
18:13:33 *** wallabra_ has joined #openttd
18:13:39 <Arastais> it can be our little secret 😛
18:13:44 <petern> Inside OnInitialPosition of ErrmsgWindow. Which is not something I've touched. Weird.
18:16:27 *** wallabra_ has quit IRC ()
18:17:36 *** wallabra_ has joined #openttd
18:18:17 *** wallabra has quit IRC (Ping timeout: 480 seconds)
18:18:17 *** wallabra_ is now known as wallabra
18:18:54 <petern> Ah, probably the window is too big to fit in the screen.
18:19:14 <LordAro> probably shouldn't crash regardless
18:19:35 <petern> Which easy easier to achieve with larger scaling available the touch of a slider.
18:19:42 <petern> Yes of course, just finding the cause 🙂
18:19:50 <LordAro> :)
18:21:08 *** HerzogDeXtEr has joined #openttd
18:21:14 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1041779861481603174/image.png
18:21:14 <petern> Okay, that's gotta be fixed!
18:21:44 *** Wolf01 has joined #openttd
18:28:13 <petern> (Hmm, doesn't crash in 12.x, but then again no asserts)
18:28:27 *** gelignite has joined #openttd
18:32:23 <petern> Ah std::clamp is C++17, that's probably why we don't use it.
18:38:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10172: Fix: Crash if error message window is too large for game window. https://github.com/OpenTTD/OpenTTD/pull/10172
18:40:43 <TallTyler> Wonder if a related fix would work for #9632
18:42:06 *** Flygon has quit IRC (Read error: Connection reset by peer)
18:50:18 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/cbb2d5303d2496bb88a1bdeeae9b89131a7180e5
18:50:19 <DorpsGek> - Update: Translations from eints (by translators)
18:50:48 <LordAro> or indeed the linked #8869
18:55:54 <petern> Alternatively, use std::clamp where we want the assert, and remove the assert from Clamp()
18:56:54 <LordAro> except std::clamp doesn't assert, it's just undefined
18:56:56 <LordAro> which is much worse
18:57:15 <petern> Oh yes, misremembered.
18:57:32 <petern> That's... one reason to not use std::clamp. How silly.
18:59:37 <LordAro> perhaps 2 clamp functions - one with constexpr low/high values that you can static_assert are the right way round, and another that uses min/max combination
18:59:48 <LordAro> *the former then using std::clamp
19:02:50 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #10173: Change: Allow building canal by area outside editor https://github.com/OpenTTD/OpenTTD/pull/10173
19:24:15 <petern> I can only imagine how much money you can accidentally spend with that 🙂
19:33:12 <andythenorth> can we have modifier combos?
19:33:17 <andythenorth> like shift-ctrl-click?
19:37:56 <nielsm> that just shows the cost of the ctrl-click action
19:48:23 <TallTyler> I think most ship newgrfs make canals significantly more affordable
20:01:39 *** DDR has joined #openttd
20:13:53 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
20:55:10 <DorpsGek> [OpenTTD/OpenTTD] tomkcook commented on issue #10008: [Bug]: Ctrl modifier not detected https://github.com/OpenTTD/OpenTTD/issues/10008
21:15:15 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:27:50 <andythenorth> we can't use alt 😛
21:27:56 <andythenorth> I've been told before
21:28:38 *** Wormnest has joined #openttd
21:28:58 *** WormnestAndroid has joined #openttd
21:50:41 <petern> I mean... you can 😉
21:51:44 <andythenorth> some people don't have alt? 😛
21:51:54 <petern> What key do you use to get into the Cheat menu?
21:52:08 <andythenorth> 'option'
21:52:09 <andythenorth> 😛
21:52:43 <petern> Well if your hardware will mislabel the Alt key...
21:52:56 <petern> `Hotkey(WKC_CTRL | WKC_ALT | 'C', "cheats", MTHK_CHEATS),`
21:53:49 <petern> The problem with alt is that mouse-button-down and dragging can be caught by a window manager.
21:54:12 *** sla_ro|master has quit IRC ()
21:54:46 <petern> So using alt combined with mouse-based operations would be problematic as a default.
21:54:46 <andythenorth> oops
21:54:50 <andythenorth> ok
21:56:57 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1041834143107010610/image.png
21:56:57 <petern> Such chunk
21:58:27 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1041834524675420160/image.png
21:58:27 <petern> Position is good at 1x, except I can't see it 😄
21:58:57 <andythenorth> chunky flip-vehicle-in-depot
21:59:11 <andythenorth> but with a different hotkey, which increments a byte and rolls over
21:59:40 <andythenorth> a long time ago I was going to patch that destination / speed display
21:59:56 <andythenorth> for this case https://www.youtube.com/watch?v=d6-a4qjgY1Q
22:00:08 <andythenorth> hmm no copy-paste failed
22:00:16 <andythenorth> oh I didn't hold ctrl 😛
22:00:19 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1041834996350070855/image.png
22:00:38 <andythenorth> Moving the string code for speed to the front of the string would stop the speed being hidden
22:00:50 <andythenorth> I did the patch, but there was some edge case, and my attention span failed
22:01:09 <andythenorth> then I think someone complained in the channel about it needing state machines or something
22:01:10 <andythenorth> 🙂
22:01:14 <andythenorth> so I didn't return to it
22:02:27 <petern> Oh, make it a ticker 😄
22:03:19 <andythenorth> marquee scroll?
22:09:58 <petern> <marquee>
22:10:48 <DorpsGek> [OpenTTD/OpenTTD] James103 opened issue #10174: [Bug]: Error messages are bigger as of the recent UI changes https://github.com/OpenTTD/OpenTTD/issues/10174
22:16:52 <DorpsGek> [OpenTTD/OpenTTD] Arastais updated pull request #10104: Update: [Dev] Improve .editorconfig with C++ formatting rules based on OpenTTD code style https://github.com/OpenTTD/OpenTTD/pull/10104
22:21:19 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #10174: [Bug]: Error messages are bigger as of the recent UI changes https://github.com/OpenTTD/OpenTTD/issues/10174
22:34:45 *** Speedy` has joined #openttd
22:34:47 *** Speedyn has quit IRC (Ping timeout: 480 seconds)
22:39:17 *** gelignite has quit IRC (Quit: Stay safe!)
23:09:09 *** Tirili has joined #openttd
23:23:37 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
23:28:26 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
23:31:40 <petern> Hmm left or centre... not sure.
23:36:12 <glx[d]> petern: `set(CMAKE_CXX_STANDARD 17)` so we could use it actually
23:36:53 <petern> Yes I meant it's not that long ago that we did (IIRC) so wouldn't have got around to using all the things yet.
23:37:15 <petern> But anyway, the undefined behaviour is a bit of a bum.