Back to Home / #openttd / 2019 / 04 / Prev Day | Next Day
#openttd IRC Logs for 2019-04-10

---Logopened Wed Apr 10 00:00:50 2019
00:32-!-tycoondemon [~ashnohoe@82-197-216-129.dsl.cambrium.nl] has quit [Ping timeout: 480 seconds]
02:11-!-tycoondemon [~ashnohoe@82-197-216-129.dsl.cambrium.nl] has joined #openttd
02:11-!-tycoondemon is "..." on #openttd.dev #openttd
02:11-!-chomwitt is "chomwitt" on #debian #debian-games
02:11-!-chomwitt [~chomwitt@ppp-94-69-11-2.home.otenet.gr] has joined #openttd
02:16-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has joined #openttd
02:16-!-andythenorth is "andythenorth" on #openttd
02:36<andythenorth>hi
02:37<DorpsGek_II>[OpenTTD/OpenTTD] GabdaZM closed pull request #7369: Add: k-d tree element checker function https://git.io/fhjZB
02:37<DorpsGek_II>[OpenTTD/OpenTTD] GabdaZM commented on pull request #7369: Add: k-d tree element checker function https://git.io/fjqJ8
03:12<andythenorth>hmm
03:13<andythenorth>canned fish? o_O
03:53-!-Laedek [~quassel@172.92.127.86] has quit [Quit: Laedek]
04:24-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has quit [Ping timeout: 480 seconds]
04:27-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has joined #openttd
04:27-!-andythenorth is "andythenorth" on #openttd
04:31-!-Laedek [~quassel@172.92.127.86] has joined #openttd
04:31-!-Laedek is "Laedek" on #openttd
04:43-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has quit [Quit: andythenorth]
06:53<@peter1138>Hmm
07:36<DorpsGek_II>[OpenTTD/OpenTTD] stale[bot] commented on pull request #7362: Feature: Reverse train at waypoint orders (jgr) https://git.io/fjqtV
07:36<DorpsGek_II>[OpenTTD/OpenTTD] stale[bot] commented on pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fjqtw
07:39<DorpsGek_II>[OpenTTD/OpenTTD] planetmaker commented on pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fjqto
07:41<Heiki>https://tracker.debian.org/news/1037780/accepted-openttd-191-1-source-into-unstable/ oh, that was surprisingly quick
07:41<DorpsGek_II>[OpenTTD/OpenTTD] PeterN dismissed a review for pull request #7485: Add: Dropdown build menus from toolbar have icons https://git.io/fjtIz
07:44<Heiki>but much too late for Debian 10, I suppose
07:44<@planetmaker>hm?
07:44<@planetmaker>already too late? That'll be sad
07:45<Heiki>https://release.debian.org/buster/freeze_policy.html at least according to that timeline
07:45<@planetmaker>I sometimes wonder... I really love debian for its stance on stability over 'latest and greatest'. Yet I'm not convinced that it is a stance which works for *every* software package on debian equally well
07:46<@planetmaker>1.9.1 actually fixes at least one quite critical bug over 1.9.0
07:46-!-WWacko1976-work [~IceChat9@ip-80-113-66-42.ip.prioritytelecom.net] has joined #openttd
07:46-!-WWacko1976-work is "YO!" on #openttd #/r/openttd
07:46<@planetmaker>well...critical maybe not. But at least severe as in reducing user experience with a lot of newgrfs
07:46<milek7>but 1.9.0 won't be in buster either?
07:47<Heiki>no, 1.8.0 it is
07:47<@planetmaker>I guess, yes. Oh well
07:51<@peter1138>Well, Matthijs Kooijman is an OpenTTD dev, so it's not all that surprising ;)
07:56<Heiki>stretch (the current stable Debian) has 1.6.1 (:
07:57<@planetmaker>OpenTTD is in quite a good position to be reasonably well updated in many distros... IMHO
08:02<DorpsGek_II>[OpenTTD/OpenTTD] PeterN commented on pull request #7493: Fix: Creating a cargo subsidy with a town as source was not considering min population. https://git.io/fjqqv
08:06<DorpsGek_II>[OpenTTD/OpenTTD] PeterN requested changes for pull request #7493: Fix: Creating a cargo subsidy with a town as source was not considering min population. https://git.io/fjqqI
08:07-!-tokai|noir [~tokai@00012860.user.oftc.net] has joined #openttd
08:07-!-tokai|noir is "Christian Rosentreter" on #openttd
08:07-!-mode/#openttd [+v tokai|noir] by ChanServ
08:07<@peter1138>Well there's the ICU thing :)
08:14-!-tokai [~tokai@00012860.user.oftc.net] has quit [Ping timeout: 480 seconds]
08:16<@LordAro>debian's package freezing works well in some cases, not in others. that's to be expected
08:16<@LordAro>as a server, it's great
08:16<@LordAro>as a desktop, i'd never recommend using debian stable, testing at least, maybe even sid
08:21<@planetmaker>well. Actually I do recommend to use debian stable. For some desktop usages. Like mine at work here
08:21<@planetmaker>I have to rely on a stable system where I can just do my work. I don't need the latest and greatest. But it "just needs to work"
08:22<@planetmaker>at home... a different story
08:22<milek7>..until it turns out some driver is broken because it is too old ;P
08:22<@planetmaker>nah... not on this old hardware :P
08:23<@planetmaker>but driver issues were the reason I installed fedora on my laptop. Works very well
08:26<@peter1138>Yup.
08:27<@peter1138>Also using the same version as servers means my build environment isn't going to be targetting too-new stuff.
08:27-!-Samu [~Ricardo@pa4-84-91-142-34.netvisao.pt] has joined #openttd
08:27-!-Samu is "realname" on #openttd
08:28<DorpsGek_II>[OpenTTD/OpenTTD] PeterN commented on pull request #7490: Add: Press ctrl to build diagonal rivers in scenario editor. https://git.io/fjqqp
08:37<Samu>hi
08:39-!-pnda [~oftc-webi@x4db4a15a.dyn.telefonica.de] has joined #openttd
08:39-!-pnda is "OFTC WebIRC Client" on #openttd
08:47<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7493: Fix: Creating a cargo subsidy with a town as source was not considering min population. https://git.io/fjqeT
08:51<Xaroth>@seen Zuu
08:51<@DorpsGek>Xaroth: Zuu was last seen in #openttd 3 weeks, 4 days, 12 hours, 50 minutes, and 54 seconds ago: <Zuu> I don't know if that one that charges does it all by the book but basically the OpenTTD license doesn't forbid anyone to fork OpenTTD and sell it but you need to provide the source code when you sell it.
09:13<DorpsGek_II>[OpenTTD/OpenTTD] nikolas updated pull request #7086: Change #6173: Update SDL driver to use SDL 2.0 https://git.io/fhamZ
09:30<DorpsGek_II>[OpenTTD/OpenTTD] nikolas opened issue #7494: Program crashes when opening Game Options on title screen https://git.io/fjqY4
09:35<@LordAro>oops.
09:43-!-nielsm [~nielsm@176-23-103-56-cable.dk.customer.tdc.net] has joined #openttd
09:43-!-nielsm is "Niels Martin Hansen" on #openttd
09:46-!-Wormnest [~Wormnest@35.136.176.177] has joined #openttd
09:46-!-Wormnest is "Wormnest" on #openttd
09:51-!-Alberth [~alberth@00015f9e.user.oftc.net] has joined #openttd
09:51-!-mode/#openttd [+o Alberth] by ChanServ
09:51-!-Alberth is "purple" on @#openttd
09:51<@Alberth>o/
10:00-!-Wormnest [~Wormnest@35.136.176.177] has quit [Quit: Leaving]
10:04<Xaroth>o7
10:14<Samu>milek7,
10:14<Samu>can you create a branch with 240 AI in it?
10:15<Samu>or was it 240 companies
10:15<milek7>patch is here, but it likely bitrotten since 1.6.1
10:15<milek7>https://www.tt-forums.net/viewtopic.php?t=75060
10:17<Samu>:(
10:21<Samu>eww so many failed hunks
10:39<@peter1138>"there is only 16 colors" I have a patch for that ;)
10:46-!-synchris [~synchris@139.138.202.72] has joined #openttd
10:46-!-synchris is "Synesios Christou" on #openttd
10:49<Samu>plz fix it, I feel like starting 1 AI of a kind, there's 50 AIs i think
10:49<Samu>all in one map for fun
10:49<Samu>or i can try 1.6.1
10:49<Samu>but civilai is already using 1.9 features :(
10:54<Samu>http://uk.binaries.openttd.org/index.html it's empty :8
10:56<Samu>https://finger.openttd.org/ stuff about subversion there
11:00<@peter1138>Samu, apply it to 1.6.1 then rebase :-)
11:02-!-Wormnest [~Wormnest@35.136.176.177] has joined #openttd
11:02-!-Wormnest is "Wormnest" on #openttd
11:06<@peter1138>Are there any NewGRFs with passenger-like cargos?
11:06<@peter1138>IIRC something about tourists...
11:08<@planetmaker>ECS?
11:09<Samu>git hub svn-trunk revision number doesn't match the old svn-trunk revision number :(
11:09<@planetmaker>what?
11:12-!-Flygon [~Flygon@58-6-72-150.dyn.iinet.net.au] has quit [Quit: A toaster's basically a soldering iron designed to toast bread]
11:17<Samu>22403 is the old svn r27606
11:17<@planetmaker>github... is a git repo... not svn. So yes... if you access it via svn, the revisions of course don't match as the original svn repo contained more than just OpenTTD trunk + release branches
11:20<@peter1138>Use git log to find the svn revision.
11:21<@peter1138>But
11:21<@peter1138>It's against 1.6.1
11:21<@peter1138>So you can just check out that tag.
11:24-!-sla_ro|master [slamaster@84.117.88.126] has joined #openttd
11:24-!-sla_ro|master is "slamaster" on #sla #openttd
11:34<V453000>wow I actually made NUTS 0.8.0 but I didn't release it :D now to wonder what is missing from what I wanted to do
11:35<@planetmaker>publish it and wait for people to complain :P
11:35<@planetmaker>they will tell you, no doubt ;)
11:36<V453000>yeah I will, but yeah they won't :P
11:36<@planetmaker>:D
11:37<V453000>I'll just add some more things to make sure something gets fucky
11:37<@planetmaker>hehe
11:39<V453000>ah I see how naive I was now
11:39<V453000>wasn't even commited :)
11:41<@peter1138>Heh
11:41<@peter1138>Those are the best changes. I have loads.
11:41-!-synchris [~synchris@139.138.202.72] has quit [Ping timeout: 480 seconds]
11:43<Samu>conflicts :(
11:43<Samu>TortoiseMerge editor seems similar to that of visual studio
11:47<Samu>i have no idea what im merging
11:53<Samu>can't build, no available language packs, can't merge
11:53<Samu>i give up
11:53-!-synchris [~synchris@139.138.202.72] has joined #openttd
11:53-!-synchris is "Synesios Christou" on #openttd
11:59<Samu>was FastPTPAI updated recently? I remember it crashing early
12:00<Samu>it has 218 trains right now without crashing yet
12:04<DorpsGek_II>[OpenTTD/OpenTTD] JGRennison updated pull request #7366: Add: List recently executed commands in crashlog output. https://git.io/fhjJc
12:05<Samu>nop
12:05<Samu>mere luck
12:05<Samu>or maybe desert tileset is more favourable
12:06<milek7>it should build on 1.6.1, it doesn't?
12:06<Samu>it's even beating trAIns, the king of trains
12:06<DorpsGek_II>[OpenTTD/OpenTTD] JGRennison commented on pull request #7366: Add: List recently executed commands in crashlog output. https://git.io/fjqsi
12:06<Samu>fails to build, complains about include files, png and such
12:07<Samu>and your exe on the forum has no available language packs
12:07<Samu>and i can't rebase to master, too many conflicts
12:16<@LordAro>peter1138: midweek cycle time!
12:16<@LordAro>because it's summer now
12:17<Samu>I wasn't expecting WormAI to stall, but there it is
12:17<Samu>it's one of the AIs that occasionally are adjusted all the way down to 500 ops
12:18<milek7>that's weird, saveload code is broken
12:19<milek7>but it definitely worked before
12:23<Samu>Jazz Jukebox
12:23<Samu>is Jazz really a must?
12:23<Samu>when submitting music
12:27<nielsm>it's what john broomhall made for chris sawyer back when
12:43-!-Progman [~progman@p4FD66A3A.dip0.t-ipconnect.de] has joined #openttd
12:43-!-Progman is "Peter Henschel" on #openttdcoop.dev #openttd
12:43<Samu>hmm there should be something done to dead AIs...
12:43-!-andythenorth [~andytheno@81.171.232.172] has joined #openttd
12:43-!-andythenorth is "andythenorth" on #openttd
12:43<andythenorth>peter1138: small lol....that eBay seller is located roughly where Microprose office used to be
12:43<Samu>company autoclean
12:44<Samu>but i know these are network related
12:44<Samu>doesn't fit for single player
12:50<Eddi|zuHause>?
12:54<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh updated pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fhho4
12:55<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh commented on pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fjqGR
12:56-!-HerzogDeXtEr [~farci@ip92340ef6.dynamic.kabel-deutschland.de] has joined #openttd
12:56-!-HerzogDeXtEr is "purple" on #openttd
12:58<DorpsGek_II>[OpenTTD/OpenTTD] planetmaker commented on pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fjqGz
13:01<Samu>getseed command could copy the value into clipboard
13:01<Samu>then i'd just need to do ctrl-v
13:03-!-gareppa [~gareppa@151.27.214.162] has joined #openttd
13:03-!-gareppa is "gareppa" on #llvmlinux #ext4 #mm #moocows #oftc #openttd #scilab #C
13:04-!-andythenorth [~andytheno@81.171.232.172] has quit [Quit: andythenorth]
13:04-!-andythenorth [~andytheno@81.171.232.172] has joined #openttd
13:04-!-andythenorth is "andythenorth" on #openttd
13:04-!-andythenorth [~andytheno@81.171.232.172] has left #openttd []
13:06<Samu>i'd like a performance rating to be scaled by map size
13:06<Samu>can you do that nielsm
13:07<nielsm>with GS-controlled performance ratings anything would be possible
13:07<Samu>oh, that would be interesting
13:07<Samu>but it's another GS :(
13:07<Samu>not into thereal game
13:08<nielsm>but as I wrote in that comment: changing the performance rating calculation is getting removed from that PR
13:18<Samu>https://imgur.com/Vbpi2JL darn, the 250,000 ops is not besides the GS
13:18<Samu>i failed
13:18-!-Wormnest [~Wormnest@35.136.176.177] has quit [Ping timeout: 480 seconds]
13:19-!-Wormnest [~Wormnest@35.136.176.177] has joined #openttd
13:19-!-Wormnest is "Wormnest" on #openttd
13:21-!-Wolf01 [~wolf01@0001288e.user.oftc.net] has joined #openttd
13:21-!-Wolf01 is "Wolf01" on #openttd
13:22<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh updated pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fhho4
13:24<DorpsGek_II>[OpenTTD/OpenTTD] glx22 commented on issue #7494: Program crashes when opening Game Options https://git.io/fjqZI
13:27-!-glx [kvirc@000128ec.user.oftc.net] has joined #openttd
13:27-!-mode/#openttd [+v glx] by ChanServ
13:27-!-glx is "Loïc GUILLOUX" on +#openttd
13:27<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh commented on pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fjqZO
13:29<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh updated pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fhho4
13:30-!-gareppa [~gareppa@151.27.214.162] has quit []
13:36<@peter1138>Hmm, I wonder which ECS file contains tourists :p
13:38-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has joined #openttd
13:38-!-andythenorth is "andythenorth" on #openttd
13:38<@peter1138>Ah, towns
13:38<DorpsGek_II>[OpenTTD/OpenTTD] glx22 commented on issue #7494: Program crashes when opening Game Options https://git.io/fjqZo
13:40<nielsm>https://0x0.st/zZdP.jpg those taller top info boxes look strange when you're used to the classic four lines
13:42<@peter1138>Would it fit next to profit?
13:42<nielsm>that's what I did originally
13:42<nielsm>https://user-images.githubusercontent.com/1062071/54077397-d9adb880-42b7-11e9-8606-d5a391d84bbf.png
13:42<nielsm>wanted to try it on more lines instead
13:44<+glx>oh of course QSort and std::sort need different sort function
13:49-!-frosch123 [~frosch@00013ce7.user.oftc.net] has joined #openttd
13:49-!-frosch123 is "frosch" on +#openttd.dev #openttd
13:50<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh commented on pull request #7353: Feature: Measure vehicle capacity utilisation efficiency https://git.io/fjqnf
13:53<Samu>fixed!
13:53<DorpsGek_II>[OpenTTD/OpenTTD] glx22 opened pull request #7495: Fix: std::sort() and qsort() use different comparators https://git.io/fjqnl
13:54<+glx>will check other std:sort calls, just in case
13:55<+glx>seems to be the only one
13:56<DorpsGek_II>[OpenTTD/OpenTTD] glx22 updated pull request #7495: Fix: std::sort() and qsort() use different comparators https://git.io/fjqnl
13:58<Samu>Your script made an error: the index 'RONDJE_ALERT' does not exist? what the heck is this?
13:58-!-gelignite [~gelignite@55d4cd80.access.ecotel.net] has joined #openttd
13:58-!-gelignite is "gelignite" on #openttd
13:59<nielsm>a bug?
13:59<nielsm>in the script?
13:59<Samu>Otvi crash
14:00<Samu>i'm used to seeing many weird crashes, but this one is special
14:10<milek7>I compiled that patch without asserts back then, turns out one fails too eagerly
14:11<milek7>maybe it is better to stop fit into old loading code and define new vartype, for saveloading std::bitset
14:11<milek7>then my custom bitset class could go out
14:12<milek7>s/stop fit/stop trying to fit/
14:18-!-Wormnest [~Wormnest@35.136.176.177] has quit [Quit: Leaving]
14:24<DorpsGek_II>[OpenTTD/OpenTTD] michicc approved pull request #7495: Fix: std::sort() and qsort() use different comparators https://git.io/fjqcv
14:27<Samu>ahm... the forum doesn't let me upload files bigger than 25 MB?
14:27<Samu>i got a 37 mb savegaem
14:28<Samu>guess i should stop using lzo
14:29<milek7>huh, there is m8 now, lots of free space
14:29<milek7>maybe i could make PR from that patch
14:33-!-HerzogDeXtEr1 [~farci@ip92340ef6.dynamic.kabel-deutschland.de] has joined #openttd
14:33-!-HerzogDeXtEr1 is "purple" on #openttd
14:39<Samu>cool
14:39-!-HerzogDeXtEr [~farci@ip92340ef6.dynamic.kabel-deutschland.de] has quit [Ping timeout: 480 seconds]
14:55<DorpsGek_II>[OpenTTD/OpenTTD] glx22 merged pull request #7495: Fix: std::sort() and qsort() use different comparators https://git.io/fjqnl
14:56<DorpsGek_II>[OpenTTD/OpenTTD] glx22 closed issue #7494: Program crashes when opening Game Options https://git.io/fjqY4
14:56<pnda>I really don't know what to do now. Don't think anything else will work other than {LRE}.... https://github.com/OpenTTD/OpenTTD/pull/7480
15:11-!-synchris [~synchris@139.138.202.72] has quit [Ping timeout: 480 seconds]
15:11<frosch123>https://blog.jalproductions.co.uk/2013/04/19/how-to-fix-rtl-bracketsparenthesis-problem/ <- according to that you should use {RLM}
15:11<frosch123>which apparently is a zero-width space
15:11<frosch123>it also does not need that PDF pop command
15:16<frosch123>though in #6666 actually the whole line is ltr
15:20<+michi_cc>pnda: Which platform are you running/testing on? As for me for example STR_ROAD_TOOLBAR_TOOLTIP_BUILD_TRAM_VEHICLE_DEPOT renders most likely properly, or at least without visible bracket issues.
15:20<pnda>I am on Windows
15:20<pnda>Everything does work fine, no issues.
15:20<+michi_cc>So am I.
15:21<pnda>Just confused if I have to change something due to your comments
15:21<frosch123>what is actually the expected result? "Eskigac (1770)" or "(1770) Eskigac" ?
15:21<pnda>the second
15:21<pnda>and even with LTR encoding the second one appears
15:21<+michi_cc>The string is simply wrong, it has to be :{ORANGE}{TOWN}{BLACK} ({COMMA})
15:22<+michi_cc>STR_SORT_BY_ENGINE_ID is also just wrong, proper string would be :نوع المحرك (قياسي)
15:22<pnda>No well thats what we had before
15:22<+michi_cc>Assuming your IRC client displays that in any way proberly
15:23<pnda>I am using web irc
15:23<+michi_cc>No, before was :نوع المحرك (قياسي(
15:23<pnda>Well though if we had that bracket at the end of the string, that would be at the end of the string in game aswell
15:23<pnda>because that's then detached from the string, but we want the bracket to appear at for us the beginning of the string
15:27<frosch123>if "(1770) Eskigac" is the desired result, it should be {ORANGE}{TOWN}{BLACK} {RLM}({COMMA})
15:27<pnda>In that case {RLM} is not needed. The string works as expected
15:29<pnda>In a case like :{LRE}(نوع المحرك (قياسي, without LRE, the string would be rendered like this: نوع المحرك (قياسي(, as far as I remember
15:30-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has left #openttd []
15:30<DorpsGek_II>[OpenTTD/OpenTTD] heirecka opened issue #7496: Crashes on start https://git.io/fjqCX
15:30<pnda>Or actually that was rendered wrong by IRC, for me at least.
15:31<+michi_cc>pnda: This diff http://www.icosahedron.de/openttd/patches/fix_arabic_brackets.diff makes all strings from your commit render properly on Windows/Uniscribe (the strings not in it didn't need any changes).
15:33<+michi_cc>Except STR_TOWN_DIRECTORY_CITY as that isn't in the trunk lang file so I forgot it.
15:33<pnda>Wait I am so confused why does that fix it
15:34<pnda>Mostly because VS was just throwing the brackets arround like it just wanted to
15:34<pnda>STR_TOWN_DIRECTORY_TOWN is exactly what I have anyway
15:35<frosch123>michi_cc: that creates "Eskigac (1770)" for me
15:35<+michi_cc>So ICU renders it differently. Joy!
15:35<@LordAro>ha.
15:36-!-HerzogDeXtEr1 [~farci@ip92340ef6.dynamic.kabel-deutschland.de] has quit [Read error: Connection reset by peer]
15:37<+michi_cc>frosch123: Just as an experiment, can you change the UBIDI_DEFAULT_RTL in gfx_layout.cpp line 211 to UBIDI_DEFAULT_LTR?
15:37-!-pnda [~oftc-webi@x4db4a15a.dyn.telefonica.de] has quit [Remote host closed the connection]
15:38<frosch123>hmm, i added a {RLE} code to STR_TOWN_DIRECTORY_TOWN, now i have "SmallTown (100)" and "(1'000) BigTown" :p
15:39<+michi_cc>Or maybe 0 actually. Basically whatever gets ICU to do auto-detecting.
15:39<frosch123>the thousand separator affects it :p
15:40<+michi_cc>Okay, updated the diff to include STR_TOWN_DIRECTORY_CITY.
15:41<+michi_cc>Uniscribe renders both small and big towns like "(123) Town" with it.
15:41<frosch123>michi_cc: what does your language selection box say? "English (UK)" or "(UK) English"?
15:42<+michi_cc>English (UK)
15:42<frosch123>how does it distinguish "English (UK)" from "(1'000) Town"?
15:43<frosch123>michi_cc: changing 211 to LTR or 0 does not seem to affect anything
15:43-!-gelignite [~gelignite@55d4cd80.access.ecotel.net] has quit [Quit: Good fight, good night!]
15:43<+michi_cc>I think the explicit ubidi direction specification is wrong, both Uniscribe and OSX do not set any special direction for RTL languages.
15:45<+michi_cc>frosch123: The German wikipedia has a nice screenshot: https://de.wikipedia.org/wiki/Unicode-Bidi-Algorithmus#/media/File:Screenshot_UBA_Gecko_vs_Blink.png
15:46<+michi_cc>They talk about a wrong old version and a right new version. Given that ICU layout is old, it might simply not implement the current algorithm.
15:47<DorpsGek_II>[OpenTTD/OpenTTD] michicc commented on pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjqCp
15:47<frosch123>well, but the issue with the town directory is, that both the town name and the population are LTR
15:48<frosch123>the examples on wiki and stackex always include arabic characters
15:48<frosch123>(or hebrew)
15:48<+michi_cc>Okay, sorry, I have to correct myself here: state.uBidiLevel = _current_text_dir == TD_RTL ? 1 : 0;
15:48<+glx>I guess #7496 is a local issue
15:49<@LordAro>glx: the malloc error message is a bit suspect
15:49<+michi_cc>I.e. for Uniscribe code does apply a default RTL direction for RTL languages. I remembered wrong, even if I wrote it :)
15:49<DorpsGek_II>[OpenTTD/OpenTTD] nielsmh commented on issue #7496: Crashes on start https://git.io/fjqWJ
15:49<+michi_cc>If I go with that, line 211 should be 1 as well, unless ICU uses some special internal coding there.
15:51<DorpsGek_II>[OpenTTD/OpenTTD] LordAro commented on issue #7496: Crashes on start https://git.io/fjqWU
15:52-!-Alberth [~alberth@00015f9e.user.oftc.net] has left #openttd []
15:52<nielsm>could we get some std::string into the file routines? :)
15:53<@LordAro>nielsm: std::filesystem ;)
15:57<Xaroth>peter1138: to respond to your message from.... hours ago. No, I don't have a C/C++ dev env on my main machine
15:59<Xaroth>The adminport script I used to test is part of the example scripts in my rework branch ( https://github.com/Xaroth/libottdadmin2/tree/rework-wip )
15:59<frosch123>michi_cc: with "? 1: 0" it is better, i can save some control codes
15:59<frosch123>i still need some {RLM} or it behaves different for "(100)" and "(1'000)"
16:05<frosch123>https://paste.openttdcoop.org/po4oxpeda <- with those RLM and the "? 1: 0" it works fine for me
16:06<frosch123>nielsm: i think alberth had some patches with that
16:08<frosch123>nielsm: http://devs.openttd.org/~alberth/diffs/fios_rewrite2/ maybe you find some ideas
16:09<frosch123>btw. i hate that std::filesystem uses operator/
16:10<@LordAro>python's pathlib does the same
16:10<@LordAro>i kinda like it
16:10<DorpsGek_II>[OpenTTD/OpenTTD] heirecka commented on issue #7496: Crashes on start - malloc(): invalid next size (unsorted) https://git.io/fjqW8
16:10<@LordAro>in an iostreams sort of way
16:10<frosch123>someone once suggested to use operator% to mirror roadbits in ottd :p
16:11<milek7>std::filesystem? some new standard copied from boost?
16:11<@LordAro>pretty much, yes
16:14<frosch123>it's c++17, so not as new as coroutines
16:14<frosch123>or flat_map
16:14<frosch123>(if you want to talk about boost c&p)
16:15<frosch123>actually, i think std::flat_map is significantly different from boost::flat_map
16:15<frosch123>std::flat_map is pair<vector<Key>,vector<Value>, while boost::flat_map is vector<pair<Key,Value>>
16:16<frosch123>(iirc)
16:17<nielsm>pair of vectors seems strange, but it might have some advantage in locality when searching for keys I guess?
16:18-!-nielsm [~nielsm@176-23-103-56-cable.dk.customer.tdc.net] has quit [Quit: wroom]
16:35-!-sla_ro|master [slamaster@84.117.88.126] has quit []
16:39<DorpsGek_II>[OpenTTD/OpenTTD] LordAro dismissed a review for pull request #7493: Fix: Creating a cargo subsidy with a town as source was not considering min population. https://git.io/fjqqI
17:07<DorpsGek_II>[OpenTTD/OpenTTD] M3Henry updated pull request #7407: Codechange: Use null pointer literal instead of the NULL macro https://git.io/fjJDX
17:08<@peter1138>evening
17:11<DorpsGek_II>[OpenTTD/OpenTTD] michicc commented on pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjqlu
17:13<DorpsGek_II>[OpenTTD/OpenTTD] spnda commented on pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjqlg
17:20<DorpsGek_II>[OpenTTD/OpenTTD] michicc commented on pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjqlM
17:21<DorpsGek_II>[OpenTTD/OpenTTD] michicc approved pull request #7407: Codechange: Use null pointer literal instead of the NULL macro https://git.io/fjqlD
17:21<+michi_cc>Time to break every single patch/PR ;p
17:21<@LordAro>:o
17:21<+glx>hehe
17:22-!-Progman [~progman@p4FD66A3A.dip0.t-ipconnect.de] has quit [Remote host closed the connection]
17:22<DorpsGek_II>[OpenTTD/OpenTTD] michicc merged pull request #7407: Codechange: Use null pointer literal instead of the NULL macro https://git.io/fjJDX
17:22-!-m3henry [~m3henry@host-212-139-212-35.static.as9105.net] has joined #openttd
17:22-!-m3henry is "realname" on #openttd
17:25<m3henry>woo
17:25<@LordAro>m3henry: what's next? :p
17:26<m3henry>std::pair perhaps
17:26<frosch123>is std::string already done?
17:27-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has joined #openttd
17:27-!-andythenorth is "andythenorth" on #openttd
17:28<andythenorth>@seen pikka
17:28<@DorpsGek>andythenorth: pikka was last seen in #openttd 1 day, 3 hours, 56 minutes, and 31 seconds ago: <Pikka> well
17:28<andythenorth>exactly
17:28<DorpsGek_II>[OpenTTD/OpenTTD] michicc updated pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjLdR
17:30<DorpsGek_II>[OpenTTD/OpenTTD] michicc commented on pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjq8Y
17:31<+michi_cc>frosch123: No, it's still free, just for you :p
17:33-!-Laedek [~quassel@172.92.127.86] has quit [Quit: Laedek]
17:34<frosch123>:p
17:37<@peter1138>Hmm
17:37<@peter1138>"Subsidies on offer:"
17:37<@peter1138>"Tourists from Tronningley to Defingley"
17:37<@peter1138>So... kinda...
17:44<@peter1138>nullptr... so every PR conflicts now :p
17:48-!-andythenorth [~andytheno@cpc87219-aztw31-2-0-cust178.18-1.cable.virginm.net] has quit [Quit: andythenorth]
17:48<DorpsGek_II>[OpenTTD/OpenTTD] stormcone opened pull request #7497: Feature: Selective demolition tool. https://git.io/fjq82
17:49<DorpsGek_II>[OpenTTD/OpenTTD] frosch123 commented on pull request #7480: Fix #6666: brackets in arabic strings https://git.io/fjq8a
17:51-!-Wolf01 [~wolf01@0001288e.user.oftc.net] has quit [Quit: Once again the world is quick to bury me.]
17:55-!-frosch123 [~frosch@00013ce7.user.oftc.net] has quit [Quit: be yourself, except: if you have the opportunity to be a unicorn, then be a unicorn]
18:18-!-chomwitt [~chomwitt@ppp-94-69-11-2.home.otenet.gr] has quit [Ping timeout: 480 seconds]
18:24<DorpsGek_II>[OpenTTD/OpenTTD] stormcone updated pull request #7497: Feature: Selective demolition tool. https://git.io/fjq82
18:26<DorpsGek_II>[OpenTTD/OpenTTD] PeterN updated pull request #7380: Feature: Multi-tile docks. https://git.io/fjv8j
18:29-!-m3henry [~m3henry@host-212-139-212-35.static.as9105.net] has quit [Quit: Leaving]
18:39<DorpsGek_II>[OpenTTD/OpenTTD] stormcone updated pull request #7497: Feature: Selective demolition tool. https://git.io/fjq82
18:44<DorpsGek_II>[OpenTTD/OpenTTD] PeterN updated pull request #7441: Feature: Per-group wagon removal flag https://git.io/fjkJl
18:54<DorpsGek_II>[OpenTTD/OpenTTD] PeterN updated pull request #7446: Feature: Show coverage area for existing stations and towns https://git.io/fjksj
18:57<Samu>nullptr?
18:57<Samu>what about NULL? is it gone?
18:58-!-drac_boy [~oftc-webi@72.1.195.4] has joined #openttd
18:58-!-drac_boy is "OFTC WebIRC Client" on #openttd
18:58<drac_boy>hi there
18:58<@peter1138>NULL still works but don't use it.
18:58<@peter1138>RIP NULL
18:59<Samu>hmm ok
18:59<@peter1138>I believe it's still there for 3rdparty code.
19:00<Samu>LZMA is 3rd party?
19:00<Samu>still see it with NULL
19:00<Samu>and settings.h
19:01<@peter1138>I guess table/settings.h.preamble was missed
19:01<@peter1138>But yes, LZMA is 3rd party.
19:02<@peter1138>Hmm, well, do we even include lzma? no.
19:03<Samu>finding NULL in entire solution
19:03<@peter1138>It's in the settings files still.
19:03<@peter1138>But no biggie.
19:04<Samu>1461 matching lines
19:05<Samu>sqapi
19:05<milek7>various arrays in saveload code were column aligned with spaces, now they aren't ;P
19:06<Samu>those .obm files
19:06<@peter1138>That was mostly broken when I did the SLV_ stuff.
19:10<Samu>settings.ini has a NULL
19:10<Samu>company_settings.ini too
19:10-!-HerzogDeXtEr [~farci@ip92340ef6.dynamic.kabel-deutschland.de] has joined #openttd
19:10-!-HerzogDeXtEr is "purple" on #openttd
19:10<Samu>many of the ini files
19:11<Samu>too many STR_NULL
19:11<Samu>false matches
19:13<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7486: Fix: AI/GS settings with the flag SCRIPTCONFIG_RANDOM could be altered after loading from a savegame. https://git.io/fjtsp
19:13<@peter1138>Oh, you weren't excluding anything like CONDNULL or STR_NULL? :p
19:22-!-Laedek [~quassel@172.92.127.86] has joined #openttd
19:22-!-Laedek is "Laedek" on #openttd
19:22<Samu>what's the advantage of nullptr? better performance?
19:34-!-glx_ [kvirc@2a01:e35:2f59:c7c0:8418:ec3d:9184:2e22] has joined #openttd
19:34-!-glx_ is "Loïc GUILLOUX" on #openttd.noai #openttd.notice #openttd
19:34-!-glx is now known as Guest5327
19:34-!-glx_ is now known as glx
19:36<drac_boy>samu migth be the wrong type but this helps still? https://embeddedartistry.com/blog/2017/2/28/migrating-from-c-to-c-null-vs-nullptr
19:39-!-Guest5327 [kvirc@000128ec.user.oftc.net] has quit [Ping timeout: 480 seconds]
19:45<Samu>so it's just to avoid errors
19:45<Samu>meh
19:45<Samu>k
19:46<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7424: Add: Kdtree for AirportGetNearestTown https://git.io/fjU1d
19:58<drac_boy>anyway I'm off for tonight so have fun with pointers samu :)
19:59-!-drac_boy [~oftc-webi@72.1.195.4] has left #openttd []
19:59<Samu>tc
20:05<Samu>ugh, the reload ai feature is bugged t.t
20:06<Samu>nice that nullptr forced me to test
20:06<Samu>it would go unnoticed
20:07-!-Netsplit resistance.oftc.net <-> charon.oftc.net quits: yur3shmukcik[m], Yexo, hylshols7qui[m], orudge, tycoondemon, TrueBrain, Mek, KenjiE20, Heiki, HerzogDeXtEr, (+63 more, use /NETSPLIT to show all of them)
20:10-!-Netsplit over, joins: XeryusTC, WWacko1976-work, tycoondemon, gnu_jj, TrueBrain, cawal[m], Lejving, orudge, _dp_, kgz (+21 more)
20:10-!-Netsplit over, joins: Hobbyboy, berndj, @planetmaker, eirc, @peter1138, HerzogDeXtEr, dihedral, seatsea, DorpsGek_II, Westie (+15 more)
20:10-!-Netsplit over, joins: grossing, debdog, Markk, Heiki, glx, Samu, Thedarkb-X40, Xaroth, Eddi|zuHause, Speedy` (+7 more)
20:10-!-Hobbyboy is "Andrew" on #openttd #oftc #moocows #linode #perl @#YourBNC
20:10-!-debdog is "Wowbagger" on #openttd #bitlbee
20:10-!-Heiki is "Heikki Piirainen" on #openttd #debian-next #debian-offtopic #debian
20:10-!-Markk is "Bjum Bjumfors" on #openttd #bitlbee
20:14-!-kgz [~kragniz@nyaa.kragniz.eu] has quit [Ping timeout: 480 seconds]
20:17<Samu>glx, looks like I found the reason why this line is after DC_EXEC https://github.com/OpenTTD/OpenTTD/pull/7376/files#diff-a642ac2d124f2c9d89c466a024a6a47eR887
20:17<Samu>the company is still alive
20:17<Samu>network delays the command
20:18<Samu>the previous command that was told to bankrupt the company has yet to be executed
20:21<Samu>at least it's what I think
20:21<Samu>still investigating
20:22<Samu>yep, this is still in the same tick
20:22<Samu>the bankrupt command is still in the queue
20:22-!-kgz [~kragniz@nyaa.kragniz.eu] has joined #openttd
20:22-!-kgz is "kragniz" on #openttd
20:36-!-HerzogDeXtEr [~farci@ip92340ef6.dynamic.kabel-deutschland.de] has quit [Read error: Connection reset by peer]
20:39<Samu>i edited the comment to indicate this
20:41<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7376: More than max_no_competitors could be created in network games https://git.io/fjvY8
20:56<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7193: Fix #6468: Load correct version of AI as specified during the time of its save. https://git.io/fhHI1
21:00<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7190: Fix #7188: AI instance crash when reloading AI in a server. https://git.io/fh9jW
21:03<Samu>ugh
21:04<Samu>editing on github website broke it
21:06<rubenwardy>you shouldn't edit on the github website, that's not how you git :P
21:08<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7190: Fix #7188: AI instance crash when reloading AI in a server. https://git.io/fh9jW
21:14<Samu>visual studio decided to hang in the middle of a rebase.. great!
21:21<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7104: Fix #5405: Aircraft could route to depots outside their range https://git.io/fhKsL
21:29<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7094: Fix #7088: Retrieve an appropriate name for a non-existant AI/GS when displaying a textfile https://git.io/fhrqY
21:34<Samu>why did this fail? https://github.com/OpenTTD/OpenTTD/pull/7486/checks?check_run_id=98495762
21:39<DorpsGek_II>[OpenTTD/OpenTTD] SamuXarick updated pull request #7094: Fix #7088: Retrieve an appropriate name for a non-existant AI/GS when displaying a textfile https://git.io/fhrqY
21:52<DorpsGek_II>[OpenTTD/OpenTTD] nikolas updated pull request #7086: Change #6173: Update SDL driver to use SDL 2.0 https://git.io/fhamZ
22:13-!-Flygon [~Flygon@58-6-72-150.dyn.iinet.net.au] has joined #openttd
22:13-!-Flygon is "Flygon" on #openttd
22:35-!-D-HUND [~debdog@2a00:79c0:667:c600:7a24:afff:fe8a:d04d] has joined #openttd
22:35-!-D-HUND is "Wowbagger" on #bitlbee #openttd
22:38-!-debdog [~debdog@2a00:79c0:630:8700:7a24:afff:fe8a:d04d] has quit [Ping timeout: 480 seconds]
23:01-!-glx [kvirc@000128ec.user.oftc.net] has quit []
---Logclosed Thu Apr 11 00:00:51 2019