IRC logs for #openttd on OFTC at 2024-12-20
            
00:04:34 *** Wormnest has joined #openttd
00:54:26 <_glx_> ```Warning: /home/runner/work/OpenTTD/OpenTTD/src/core/pool_type.hpp:263:53: warning: ‘*(Vehicle*)this.Vehicle::Pool<Vehicle, unsigned int, 512, 1044480>::PoolItem<(& _vehicle_pool)>.Pool<Vehicle, unsigned int, 512, 1044480>::PoolItem<(& _vehicle_pool)>::index’ is used uninitialized [-Wuninitialized]
00:54:26 <_glx_> 263 | assert(pn == Tpool->Get(pn->index));
00:54:26 <_glx_> | ~~~~^~~~~
00:54:34 <_glx_> this warning is so weird
00:55:23 <_glx_> the code is ```c++
00:55:23 <_glx_> inline void operator delete(void *p)
00:55:23 <_glx_> {
00:55:23 <_glx_> if (p == nullptr) return;
00:55:23 <_glx_> Titem *pn = static_cast<Titem *>(p);
00:55:25 <_glx_> assert(pn == Tpool->Get(pn->index));
00:55:25 <_glx_> Tpool->FreeItem(pn->index);
00:55:27 <_glx_> }
00:55:55 <_glx_> but it warns only for the assert, and only in release (using our custom assert)
01:51:45 <_glx_> hmm if I comment the assert it warns on FreeItem()
01:59:07 *** bohaska has joined #openttd
01:59:07 <bohaska> is there a particular reason why all map sizes have to be in multiples of 2
02:57:07 <_glx_> computers like power of 2
03:16:27 *** gnu_jj_ has joined #openttd
03:19:36 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
03:37:22 *** debdog has joined #openttd
03:40:56 *** D-HUND has quit IRC (Ping timeout: 480 seconds)
03:41:34 *** Wormnest has quit IRC (Quit: Leaving)
04:44:23 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/c91d4104f88126f05cc189a308a92c755247e398
04:44:24 <DorpsGek> - Update: Translations from eints (by translators)
04:58:26 *** keikoz has joined #openttd
05:25:55 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/12426027790
05:46:13 <peter1138> Oh dear
05:51:11 <wensimehrp> https://github.com/OpenTTD/OpenTTD/pull/11766 ? 😛
05:53:11 <peter1138> I should close that, it doesn't work
05:53:44 <wensimehrp> > We might even want to consider automatically adding {ZWSP} in strgen whenever one of the non-ASCII punctuations comes up in the string and there's no colour command after it.
05:55:47 <wensimehrp> wensimehrp: or maybe I shouldn't send this message. This should be handled by the layouter anyways
07:43:53 *** Flygon has quit IRC (Read error: Connection reset by peer)
08:22:03 <andythenorth> is the first meal of the day 'lunch'?
08:22:05 <andythenorth> I am hungry
08:33:59 <LordAro> andythenorth: i have coco pops
08:46:24 <andythenorth> I have porridge
08:46:27 <andythenorth> non-standard, but eh
08:46:30 <andythenorth> everything else ran out
08:51:17 <belajalilija> I’m having sandwich
08:54:23 <wensimehrp> Vanilla Bing qi ling
09:14:51 <peter1138> Pardon?
09:15:33 <Artea> kernel bagel with php sausage
09:21:04 <andythenorth> coffee
09:22:53 <Artea> andythenorth, sorry as you... but you already tried Railroad Tycoon ?
09:26:18 <belajalilija> peter1138: I think it means vanilla ice cream
09:44:45 <xarick> hi
10:08:49 <andythenorth> Artea: we what now?
10:08:50 <andythenorth> 🙂
10:36:56 *** tokai has quit IRC (Quit: c('~' )o)
10:38:11 *** SigHunter has quit IRC ()
10:41:26 *** SigHunter has joined #openttd
11:32:59 <_glx_> Pff I fix releases and translators break it
12:06:26 <Artea> andythenorth: * sorry to ask, but you ever played Railroad Tycoon ?
12:13:01 <xarick> Okay, I see what I need to do
12:13:09 <xarick> memory refreshed
12:13:55 <xarick> unify behaviour
12:14:25 <xarick> kinda like I did for my other attempt
12:21:09 <xarick> I can't build:
12:21:09 <xarick> E:\OpenTTD Visual Studio\SamuXarick\OpenTTD\src\lang\danish.txt(2598): warning : Unwanted UTF-8 character U+200B in sequence 'ingen af ​​de anmodede godkendelsesmetoder er tilgængelige' (FATAL)
12:23:06 *** florafex has joined #openttd
12:23:06 <florafex> https://cdn.discordapp.com/attachments/1008473233844097104/1319641222163468330/image.png?ex=6766b329&is=676561a9&hm=e210ecadf0768c811b07dcf0d85a09e80a39827832bbb790156b0952bf360958&
12:23:06 <florafex> xarick: you can temporarily change that line in danish.txt if you just need it to build for other reasons
12:24:10 <xarick> what is the unwanted character?
12:25:47 <florafex> two zero width spaces between 'af' and 'de'
12:26:47 <xarick> oh, interesting, there's something fishy there
12:40:59 *** tokai has joined #openttd
12:40:59 *** ChanServ sets mode: +v tokai
12:42:39 <florafex> question that i imagine an experienced ttd dev might be able to answer:
12:42:39 <florafex> if i have just an EngineID, is there any reasonable way to know the length of the vehicle it represents, factoring in articulation? Specifically this'd be about trains - I'm assuming not, due to how articulated vehicles are, but I figured I'd ask anyway
12:44:51 <florafex> https://cdn.discordapp.com/attachments/1008473233844097104/1319646698200502415/image.png?ex=6766b842&is=676566c2&hm=a0273c55fb764f982ef442785013b2b06692aeae4d3d871e3c41184053983680&
12:44:51 <florafex> i say 'reasonable' as i wanted to sort the train list by length, and i'm assuming "instantiating a train of it and measuring the resulting train" enters the realm of 'unreasonable' - and i'm guessing this is a contributor as to why the game *doesn't* have length as a sort option
12:45:14 *** kuka_lie has joined #openttd
12:46:19 <belajalilija> i was just thinking about length earlier
12:46:54 <belajalilija> something that states the length in tiles on the purchase menu would be neat
12:51:42 <_glx_> There's no easy way to get engine length, as it can depend on callbacks and they are only for vehicles
12:52:58 <peter1138> Yup, lots of things are dynamic. This is also why sometimes the displayed information isn't what you get.
12:53:25 <florafex> figures. thanks for the quick replies you two
12:57:32 <andythenorth> probably lunch
12:57:43 <andythenorth> Artea: yes lots, 1, 2 and 3
12:57:47 <andythenorth> many many hours
13:43:02 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #13177: Doc: fix typo in script_vehicle.hpp https://github.com/OpenTTD/OpenTTD/pull/13177
13:43:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #13175: Codefix: Remove duplicated include https://github.com/OpenTTD/OpenTTD/pull/13175#pullrequestreview-2517582386
13:44:26 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #13176: Codefix: Ensure assertions are properly encapsulated within WITH_ASSERT directive https://github.com/OpenTTD/OpenTTD/pull/13176#pullrequestreview-2517583951
13:50:39 <Artea> RAWR
13:50:59 <Artea> I bought the Locomotive Tycoon and still didn't played lol
14:12:27 <peter1138> I've not heard of Locomotive Tycoon.
14:40:24 <_glx_> https://github.com/OpenTTD/OpenTTD/pull/12935 <-- and it's the same language again
14:42:04 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1319676199479283743/image.png?ex=6766d3bc&is=6765823c&hm=559e9b952c727be51a4239d5b48fd021c955973940503cc40fc04456558253e4&
14:42:04 <_glx_> not one, but two
14:44:46 <peter1138> Yup.
14:45:55 <peter1138> For me, VS Code helpfully shows nothing because it's a zero-width space...
14:46:16 <_glx_> VS showed nothing when I removed the first one
14:52:45 <belajalilija> looks like danish or norwegian
15:06:51 <Rubidium> _glx_: regarding the pool warning... the compiler might think that the value is not initialised in case the constructor fails (new called => constructor throws => delete called). If that's the case, maybe making the constructor noexcept solves the issue?
15:15:59 <peter1138> I was trying to use "Gnome Boxes" to set up a Debian testing/sid VM, and... every time it failed and said there was no suitable kernel to install :S
15:21:43 <peter1138> Hmm, maybe something doesn't allow its networking to network.
15:32:37 *** nielsm has joined #openttd
16:16:55 *** gelignite has joined #openttd
16:18:16 <xarick> is the danish situation resolved?
16:20:27 <xarick> nope :(/
16:27:21 <_glx_> it won't fix itself
16:29:03 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #13179: Fix c91d4104f8: \u200B is not allowed in translations https://github.com/OpenTTD/OpenTTD/pull/13179
16:29:45 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #13179: Fix c91d4104f8: \u200B is not allowed in translations https://github.com/OpenTTD/OpenTTD/pull/13179#pullrequestreview-2517906069
16:30:20 <talltyler> Fastest gun in the west
16:31:13 <_glx_> oh it was ready 2h ago but I had to go to airport to pickup someone
16:52:00 <_glx_> Rubidium: I think it's related to the empty default constructors, and the way we actually initialize PoolItem::index, through AllocateItem for only 2 operator new overloads
16:53:10 <_glx_> even for human it's hard to follow some pool stuff
16:59:55 <_glx_> it feels false positive to me
17:07:59 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #13179: Fix c91d4104f8: \u200B is not allowed in translations https://github.com/OpenTTD/OpenTTD/pull/13179
17:08:09 <Rubidium> it is, unless the compiler (re)initializes index in the constructor
17:16:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13180: Fix: no need to guard free() https://github.com/OpenTTD/OpenTTD/pull/13180
17:24:27 <peter1138> Yes, because it's a void pointer, converting it is a bit of a pain.
17:25:11 *** Wolf01 has joined #openttd
17:28:53 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #13180: Fix: no need to guard free() https://github.com/OpenTTD/OpenTTD/pull/13180#pullrequestreview-2518013808
17:45:31 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181
17:46:20 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181
17:48:10 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181#pullrequestreview-2518048907
17:49:42 <peter1138> Aww, mac OS.
17:50:22 <andythenorth> blame Tim Apple
18:08:14 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #13180: Fix: no need to guard free() https://github.com/OpenTTD/OpenTTD/pull/13180
18:26:23 <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181#pullrequestreview-2518048907
18:26:26 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181
18:45:49 *** Flygon has joined #openttd
18:46:12 *** tokai|noir has joined #openttd
18:46:12 *** ChanServ sets mode: +v tokai|noir
18:53:11 *** tokai has quit IRC (Ping timeout: 480 seconds)
18:59:34 *** Wormnest has joined #openttd
19:02:36 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181#pullrequestreview-2518162755
19:05:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #13181: Codechange: Use snake_case and structured binding for layouters' font map. https://github.com/OpenTTD/OpenTTD/pull/13181
19:22:47 <andythenorth> hmm
19:22:55 <andythenorth> did we do it yet?
19:27:13 <truebrain> yes
19:29:24 <peter1138> Dropped MacOS support?
19:30:00 <andythenorth> would also reduce newgrf support requests
19:30:07 <andythenorth> double benefit
19:40:29 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #13182: Change: Hide buttons in Found Town window that can't be used. https://github.com/OpenTTD/OpenTTD/pull/13182
19:42:47 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #13182: Change: Hide buttons in Found Town window that can't be used. https://github.com/OpenTTD/OpenTTD/pull/13182#issuecomment-2557621161
19:53:40 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #13182: Change: Hide buttons in Found Town window that can't be used. https://github.com/OpenTTD/OpenTTD/pull/13182#pullrequestreview-2518255401
20:40:49 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13182: Change: Hide buttons in Found Town window that can't be used. https://github.com/OpenTTD/OpenTTD/pull/13182
20:51:03 *** Tirili has joined #openttd
20:52:16 *** kuka_lie has quit IRC (Quit: Lost terminal)
20:58:53 *** nielsm has quit IRC (Ping timeout: 480 seconds)
20:59:26 *** Tirili has quit IRC (Remote host closed the connection)
21:08:35 *** Tirili has joined #openttd
22:50:46 <xarick> nice, thx, it builds now
22:56:07 *** Tirili has quit IRC (Quit: Leaving)
23:02:05 *** jinks has quit IRC (Quit: ZNC - http://znc.in)
23:02:25 *** jinks has joined #openttd
23:18:51 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)