⏴ go to previous day00:18:11 *** WormnestAndroid has quit IRC (Remote host closed the connection)00:18:20 *** WormnestAndroid has joined #openttd01:43:32 *** WormnestAndroid has quit IRC (Remote host closed the connection)01:43:50 *** WormnestAndroid has joined #openttd01:59:50 *** Wormnest has joined #openttd02:28:02 *** gnu_jj_ has joined #openttd02:31:20 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)02:42:01 *** Wormnest has quit IRC (Quit: Leaving)03:55:48 *** WormnestAndroid has quit IRC (Remote host closed the connection)03:55:53 *** WormnestAndroid has joined #openttd04:08:07 *** keikoz has joined #openttd04:46:57 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/7bec2029fa6fc75be7edbd8e8a739710dc9a566004:46:58 <DorpsGek> - Update: Translations from eints (by translators)05:21:09 *** keikoz has quit IRC (Ping timeout: 480 seconds)05:45:33 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/1474733497506:16:25 *** Wolf01 has joined #openttd06:18:08 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris commented on pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8#issuecomment-284091883906:18:57 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13948: Codechange: Move ownership of Orders to OrderList. https://github.com/OpenTTD/OpenTTD/pull/1394806:25:05 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14167: Fix 261e09a549: Compilation failure with Fluidsynth enabled. https://github.com/OpenTTD/OpenTTD/pull/14167#pullrequestreview-280587692506:28:01 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14167: Fix 261e09a549: Compilation failure with Fluidsynth enabled. https://github.com/OpenTTD/OpenTTD/pull/1416706:30:29 <DorpsGek> [OpenTTD/OpenTTD-TTF] PeterN approved pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8#pullrequestreview-280588792006:33:05 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris commented on pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8#issuecomment-284094559506:33:15 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris merged pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/806:42:27 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris opened issue #9: The new shekel currency symbol is too tall https://github.com/OpenTTD/OpenTTD-TTF/issues/906:51:55 <truebrain> Okay the nightly this time is an actual build failure 🙂06:53:25 <DorpsGek> [OpenTTD/OpenTTD] Iris-Persephone updated pull request #14158: Add: "Many random towns" button in scenario editor now prompts for the number of towns, with defaults based on the current new game settings https://github.com/OpenTTD/OpenTTD/pull/1415807:21:40 <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris opened pull request #10: Feature: Hebrew Alphabet for OpenTTD Serif https://github.com/OpenTTD/OpenTTD-TTF/pull/1007:23:36 <peter1138> _zephyris, just in case you hadn't had enough of fonts... https://en.wikipedia.org/wiki/Vietnamese_alphabet07:25:26 <_zephyris> vietnamese07:25:26 <_zephyris> OpenTTD-Sans missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)07:25:26 <_zephyris> OpenTTD-Serif missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)07:25:26 <_zephyris> OpenTTD-Mono missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)07:25:26 <_zephyris> OpenTTD-Small missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)07:25:28 <_zephyris> OpenTTD-SmallCaps missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)07:25:36 <_zephyris> The problem is the double-stacked diacritics07:25:44 <peter1138> Yup.07:26:21 <_zephyris> Hmm, though they are mostly lower case double stacked...07:29:04 <_zephyris> Not sure how possible `Ấ` is in 10px 😉07:29:17 <Heiki> Ắ Ấ Ằ Ầ Ẳ Ẩ Ẵ Ẫ are all distinct characters07:31:14 <_zephyris> Yeah, the list above is just the ones used in OTTD translations07:32:13 <peter1138> My side-by-side fonts branch probably needs some logic to determine if OpenTTD Sans should come before or after the fallback autodetected font.07:34:22 <_zephyris> I think Vietnamese is the only Latin alphabet language not supported now07:34:22 <peter1138> For non-latin script, before works reasonably well.07:34:56 <_zephyris> How's it decided now?07:35:10 <peter1138> In master? It's not, you only get one font.07:36:54 <peter1138> And, still only experimenting. I didn't get as far as making this branch work on Windows or MacOS.07:37:26 <_zephyris> What does the side-by-side fonts do? Mixing characters from different fonts?07:38:40 <peter1138> Yes, there's some bits missing that make it properly useful though.07:39:05 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367042561276907520/image.png?ex=68132519&is=6811d399&hm=027314cf2fe227131763dc4e243fd8ab1ad560dbca78307ddfa5df48f20b2f1e&07:39:25 <peter1138> This is with `default;ChillRoundF`07:43:27 <peter1138> The fallback font system has a few issues, mainly it only checks for characters in the compiled language files.07:43:48 <peter1138> Anything in a savegame, online content, or servers is ignored.07:45:25 <peter1138> So it would be nice if the text system could discover missing glyphs and load fonts "on demand".07:46:01 <peter1138> But doing so shouldn't replace the current font for things that can be rendered.07:48:28 <peter1138> Not sure how feasible it is to actually scan and load for fonts on demand.07:49:26 <peter1138> What I am sure if is that changing the font configuration during text layouting (which is where unicode is mapped to glyphs) is not a great idea.07:49:35 <peter1138> But something deferred might work.07:51:40 <peter1138> Make a list of missing characters. Layout with usual missing glyph glyph. Then periodically check the list of missing characters and search for a font.07:52:55 <peter1138> Our text rendering flow is somewhat different to everywhere else, I think :)08:06:49 <_zephyris> I see the aim 🙂08:07:44 <_zephyris> Does look nice, but you would want to avoid things like Vietnamese where one character per word might fall back to a different font...08:08:30 <peter1138> Exactly.08:09:34 <peter1138> That's the context for my statement at 08:32 BST :-)08:35:32 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)08:36:20 *** mindlesstux has joined #openttd08:40:56 <xarick> hi08:43:35 <_zephyris> Hmm, need to learn how to do multiple combining diacritic anchor points...08:49:53 <_zephyris> https://cdn.discordapp.com/attachments/1008473233844097104/1367060381163589662/image.png?ex=681335b1&is=6811e431&hm=cec796082fd926fcc3c63e5a768be6cad0b2dfd9c7245aa1af0b9dc263f7371d&08:49:53 <_zephyris> 😉08:51:24 <peter1138> With these, I would aim to make the scalable font look good at 2x interface scale, rather than trying to squash it to fit into 1x.08:51:55 <peter1138> (But I'm not drawing it :))08:52:50 <_zephyris> Trust in anti-aliasing08:55:07 <_zephyris> Based on the translations only `ẤẦẨ` are the problematic ones.08:55:10 <_zephyris> https://cdn.discordapp.com/attachments/1008473233844097104/1367061708987826288/image.png?ex=681336ee&is=6811e56e&hm=63c534e1f1487d8e815347653681ebe0cc13f906e57c74ef470479b3662a76a1&08:56:36 <_zephyris> Unconventional to push the second diacritic to the side, but maybe it's readable?08:56:55 <_zephyris> `Ắ` etc. would need a very different strategy...09:08:26 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416909:15:45 <Heiki> “When a tone mark is used together with another diacritic, offsetting the tone mark to the right preserves consistency and avoids slowing down saccades.” https://en.wikipedia.org/wiki/Vietnamese_language_and_computers#Vietnamese_alphabet (and there’s also an example)09:15:51 <peter1138> Yes, I have some fonts that just go up & up (Arial), and others that place adjacent (ChillRoundF)09:21:20 <kuhnovic> Must be frustrating sometimes, working on fonts of languages you don't know / can't verify.09:32:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14165: Codechange: replace char* with C++ style strings https://github.com/OpenTTD/OpenTTD/pull/14165#pullrequestreview-280643619909:34:23 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416909:48:34 <_zephyris> 1000%09:52:33 <_zephyris> The offset is interesting, but I'd want to put it even more to the right than that example - ie. both diacritics directly above the letter09:53:00 <DorpsGek> [OpenTTD/OpenTTD] hpiirai opened issue #14170: [Bug]: Missing space in timetable window https://github.com/OpenTTD/OpenTTD/issues/1417009:54:50 <LordAro> "Muffinpooland"09:54:56 <LordAro> hilarous.09:55:01 <LordAro> +i09:55:55 <kuhnovic> I had to really double check, I didn't see the issue at first 😛09:56:06 <Heiki> haha09:56:42 <Heiki> (16 seconds from GitHub to DorpsGek, not bad)09:58:08 <xarick> I noticed that in the portuguese translation too, but it was fine in the english one, i thought it was just a bad translator job09:59:01 <xarick> let me dig the screenshot10:01:12 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367078327252156426/image.png?ex=68134668&is=6811f4e8&hm=38694eb724a830fa0e043036f202a0b11a76b521d716f6af8599708363b59cc3&10:01:12 <xarick> oh, actually...10:01:26 <xarick> didn't even notice10:03:04 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367078797777829959/image.png?ex=681346d8&is=6811f558&hm=6b9bd6ba863bdfcd515b7c7eff198e68d99942ac5c5b5261a2e682fd2e0bff8d&10:03:04 <xarick> yeh, this is what I noticed first, the portuguese affected. Should be "Munington e "10:03:18 <xarick> "Letown e"10:04:36 <xarick> "ficar" isn't quite the ideal word, but it's okay10:05:10 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14165: Codechange: replace char* with C++ style strings https://github.com/OpenTTD/OpenTTD/pull/1416510:12:20 <peter1138> 3Yeah, the non-estimated strings are missing {SPACE}, because I presumably had a malfunction.10:12:49 <peter1138> Not entirely keen on this bit of string concatenation though.10:15:45 <peter1138> The strings which are in brackets are (probably) fine.10:24:33 <peter1138> Nice, frosch123 removed the fetch_metadata macro on the sly, I've been failing to do that for ages :p10:44:57 <reldred> Load average: 23.69 15.68 6.5010:44:59 <reldred> hmmmmm10:49:05 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417110:52:27 <peter1138> Oh right, I have to actually switch to the PR branch to compile it, not just fetch it :D10:53:57 <goddess_ishtar> mood10:55:25 <LordAro> reldred: depends how many cores you have :p10:59:20 <_zephyris> https://cdn.discordapp.com/attachments/1008473233844097104/1367092955638140938/kerning-on-the-popes-tombstone-v0-rpdfp1tnvcxe1.png?ex=68135407&is=68120287&hm=09aecb54162f028abfe282590b7f92cc140764a837c13ae0125aa345d2f886d8&10:59:20 <_zephyris> kuhnovic: But, at least it's a small audience.... Not like messing up the kerning on the pope's tombstone.11:00:20 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-284160641811:03:11 <reldred> LordAro: twelve, but they're on someone elses probably very oversubscribed xeon silver11:03:50 <_zephyris> On the subject of basesets and versions - I've been poking the workflow to make ogfx2 releases happen11:04:03 <_zephyris> But, can't find where/how the workflow knows the version, so how it can find the build file11:04:18 * peter1138 attempts to debug the unit test... :o11:05:04 <LordAro> reldred: alas11:05:15 <LordAro> reldred: have you tried downloading more?11:05:41 <reldred> LordAro: yeah but it costs more money and I haven't gotten around to downloading more of that either11:08:18 <LordAro> reldred: try downloading some time11:09:53 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417111:10:16 *** keikoz has joined #openttd11:14:48 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416911:15:45 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-284165212111:17:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#pullrequestreview-280671724511:21:19 *** frosch123 has quit IRC (Remote host closed the connection)11:21:19 *** andythenorth has quit IRC (Remote host closed the connection)11:21:19 *** belajalilija has quit IRC (Remote host closed the connection)11:21:19 *** DorpsGek_vi has quit IRC (Remote host closed the connection)11:21:19 *** reldred has quit IRC (Remote host closed the connection)11:21:19 *** jfkuayue has quit IRC (Remote host closed the connection)11:21:19 *** talltyler has quit IRC (Remote host closed the connection)11:21:19 *** alpapilus has quit IRC (Remote host closed the connection)11:21:19 *** tabytac has quit IRC (Remote host closed the connection)11:21:19 *** peter1138[d] has quit IRC (Remote host closed the connection)11:21:19 *** honza_ has quit IRC (Remote host closed the connection)11:21:19 *** _glx_ has quit IRC (Remote host closed the connection)11:21:19 *** digitalfox has quit IRC (Remote host closed the connection)11:21:19 *** _jgr_ has quit IRC (Remote host closed the connection)11:21:19 *** _sara_08 has quit IRC (Remote host closed the connection)11:21:19 *** exceptik has quit IRC (Remote host closed the connection)11:21:19 *** _zephyris has quit IRC (Remote host closed the connection)11:21:19 *** jfs has quit IRC (Remote host closed the connection)11:21:19 *** sosmakaroni has quit IRC (Remote host closed the connection)11:21:19 *** kuhnovic has quit IRC (Remote host closed the connection)11:21:19 *** squirejames has quit IRC (Remote host closed the connection)11:21:19 *** locosage has quit IRC (Remote host closed the connection)11:21:19 *** brickblock19280 has quit IRC (Remote host closed the connection)11:21:19 *** zanooda2000 has quit IRC (Remote host closed the connection)11:21:19 *** truebrain has quit IRC (Remote host closed the connection)11:21:19 *** goddess_ishtar has quit IRC (Remote host closed the connection)11:21:19 *** xarick has quit IRC (Remote host closed the connection)11:21:36 *** DorpsGek_vi has joined #openttd11:36:50 <peter1138> Well.11:37:24 <LordAro> who killed the bridge this time?11:37:32 <LordAro> did andy try to post unicode again?11:43:22 <peter1138> ASCII only now, no unicode, we're Bri'ish.11:46:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14158: Add: "Many random towns" button in scenario editor now prompts for the number of towns, with defaults based on the current new game settings https://github.com/OpenTTD/OpenTTD/pull/14158#pullrequestreview-280677961711:49:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-284172374411:52:43 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417111:55:21 *** reldred has joined #openttd11:55:21 <reldred> peter1138: proper brexit geezers11:55:51 <peter1138> Brexit Benefits.11:56:28 <peter1138> UKP 350,000,000 to the NHS every week.11:56:44 <peter1138> Although maybe we should go back to L/S/D.11:56:58 <LordAro> "UKP"11:57:34 <peter1138> The symbol is non-ASCII :)11:57:54 <LordAro> yeah, but GBP isn't :p11:58:13 <peter1138> Normally.11:58:18 <peter1138> But frankly I had a moment.11:59:22 <peter1138> GREAT BREXIT POUNDS.12:03:08 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417112:11:10 <DorpsGek> [OpenTTD/OpenTTD] frosch123 closed pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416912:11:13 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-284177498012:11:22 <DorpsGek> [OpenTTD/OpenTTD] frosch123 reopened pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416912:11:43 <peter1138> Uh...12:11:44 *** frosch123 has joined #openttd12:11:44 <frosch123> Wrong button :)12:12:21 <peter1138> It's got to CI again ;(12:12:47 <frosch123> Is hard-coding the baseset short-names ok?12:26:33 <peter1138> Dunno, however as this now matches existing behaviour, any potential change can be done separately later.12:33:10 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416912:33:13 <DorpsGek> [OpenTTD/OpenTTD] frosch123 dismissed a review for pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#pullrequestreview-280671724512:33:26 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-284182897313:00:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#pullrequestreview-280697484113:23:31 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/1416913:25:32 *** WormnestAndroid has quit IRC (Remote host closed the connection)13:25:44 *** WormnestAndroid has joined #openttd13:26:54 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417113:31:13 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417113:36:42 *** talltyler has joined #openttd13:36:42 <talltyler> Infinite water could use a review if anyone is feeling inspired 🙂13:37:14 *** xarick has joined #openttd13:37:14 <xarick> there is a vehicle-details-tooltip branch on OpenTTD13:37:50 <xarick> and a dependabot etc...13:37:57 <xarick> https://github.com/OpenTTD/OpenTTD/tree/dependabot/github_actions/actions-c710d2a3e313:38:41 <xarick> is that a mistake13:40:42 <LordAro> the latter definitely is13:41:44 *** _glx_ has joined #openttd13:41:44 <_glx_> Dependabot branch is correct13:44:35 <peter1138> Easy to do.13:44:41 <LordAro> mm13:45:18 <LordAro> git remote set-url --push origin not-a-url :)13:46:10 <xarick> strange, industry generation is now fast again13:48:45 <xarick> one of these recent merges must have fixed something13:52:17 *** Flygon has quit IRC (Remote host closed the connection)14:25:08 *** kuhnovic has joined #openttd14:25:08 <kuhnovic> Must be the new power14:30:13 *** nielsm has joined #openttd14:30:58 <peter1138> Overclocked power supply.15:17:57 *** Wormnest has joined #openttd15:54:30 <xarick> enum still doesn't work, just tried15:55:20 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367167448771596308/image.png?ex=68139968&is=681247e8&hm=3a73ab584cf0c2bc36c170af5a5c414a3be0682b80eeb38a37f3510e7bdf6eff&15:55:20 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367167449056940092/image.png?ex=68139968&is=681247e8&hm=c5a6c86db9b9954f3f5ed6dbba0b40a62e2f6b4e7ce1e3e66308dba941dc7889&15:56:04 <xarick> am I doing something wrong?16:05:30 <_glx_> Maybe16:06:20 <LordAro> probably16:06:40 <_glx_> Syntax seems correct according to http://squirrel-lang.org/doc/squirrel2.html#constants16:07:18 <LordAro> there's no commas in that link16:07:41 <LordAro> oh, seems optional16:07:47 <LordAro> trailing comma/trailing semicolon, maybe?16:17:11 <xarick> reverted back to class16:32:33 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)16:32:48 *** Smedles has joined #openttd16:49:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171#pullrequestreview-280776455716:49:19 <_glx_> according to doc it should work, maybe there's an issue with `require()`16:59:57 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14121: Add: [NewGRF] Add purchase list Action3 CID for houses, industries, industry tiles, airports and airport tiles. https://github.com/OpenTTD/OpenTTD/pull/14121#pullrequestreview-280780637717:04:50 <_glx_> I remembered something like that happening before, and it's supposed to be fixed since <https://github.com/OpenTTD/OpenTTD/pull/10573>17:13:12 <_glx_> ok I can confirm it's broken, need to verify it was actually fixed17:19:57 <_glx_> well seems the fix isn't working17:23:37 <_glx_> hmm garbage from other scripts detection is not helping17:26:36 <_glx_> ok without the garbage it's easy to see it doesn't work at all17:33:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13993: Add: Player configuration of badge display. https://github.com/OpenTTD/OpenTTD/pull/13993#pullrequestreview-280778089817:34:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/1417117:51:02 *** peter1138[d] has joined #openttd17:51:02 <peter1138[d]> Hmm.17:51:35 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367196704025219102/image.png?ex=6813b4a7&is=68126327&hm=b235b9d267ad2ca32979cb74896e3a35156877b8da0ddaf1ea0be215230a91b9&17:51:35 <peter1138[d]> Lots of slight string variations.17:52:03 <peter1138[d]> And it's missing the error bit.17:53:00 <peter1138[d]> And it assumes anything in brackets is still fine to be concatenated.17:53:25 <peter1138[d]> (There's a lot of combinations)17:54:24 <talltyler> Yeah, it’s a mess 🙂17:54:41 <peter1138[d]> These strings are new...17:55:34 <peter1138[d]> When I split it out, most of the concatenation is strings in brackets, so it's "okay".17:55:42 <talltyler> The need to display all these things is a mess, I guess is what I mean 🙂17:56:58 <frosch123> I have plans to redesign the order list, but after release17:57:08 <peter1138[d]> Hmm.17:57:19 <peter1138[d]> Should I just add two missing {SPACE} for now then?17:57:26 <peter1138[d]> Leave it concatenated, as it's always been.17:57:44 <frosch123> Should be fine for 1518:01:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/1417218:03:08 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/14172#pullrequestreview-280799376618:10:01 <peter1138[d]> Hmm, is there space for a small left arrow.18:10:37 <Heiki> →18:10:38 <Heiki> oops, ←18:15:13 <peter1138> Sprite 145/269 (and 593 but that isn't defined)18:15:19 <peter1138> That's the small right arrow.18:15:36 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/14172#pullrequestreview-280802095318:16:10 <peter1138> Er, 369, not 269.18:17:23 <peter1138> 128/352 seems to be free.18:17:53 <peter1138> Hmm, don't we add some symbols...18:19:19 <peter1138> Might just be the monospace font.18:26:28 <peter1138> Oh, chars. D'oh.18:31:48 *** gelignite has joined #openttd18:35:07 <peter1138> Hmm, why can I no longer type into an editbox.18:35:44 <frosch123> Many glyphs are added as font chars only, via action12.18:35:44 <frosch123> Do you need a spriteid? Or is a string code enough?18:36:22 <frosch123> Did I break edit boxes?18:37:10 <peter1138> Yeah, I'm adding to chars.png and using Action 12. I just wanted to find a control code that is not otherwise occupied :)18:43:57 <frosch123> Ah, it picked the wrong insert overload18:48:50 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14173: Fix 66733e2a: Textbuf was broken, and did not insert any input. https://github.com/OpenTTD/OpenTTD/pull/1417318:49:27 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367211265558184008/image.png?ex=6813c237&is=681270b7&hm=26169247563195719ca66a3f6326dd8f77548f4472107b9ba6698783749fdfc9&18:49:34 <peter1138[d]> Such a small detail.18:50:06 <frosch123> I thought I can't test ime stuff, and did not expect to break the boring case18:50:19 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14173: Fix 66733e2a: Textbuf was broken, and did not insert any input. https://github.com/OpenTTD/OpenTTD/pull/14173#pullrequestreview-280809765118:51:25 *** alpapilus has joined #openttd18:51:25 <alpapilus> peter1138[d]: Outdated Concorde.18:52:05 <alpapilus> peter1138[d]: Oh yeah will that be for trains and cars?18:52:07 <alpapilus> Well18:52:13 <alpapilus> Idk will that work.18:52:38 <peter1138[d]> I'm not adding anything new except making the tiny arrow direction point the correct way in RTL languages.18:54:11 <frosch123> Looks good18:54:58 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/1417218:55:01 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #14170: [Bug]: Missing space in timetable window https://github.com/OpenTTD/OpenTTD/issues/1417018:56:14 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367212972832981072/image.png?ex=6813c3ce&is=6812724e&hm=fd837f00e177fa1b99ab8e77f84e1c128f20ab4301d69504f3c799720b10e2f1&18:56:14 <xarick> strange18:57:33 <xarick> probably makes sense somewhere19:00:01 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14174: Fix: Small order list arrow was wrong direction for RTL text. https://github.com/OpenTTD/OpenTTD/pull/1417419:05:38 <xarick> can we create a 3rd class of aircraft?19:06:02 <xarick> erm 4th19:06:39 <xarick> small, large and intercontinental or another apropriate name19:07:30 <xarick> requires the "hub" class airports to land properly19:10:03 *** andythenorth has joined #openttd19:10:03 <andythenorth> probably not19:10:26 <andythenorth> I have seen 747s land on quite short, ill equipped runways19:10:48 <xarick> oh, real life...19:11:28 <xarick> these airports would have an advantage19:11:38 <xarick> they would unload and load fast19:11:49 <alpapilus> andythenorth: Kaitak.19:12:51 *** _zephyris has joined #openttd19:12:51 <_zephyris> Why? Would it be fun?19:13:04 <_zephyris> People already complain that planes are broken and crashes are annoying.19:13:21 <alpapilus> Just don’t play planes.19:13:30 <xarick> oh that's due to a feature killed by... you know it19:13:45 <xarick> no plane crashes was a setting before19:15:27 <xarick> i mean... these hub like airports should have a more prominent purpose19:18:05 *** D-HUND is now known as debdog19:18:44 <xarick> no newgrfs , i rather have it in the core game19:18:54 <xarick> newgrfs just ruin the rules too much19:30:58 *** nielsm has quit IRC (Ping timeout: 480 seconds)19:31:57 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14173: Fix 66733e2a: Textbuf was broken, and did not insert any input. https://github.com/OpenTTD/OpenTTD/pull/1417319:33:30 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/1417519:35:23 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13665: Codechange: Code style changes to AI/GS GUI files https://github.com/OpenTTD/OpenTTD/pull/1366519:44:52 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13565: Add #13519, e4c511d: [Script] Saveload and config file support for handpicked configs https://github.com/OpenTTD/OpenTTD/pull/1356519:48:37 <_glx_> so it really looks like I assumed things without actually testing in #1057319:54:35 *** kuka_lie has joined #openttd19:56:13 <xarick> displaying general company information taxes a bit in graphics rendering19:59:24 <xarick> when my ai is building rail pieces, the thing goes to 10 ms graphics rendering time20:02:05 <xarick> hmm20:10:53 <xarick> is the vehicle tile hash faster now?20:17:23 <peter1138[d]> xarick: Because it refreshes the company value now.20:20:44 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/1417520:23:33 <xarick> oh20:23:54 <xarick> calculating company value is expensive20:26:24 <peter1138> No, but drawing windows can be.20:30:30 <_zephyris> Maybe I should charge for drawing windows20:35:26 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14174: Fix: Small order list arrow was wrong direction for RTL text. https://github.com/OpenTTD/OpenTTD/pull/14174#issuecomment-284321866620:46:56 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/1417520:48:12 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-280838520920:51:05 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-280840616320:58:07 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/1417521:03:19 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-280844042821:06:06 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/1417521:11:37 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367247041885114489/image.png?ex=6813e389&is=68129209&hm=2aa5eba4579421ab7f90ec09b2404f4d28c5702d9967b22719f438993dc4e050&21:11:37 <peter1138[d]> Hmm21:11:55 <andythenorth> when are you level 41+?21:13:13 <xarick> what is SA_FORCE?21:14:04 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #14176: Fix: [Script] Access to enum/consts defined outside of main.nut https://github.com/OpenTTD/OpenTTD/pull/1417621:14:40 <_glx_> so #10573 actually partially worked21:16:38 <xarick> so many changes in a short amount of time21:16:50 <xarick> rebasing is complicated21:17:42 <_glx_> helps to understand the changes 🙂21:19:56 *** Wormnest has quit IRC (Ping timeout: 480 seconds)21:21:29 <peter1138[d]> Well.21:21:34 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367249545196474499/image.png?ex=6813e5dd&is=6812945d&hm=dbfca90e191c534255f7c17bf6b6d2ed7b73a2d0b21381ef07463f25c5bc76f6&21:21:34 <peter1138[d]> With the wrong font...21:22:35 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367249801711849613/image.png?ex=6813e61b&is=6812949b&hm=5f0771a9bf33d23e13c30abce1dc964746b88cf4b5a87c1eb91ffdd67afbe9d8&21:22:35 <peter1138[d]> Huh, worse even.21:23:26 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-280848463321:23:27 <xarick> tile hash changes are massive...21:23:34 <xarick> i can never rebase that21:25:54 <peter1138[d]> Hmm, wonder how much more memory this uses.21:26:04 <peter1138[d]> Probably "nothing" compared to the rest of the game state.21:26:59 <andythenorth> does it make Horse compile slower or faster? 😛21:27:13 <andythenorth> hmm I may have skipped lunch21:27:42 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367251091091230750/image.png?ex=6813e74e&is=681295ce&hm=9490868f0096c6679646abf1e4d6c1afa064b81709941b6fdf221f70f3816a5e&21:27:44 <peter1138[d]> Oh21:29:14 *** tokai has joined #openttd21:29:14 *** ChanServ sets mode: +v tokai21:29:41 <xarick> openttd is multithreaded21:29:44 <xarick> 100%21:29:58 <_glx_> it is, but only outside gameloop21:34:04 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)21:34:35 <xarick> i screwed up... what is that frosch commit doing there? <https://github.com/SamuXarick/OpenTTD/pull/19/commits>21:34:52 *** WormnestAndroid has joined #openttd21:35:14 <xarick> how do i fix this21:35:54 <LordAro> updating your local master, probably21:36:09 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)21:36:11 <LordAro> the master on your fork, specifically21:37:47 <xarick> i did just that... must I wait?21:38:19 <LordAro> git repos don't generally have things magically happen to them, no21:38:45 <LordAro> although you're not wrong actually21:38:46 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)21:38:52 <LordAro> just a github display issue21:39:24 *** WormnestAndroid has joined #openttd21:40:13 <xarick> i have a bad feeling about this21:45:19 *** keikoz has quit IRC (Ping timeout: 480 seconds)21:47:15 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14177: Codechange: Cache layouted text for the last used width. https://github.com/OpenTTD/OpenTTD/pull/1417721:49:10 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/1417521:51:28 <peter1138[d]> Well.21:52:00 <peter1138[d]> So this was the other result of trying to improve performance of some text rendering.21:54:01 <peter1138[d]> Hmm, that may have been older than I remember.21:55:14 <peter1138[d]> Right, what was andy's ridiculous GS doing?21:57:47 <peter1138[d]> Thought I had a savegame for it. Hmm.22:16:23 <peter1138[d]> Ah, that was it, #11226.22:18:23 *** kuka_lie has quit IRC (Remote host closed the connection)22:25:30 <_glx_> debugger in story page ?22:26:29 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367265881574604870/image.png?ex=6813f514&is=6812a394&hm=6d828a04d90a5490df58df8006bac2e773f14cfc47bd48648cbedd6bb46037b9&22:26:29 <peter1138[d]> Uh...22:26:30 <peter1138[d]> Yeah, that one.22:26:50 <peter1138[d]> Oh22:27:09 <xarick> dear _glx_ <https://github.com/SamuXarick/OpenTTD/pull/23/commits/045d18bb209ec26ccf25a11817440e5c564d13c7> I really need this22:27:41 <xarick> it's painful to have to exit to main menu, to change a setting and then start the game again22:28:25 <xarick> can you do something about it, in your style22:29:25 <_glx_> very specific use case22:30:31 <peter1138[d]> Hmm, well, I'm sure it used to work, but I can't get this test save/GS going now.22:37:22 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367268620375429231/image.png?ex=6813f7a1&is=6812a621&hm=1855529ae58a390925cbe79122ccfab151f4835fb6e5164c6b1ca9592786ed8f&22:37:22 <_glx_> not better here22:38:44 <_glx_> and `(undefined string)` everywhere in story book22:42:52 <_glx_> and in 14.0 I get a crash when opening story book22:44:20 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)22:44:40 <peter1138[d]> Heh22:44:55 <peter1138[d]> Is this an accidental regression discovery?22:45:24 <peter1138[d]> Partly the issue is that zip file isn't laid out correctly.22:47:07 <_glx_> dunno22:47:22 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367271140476719134/image.png?ex=6813f9fa&is=6812a87a&hm=5bfeb9b822d7c63d67cf2584953d897ebb50d73f3f024bf77db38306388752a4&22:47:22 <_glx_> in crash dump I can see22:47:52 <peter1138[d]> Not too worried about crashes in 14.0, given all the string changes since then.22:48:19 <_glx_> in master it was fine, just showing (undefined string)22:51:19 <_glx_> well 14.1 crashes too with the same savegame22:52:04 <_glx_> beta shows (undefined string)22:52:31 <_glx_> but yeag the zip is a pain 🙂22:53:04 *** D-HUND has joined #openttd22:55:44 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367273242368151646/image.png?ex=6813fbef&is=6812aa6f&hm=999e111650e69b72a5091d4e1167a38e5ae8c8cd97a995ef3b5fe2eb2f098178&22:55:44 <_glx_> tried new game with the script and firs from zip22:56:09 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367273350472269824/image.png?ex=6813fc09&is=6812aa89&hm=71bf712a4c1bff7dc7083afaaa3e51ea15c7a147601fa688d5dad194fcb5ff0c&22:56:15 <_glx_> that could be an issue22:56:29 *** debdog has quit IRC (Ping timeout: 480 seconds)22:58:47 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367274012480110622/image.png?ex=6813fca7&is=6812ab27&hm=d2916bb7c7c765002ccd678bb22a0569c0040b52f2663b98a2b0a71d43b4aac0&22:58:47 <_glx_> little better without this file22:59:56 <_glx_> indeed the lang file doesn't have this string23:03:21 <_glx_> not sure it's using the correct script as I don't see any settings23:06:33 *** gelignite has quit IRC (Quit: Stay safe!)23:09:34 <_glx_> ok it's loading the script I expected, but it's supposed to show settings and doesn't23:22:05 <_glx_> ok it was a hidden setting I always forgot (ai_developer_tools)23:36:19 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367283455632605315/image.png?ex=68140572&is=6812b3f2&hm=08a0d39deb1355cdc4bdf46bc7ab03ab6133dc92e7e41723537a4f57b16a3f71&23:36:19 <_glx_> now I have a version with the string defined23:36:42 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367283552219168788/image.png?ex=68140589&is=6812b409&hm=bd85814a783f0fc7ae6c99823dcf25b1b685e5267f73d2197efe4e4ffa4461a9&23:36:42 <_glx_> but it still fails with23:42:03 <_glx_> ok now I can load the save in beta and everything works fine23:42:21 <_glx_> so we have a regression in game string stuff23:54:47 *** Flygon has joined #openttdcontinue to next day ⏵