Times are UTC Toggle Colours
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 02:12:48 *** gelignite is now known as Guest9790 02:12:51 *** gelignite has joined #openttd 02:20:11 *** Guest9790 has quit IRC 03:18:09 *** Wormnest has quit IRC 03:38:28 *** D-HUND has joined #openttd 03:41:49 *** debdog has quit IRC 03:51:14 *** Tirili has quit IRC 04:31:50 *** gelignite has quit IRC 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:45 *** 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:39 <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:51 <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:41 *** 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:50 <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:12 <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 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 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:35 <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 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:37 <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 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 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 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 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 21:54:32 *** nielsm has quit IRC 22:02:41 *** Wormnest has joined #openttd 22:16:21 *** Flygon has joined #openttd 22:24:19 *** keikoz has quit IRC 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 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