⏴ go to previous day
00:03:20 *** Tirili has joined #openttd
00:12:07 <peter1138> FFS.
00:12:11 <peter1138> It's still there :D
00:22:54 <peter1138> Okay, it's gone now.
00:26:49 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1343016179920273428/image.png?ex=67bbbcc9&is=67ba6b49&hm=6e68dc191f053834330fcde636539fa980a5d715fb6ae64351a02fa39a1d47bb&
00:35:30 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1343018363630129204/image.png?ex=67bbbed1&is=67ba6d51&hm=9c01da63e9d0fc27206aa02d261afe225e67b3e54ba5d639249b3e13eac9daf4&
00:35:30 <peter1138[d]> Kinda wide.
01:02:03 *** fairyflossy has quit IRC (Quit: User went offline on Discord a while ago)
02:12:48 *** gelignite is now known as Guest9790
02:12:51 *** gelignite has joined #openttd
02:20:11 *** Guest9790 has quit IRC (Ping timeout: 480 seconds)
03:18:09 *** Wormnest has quit IRC (Quit: Leaving)
03:38:28 *** D-HUND has joined #openttd
03:41:49 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:51:14 *** Tirili has quit IRC (Remote host closed the connection)
04:31:50 *** gelignite has quit IRC (Quit: Stay safe!)
05:02:17 *** keikoz has joined #openttd
05:58:36 <wensimehrp> https://cdn.discordapp.com/attachments/1008473233844097104/1343099677272309884/image.png?ex=67bc0a8c&is=67bab90c&hm=41d1d9717171590c33c84cb82d793b78aff050cb84c5f72e5f1c1b4dacb590dd&
05:58:36 <wensimehrp> badges are fun
06:36:44 *** brickblock19280 has joined #openttd
06:36:45 <brickblock19280> Are they searchable?
06:40:06 <wensimehrp> yes!
06:41:27 <wensimehrp> brickblock19280: https://github.com/OpenTTD/OpenTTD/pull/13588
07:15:21 *** Wolf01 has joined #openttd
08:01:04 <peter1138> Well, maybe it needs a scrollbar.
08:08:34 *** HerzogDeXtEr has joined #openttd
08:22:47 <andythenorth> coffee time?
08:27:26 <peter1138> Yes, I have coffee.
08:27:55 <LordAro> good idea
08:28:34 <peter1138> Sadly no patisserie :(
08:28:59 *** kuka_lie has joined #openttd
08:29:38 <LordAro> peter1138: i seem to have bruised the top of my fibula
08:30:01 <LordAro> silly winter bike not being set up correctly
08:30:05 <peter1138> Uh oh.
08:30:37 <peter1138> My winter bike is my most comfortable bike, but... it's very heavy.
08:32:40 <andythenorth> I have a pasta del nata
08:32:42 <andythenorth> from Tesco
08:32:52 <andythenorth> I need coffee though
08:42:28 <peter1138> Hmm, `std::string x; x += a; x += b; x += c; DrawString(x);` vs `std::stringstream x; x << a; x << b; x << c; DrawString(x.str());`
08:42:55 <peter1138> From what I understand, stringstream has to copy out the string at the end.
08:43:25 <peter1138> Appending to a string may need to reallocate, but then again writing to a stringstream also needs to alocate.
08:54:12 <peter1138> Okay, updating dsda-doom didn't improve performance. Back to OpenTTD.
08:55:49 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646
08:55:55 <andythenorth> back to Horse
08:57:28 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #13639: Codechange: remove C-style memory management https://github.com/OpenTTD/OpenTTD/pull/13639
09:03:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#issuecomment-2676719219
09:03:26 *** merni has joined #openttd
09:03:26 <merni> peter1138: Lol I was breaking my head on that not so long ago
09:03:38 <merni> Order strings are complicated
09:23:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647
09:27:02 <peter1138> Well.
09:35:20 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#pullrequestreview-2635489301
09:36:14 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #13639: Codechange: remove C-style memory management https://github.com/OpenTTD/OpenTTD/pull/13639#pullrequestreview-2635489501
09:36:31 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#issuecomment-2676740501
09:46:34 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646
09:51:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#issuecomment-2676746394
09:51:23 <peter1138> Hmm.
09:51:23 *** nielsm has joined #openttd
09:52:35 <peter1138> Pom te pom.
09:53:22 <peter1138> Alright, it builds...
09:55:19 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647#issuecomment-2676747888
09:57:46 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647
09:58:50 <peter1138> Maybe I can fix STR_ORDER_IMPLICIT too, hmm.
10:01:42 <peter1138> F
10:02:47 <peter1138> Or I could replace `line = GetString(timetable ? STR_ORDER_GO_TO_STATION : STR_ORDER_GO_TO_STATION_IMPLICIT, STR_ORDER_GO_TO, order->GetDestination());`
10:02:52 <peter1138> line = GetString(STR_ORDER_GO_TO_STATION, STR_ORDER_GO_TO, order->GetDestination());
10:02:55 <peter1138> if (!timetable) {
10:02:58 <peter1138> line += ' ';
10:03:00 <peter1138> line += GetString(STR_ORDER_IMPLICIT);
10:03:03 <peter1138> }
10:03:07 <peter1138> Would keep the existing string.
10:05:01 <xarick> hi
10:09:40 *** D-HUND is now known as debdog
10:17:05 <peter1138> Hmm, well, I realised this "adding spaces" thing is probably bad for CJK languages.
10:25:54 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#pullrequestreview-2635504919
10:27:07 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1343167249632067734/image.png?ex=67bc497b&is=67baf7fb&hm=7d241457f601caf49f1ea580b6fb1d9448640771c178bc92d6f2dcfd6d0ab413&
10:27:31 <peter1138> I think this will work better, though it might confuse translators.
10:28:11 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1343167517505359872/image.png?ex=67bc49bb&is=67baf83b&hm=3d69009e2e0e262b2c461b835d148aebfcf650358e7197de2d7af91b18075cdb&
10:28:43 <peter1138> Works better for CJK which doesn't need a space because it uses （ and ）.
10:29:16 <peter1138> (That {POP_COLOUR} shouldn't have a space after it.)
10:31:49 <_glx_> Not only this tag I'd say
10:34:18 <peter1138> I remember now, the other string concatenation I did uses newlines, which are not problematic like spaces.
10:35:41 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647
10:35:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646
10:37:03 <peter1138> Wonder if I need to ":(" → ": (" for other languages.
10:37:19 <peter1138> Does eints trim strings?
10:40:44 <peter1138> CodeQL is being correct but awkward there...
10:59:49 <frosch123> It may trim on the right. There is {SPACE} to not trim
11:00:35 <peter1138> Yeah, I added that because it's very easy to miss a trailing space.
11:05:10 <frosch123> Did anyone try to use icons for non-stop/stop-everywhere, full-load/load-any, deliver/transfer?
11:07:48 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647
11:08:02 <peter1138> merni was looking at alternative ways to display this.
11:09:39 <peter1138> No code afaik though.
11:18:45 <frosch123> No icons in jgrpp
11:19:11 <frosch123> But just saw action123 for rocks
11:23:32 <peter1138> Yes, I saw that and dismissed it.
11:28:52 <peter1138> It does not follow the normal idioms for NewGRF features.
11:29:46 <peter1138> I have "ideas" about a proper landscape feature, but I'm not side-questing that yet.
11:31:26 <andythenorth> recursive quest
11:31:34 <andythenorth> how about we make a game?
11:32:32 <peter1138> Who is "we" here?
11:34:43 <andythenorth> yes
11:34:52 <andythenorth> we could make factorio?
11:37:30 <andythenorth> hmm
11:37:46 <andythenorth> do we care about more spritesheet pngs vs fewer?
11:38:20 <andythenorth> there are 4k vehicle spritesheets currently, all processed during the compile
11:39:13 <andythenorth> there are 10k vehicle variants, variants of the same model use a single consolidated spritesheet
11:39:36 <andythenorth> one spritesheet per variant would be simpler in some ways, but is much more IO
11:39:50 <brickblock19280> What is the limit on the amount of newgrf features?
11:40:42 <peter1138> Imagination.
11:41:11 <michi_cc> And if you mean that literally, 256.
11:44:00 <peter1138> Unless you give it the extended byte treatment.
11:44:58 <brickblock19280> michi_cc: Okay, good
11:45:52 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#pullrequestreview-2635534297
12:03:30 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646
12:08:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647#pullrequestreview-2635537143
12:17:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647
12:17:14 <talltyler> frosch123: No code, but I did some investigation into other games: https://github.com/OpenTTD/OpenTTD/discussions/9551
12:22:14 <xarick> cool, let's see https://www.youtube.com/watch?v=XYVyboKMZm0
12:28:30 <peter1138> talltyler, I'd like to do the "OrderList is first" approach, where you create the order list, then add vehicles to it.
12:28:52 <peter1138> That does the "shared orders" concept in a much better way.
12:29:21 <peter1138> Plus OrderList pool is already a thing, so the data structures already exist.
12:31:44 <xarick> hmm my poor AI didn't make it into the competition
12:32:11 <xarick> probably crashed or something?
12:47:11 <peter1138> talltyler, then we can add the ability to view orders, and steal JGRPP's viewport order display idea.
12:52:26 <peter1138> Hmm.
12:53:55 <andythenorth> is it bad to use classes for singletons?
12:54:00 <andythenorth> I should just use a dict?
12:56:58 <LordAro> xarick: mine neither :(
13:04:36 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647#pullrequestreview-2635576534
13:08:59 <_jgr_> peter1138: I can't take credit for that idea or the code, it's from here: <https://www.tt-forums.net/viewtopic.php?t=53394>
13:18:58 <xarick> wow
13:19:00 <xarick> stuff from 2011
13:22:44 <xarick> gonna run the same AIs as masterhellish did in the video
13:23:11 <xarick> wondering what settings he used
13:23:37 *** masterhellish has joined #openttd
13:23:37 <masterhellish> 👋
13:23:41 <xarick> NoCAB was a loser is really mindboggling
13:25:26 <masterhellish> https://cdn.discordapp.com/attachments/1008473233844097104/1343212122947190865/image.png?ex=67bc7345&is=67bb21c5&hm=5e80af24c0f1a35fc3527ab149909cbbd0b07cde82c06da20d7703cbb950c942&
13:25:26 <masterhellish> xarick:
13:26:16 <xarick> masterhellish: oh, thanks for that
13:32:59 <xarick> not sure I should use my own branch to make the test...
13:34:29 <xarick> which version of SuperSimpleAI was used? not the 54 I guess
13:35:41 <xarick> Jaume should really upload to the online content 😦
13:41:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646#pullrequestreview-2635596187
13:41:11 <xarick> oh I can imagine why NoCAB failed
13:41:15 <xarick> it eats memory
13:41:35 <xarick> easily crashes at the default 1 GB mem limit
13:44:14 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #13646: Codechange: remove SimpleSpriteAllocator https://github.com/OpenTTD/OpenTTD/pull/13646
13:44:17 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #13641: Codechange: explicitly initialise member variables of Windows 1 of 5 https://github.com/OpenTTD/OpenTTD/pull/13641#pullrequestreview-2635596825
13:51:38 <peter1138> Balls, stupid tablet detected a phantom fingerprint and it landed on a mine :(
13:52:24 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #13647: Codechange: Use local string parameters for order and timetable windows. https://github.com/OpenTTD/OpenTTD/pull/13647
13:52:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #12267: [Bug]: Space after destination in Order and Timetable windows https://github.com/OpenTTD/OpenTTD/issues/12267
13:53:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #13641: Codechange: explicitly initialise member variables of Windows 1 of 5 https://github.com/OpenTTD/OpenTTD/pull/13641
13:55:15 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #13648: Codechange: Use member initialisation and pass by reference for TileDesc. https://github.com/OpenTTD/OpenTTD/pull/13648
13:57:10 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13649: Codechange: explicitly initialise member variables of Windows https://github.com/OpenTTD/OpenTTD/pull/13649
13:58:57 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
13:59:01 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13650: Codechange: replace C-style memory management in MacOS code https://github.com/OpenTTD/OpenTTD/pull/13650
14:01:23 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13651: Codechange: explicitly initialise member variables of Windows (3 of 5) https://github.com/OpenTTD/OpenTTD/pull/13651
14:06:43 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #13650: Codechange: replace C-style memory management in MacOS code https://github.com/OpenTTD/OpenTTD/pull/13650#pullrequestreview-2635603888
14:06:56 <peter1138> Eh, maybe not, it doesn't build :s
14:07:38 <peter1138> But in principle... :p
14:09:20 <Rubidium> well.. I didn't expect the first push to build
14:10:18 <peter1138> Indeed.
14:11:18 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 dismissed a review for pull request #13650: Codechange: replace C-style memory management in MacOS code https://github.com/OpenTTD/OpenTTD/pull/13650#pullrequestreview-2635603888
14:11:21 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #13650: Codechange: replace C-style memory management in MacOS code https://github.com/OpenTTD/OpenTTD/pull/13650
14:13:33 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #13648: Codechange: Use member initialisation and pass by reference for TileDesc. https://github.com/OpenTTD/OpenTTD/pull/13648#pullrequestreview-2635606952
14:19:50 *** Rubidium_ has joined #openttd
14:21:42 *** Rubidium has quit IRC (Ping timeout: 480 seconds)
14:23:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13648: Codechange: Use member initialisation and pass by reference for TileDesc. https://github.com/OpenTTD/OpenTTD/pull/13648
14:26:02 <peter1138> Hmm.
14:26:13 <peter1138> Maybe I should get something for food tomorrow.
14:39:00 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13652: Codechange: explicitly initialise member variables of Windows (4 of 5) https://github.com/OpenTTD/OpenTTD/pull/13652
14:39:48 <xarick> I have ideas to make my AI better, but I'm so lazy
14:40:05 <xarick> and never get to implement them
14:41:02 <andythenorth> peter1138: get an small cheese
14:41:07 <andythenorth> and a octopus
14:42:59 <xarick> I thought of making 1 route to behave as seemingly 2 routes. one route does full load at A to B, the other does full load from B to A.
14:43:45 <xarick> i never managed to make full load at both stations to go smoothly
14:44:28 <xarick> i either get a bunch of vehicles clumping at one of the stations, and ratings going bad in the other
14:44:44 <xarick> because of cargo imbalances
14:46:20 <xarick> vehicle capacity usage efficiency is quite low for my AI
14:48:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13648: Codechange: Use member initialisation and pass by reference for TileDesc. https://github.com/OpenTTD/OpenTTD/pull/13648#pullrequestreview-2635616673
14:51:34 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #13653: Codechange: explicitly initialise member variables of Windows (5 of 5) https://github.com/OpenTTD/OpenTTD/pull/13653
15:14:27 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #13639: Codechange: remove C-style memory management https://github.com/OpenTTD/OpenTTD/pull/13639
15:14:29 *** aperezdc has quit IRC (Ping timeout: 480 seconds)
15:19:59 <peter1138> I got a cheese, but not an octopus.
15:29:24 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654
15:38:31 *** aperezdc has joined #openttd
15:59:26 *** Wormnest has joined #openttd
16:01:38 <peter1138> Hm.
16:06:01 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654
16:28:11 <talltyler> peter1138: Viewport order display doesn’t rely on “order list first,” does it?
16:28:44 <peter1138> No.
16:32:57 <talltyler> I’m also supportive of orders first. Sounds like it would break everybody’s spacebar heating, but worth it, IMO. I think the new order list should absorb both the concepts of shared orders and groups — although you could still allow custom groups of order lists, to take the place of today’s nested groups.
16:38:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654#pullrequestreview-2635666634
16:38:47 <peter1138> Yeah. It brings some coherence to other features that make more sense when you are just looking at orders as a whole.
16:39:15 <peter1138> Hmm.
16:45:58 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654#pullrequestreview-2635669721
16:47:13 <andythenorth> peter1138: and also not an cheese?
16:49:55 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1343263582447468615/image.png?ex=67bca332&is=67bb51b2&hm=1df2a8332e882cadeaf81c80622587c7b7e3f0a65280976e767ca02661cd9454&
16:49:55 <andythenorth> pff the big questions: are badges a root level grf concern, or am I moving badge.py into train dir? 😛
16:50:10 <andythenorth> I guess railtypes might have badges? 😮
17:04:19 <peter1138> They do, yes.
17:15:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654
17:37:57 *** gelignite has joined #openttd
17:48:28 <peter1138> Hmm. Windows, what...
17:52:12 <peter1138> Presumably stuck in a loop.
18:10:57 <wensimehrp> peter1138: No I guess?
18:10:57 <wensimehrp> 🙂
18:10:57 <wensimehrp> ：-）
18:10:57 <wensimehrp> Can't type a em dash on this keyboard
18:12:21 <talltyler> No keypad for alt codes?
18:15:55 <wensimehrp> I never used alt codes.
18:15:55 <wensimehrp> Usually I just google "em dash" and copy the first result.
18:15:55 <wensimehrp> Or when I'm using typst there is a short hand for a em dash. I think it was `---`
18:16:55 <peter1138> No what?
18:17:17 <wensimehrp> > Wonder if I need to ":(" → ": (" for other languages.
18:17:36 <wensimehrp> Can you see discord replies on IRC?
18:21:25 <peter1138> Some languages shouldn't have a space.
18:21:31 <peter1138> Some languages might need the space at the end.
18:27:56 <xarick> translators gonna have a bad time again
18:35:42 <peter1138> Hmm, no, seems it needs to be at the front even for RTL languages.
18:43:24 <peter1138> Where is Eddi these days?
18:49:14 *** tokai has joined #openttd
18:49:14 *** ChanServ sets mode: +v tokai
18:56:10 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
19:09:57 <peter1138> If it wasn't for that pesky string parameter...
19:16:31 <Rubidium_> well... won't consists solve the issue as autorenew/autoreplace probably doesn't need to replace the consist object?
19:17:00 <peter1138> I...
19:17:34 <peter1138> ConsistPool was not a side quest of mine... yet.
19:18:01 <peter1138> (I have some consist stuff but it's not poolerised. Hmm.
19:18:03 <peter1138> )
19:18:10 <peter1138> I wonder how much effort it would be to do so...
19:18:11 <peter1138> Oh no.
19:18:54 <Rubidium_> I'd just try fixing ReplaceParam and move the consist side-quest post 15
19:19:29 <peter1138> Yeah. It works for me with clang, and with mingw. So it's something different with MSVC.
19:20:39 <Rubidium_> might be MSVC being very pedantic with their iterators
19:21:31 <peter1138> Yeah, that's why I convert them to `const char *` before use.
19:22:19 <peter1138> (`std::from_chars` requires `const char *`)
19:27:29 *** warhapper has joined #openttd
19:27:29 <warhapper> https://cdn.discordapp.com/attachments/1008473233844097104/1343303236957900831/image.png?ex=67bcc821&is=67bb76a1&hm=3e4d684ffe484de1bebff17ea8f57725fcdb7a09e6ff6f09721fd156133fc1f5&
19:31:17 <Rubidium_> peter1138: I reckon that string.end() is not the pointer of the last character, but some sentinel value. So the while loop doesn't terminate
19:31:32 <Rubidium_> (for MSVC 'debug' that is)
19:38:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654
19:41:34 <michi_cc> peter1138: Well, you may freely take things from https://github.com/michicc/OpenTTD/tree/consist 😛
19:42:04 <michi_cc> Got a pool and everything 🙂
19:46:36 *** geertop has joined #openttd
19:46:36 <geertop> warhapper: Lmao
19:49:17 <xarick> is Valuate already multi-tick?
19:49:43 <xarick> i had the impression Valuate never suspended
19:52:51 <peter1138> Rubidium_, yeah, Windows passes now.
19:54:23 <Rubidium_> yay
19:54:27 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654#pullrequestreview-2635731445
19:54:28 <peter1138> Well, thank, er, me? that I decided to add a unit test...
19:55:40 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#pullrequestreview-2635731621
19:56:05 <Rubidium_> would have been a fun bug report... OpenTTD randomly and spontaneously hanging when developing with MSVC
20:00:51 <xarick> oh, 13556 is interesting
20:01:10 <xarick> gonna test
20:03:00 <xarick> saving ScriptLists is such a boon!
20:03:07 <xarick> wasn't it tried before?
20:03:13 <xarick> like a year ago
20:10:55 <johnfranklin> do you use auto word wrap in IDE?
20:23:09 <peter1138> Not usually.
20:24:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #13654: Codechange: Use EncodedStrings for News messages. https://github.com/OpenTTD/OpenTTD/pull/13654
20:24:56 <johnfranklin> I just found it; it is much necessary when typing content-rich HTML.
20:29:08 <wensimehrp> Markdown eh
20:34:52 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
20:35:54 <xarick> no, still doesn't work
20:35:57 <xarick> 13556
20:36:09 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1343320518459392071/image.png?ex=67bcd839&is=67bb86b9&hm=c1a597cdd8ad64d7f53ef78a2e91e16bf8d6553db3ada60374420cac897be948&
20:36:45 <xarick> something else needs to be saved, the type of the list
20:39:16 <xarick> can't believe it's been 2 years since last time this was tried
20:42:18 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#issuecomment-2677106454
20:47:23 *** gelignite has quit IRC (Quit: Stay safe!)
20:50:14 <_glx_> it saves lists as "List", ie basic list
20:51:58 <_glx_> I didn't think about list with extra functions
21:29:18 <DorpsGek> [OpenTTD/OpenTTD] glx22 dismissed a review for pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#pullrequestreview-2635731621
21:29:21 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556
21:29:42 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#issuecomment-2677126322
21:30:30 <_glx_> this framework is so easy to work with
21:30:33 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#issuecomment-2677126717
21:31:00 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#issuecomment-2677126903
21:31:10 *** kuka_lie has quit IRC (Quit: Lost terminal)
21:31:19 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #13556: Add: [Script] Framework for loading/saving selected ScriptObject https://github.com/OpenTTD/OpenTTD/pull/13556#issuecomment-2677127347
21:32:06 <_glx_> but should be fine now, ScriptTileList (and derivates) is the only ScriptList with extra functions
21:44:31 <_glx_> xarick: it works for me now, feel free to retest
21:53:06 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:54:32 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:02:41 *** Wormnest has joined #openttd
22:16:21 *** Flygon has joined #openttd
22:24:19 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:24:48 <andythenorth> naptime?
22:28:27 <peter1138> :badger: time?
22:48:17 <johnfranklin> 🍄time?
22:55:37 *** reldred has joined #openttd
22:55:37 <reldred> Now now, the weekend is just ending it’s not time get on the shrooms now
22:56:14 <belajalilija> i think they're out of season tbg
22:56:37 <reldred> Eh depends where you are and who you know
23:00:55 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #13648: Codechange: Use member initialisation and pass by reference for TileDesc. https://github.com/OpenTTD/OpenTTD/pull/13648
23:15:25 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
23:22:07 <peter1138> Hmm, using a different StringID just to select a different colour for a substring?
23:36:47 <_glx_> there is {COLOUR} for that now
23:44:51 <DorpsGek> [OpenTTD/OpenGFX] jayaddison commented on issue #94: gimpscript: GIMP3 compatibility problem(s) https://github.com/OpenTTD/OpenGFX/issues/94
continue to next day ⏵
