IRC logs for #openttd on OFTC at 2022-11-24
⏴ go to previous day
00:01:57 *** NGC3982 has joined #openttd
00:04:37 *** WormnestAndroid has joined #openttd
00:09:13 <petern> 1GB of NewGRF downloads, wtf...
00:09:26 <petern> I'll hazard a guess at sprite cache exhaustion
00:11:25 <JGR> World ticks is around 52 ms on vanilla on my machine
00:11:57 <JGR> Even with nothing of interest in the viewport that is a tad slow
00:19:18 <petern> Urgh, why the heck do they include a green background in the railtype icons o_O
00:30:22 <JGR> Animated house tiles seems to be the biggest cost
00:33:03 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
00:34:11 *** WormnestAndroid has joined #openttd
00:38:21 <petern> You know it's a stupidly sized map when just opening the town list pauses it
00:44:54 <JGR> It looks like the NewGRF author is using animation callback 1B to cache neighbouring tile type checks
00:46:12 <JGR> Adding or removing animated tiles is O(N) in vanilla so that soon gets expensive
03:40:26 *** debdog has quit IRC (Ping timeout: 480 seconds)
06:28:35 *** Tirili has quit IRC (Quit: Leaving)
06:42:32 *** HerzogDeXtEr has joined #openttd
07:23:36 *** sla_ro|master has joined #openttd
07:27:00 *** Speedy` has joined #openttd
07:53:00 *** D-HUND is now known as debdog
08:25:12 *** WormnestAndroid has quit IRC (Remote host closed the connection)
09:13:03 <petern> Hmm, how can I ensure a parameter passed to a function is a constant.
09:16:21 <dwfreed> you mean a literal, or just something that satisfies constexpr rules?
09:17:14 <petern> Something that satifies.
09:17:30 <reldred> ahh you want a snickers then
09:26:09 <petern> Urgh, so it's not just a keyword I'm missing 😦
11:27:28 <Eddi|zuHause> about a decade ago i read some article demanding the format parameter of printf to be a constant to avoid a large fraction of vulnerabilities
11:32:11 <Eddi|zuHause> my c++-fu doesn't go deep enough to begin to understand what these people are trying to do there
13:08:16 *** wallabra has joined #openttd
14:04:07 *** ChanServ sets mode: +v tokai
14:10:57 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
14:38:46 *** WormnestAndroid has joined #openttd
15:27:32 *** ufo-piloot has quit IRC (Ping timeout: 480 seconds)
15:45:43 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
15:45:59 *** WormnestAndroid has joined #openttd
16:23:14 *** virtualrandomnumber has joined #openttd
16:23:42 *** virtualrandomnumber has quit IRC ()
17:30:24 <dP> New command system is so confusing...
17:30:24 <dP> Command call sends vehicle->tile to CMD_SKIP_TO_ORDER:
17:30:25 <dP> but CmdSkipToOrder doesn't accept TileIndex argument:
17:30:26 <dP> I get that the first argument (error message) gets eaten by one of the Post overloads but what is the point of sending tile there and where does it get extracted from the arguments?
17:33:23 <dP> oh, nvm, just found a bunch of additional post overloads in other place
17:38:42 *** gelignite has joined #openttd
18:14:24 *** ufo-piloot has joined #openttd
18:46:28 <DorpsGek> - Update: Translations from eints (by translators)
18:46:37 *** ufo-piloot has quit IRC (Ping timeout: 480 seconds)
19:04:32 *** ufo-piloot has joined #openttd
19:13:43 <dwfreed> Eddi|zuHause: that is only checked by the compiler by convention (on anything marked as printf-like), not enforced by some standard keyword
19:14:13 <dwfreed> (and that warning can be disabled, but you get to keep the pieces)
19:54:21 <wallabra> 90% of the time, monorails are in bed with corrupt politicians
19:54:27 <wallabra> ... don't ask me how that works, sex finds a way, okay!
20:26:11 <andythenorth> every time I think I've learnt more git
20:26:29 <andythenorth> why does dropping the most recent commit produce a rebase conflict?
20:26:36 <andythenorth> surely it's just removing a ref cleanly?
20:28:40 <andythenorth> and why does `rebase -i HEAD~2` give a list of about 30 commits?
20:29:30 <andythenorth> maybe it's the presence of a merge commit 😛
20:30:56 <JGR> What operation are you trying to do?
20:31:35 <JGR> Also, I find that looking at things using gitk makes things much clearer
20:31:54 <andythenorth> (2) I want to get rid of the merge commit
20:32:23 <andythenorth> ok google has answers I think
20:34:04 <andythenorth> I never did learn a non-merge workflow 😛
20:34:14 <andythenorth> merge is standard in the day job
20:34:37 <JGR> Unless you really have to get rid of the merges, I'd just leave them be
20:34:52 *** ufo-piloot has quit IRC (Ping timeout: 480 seconds)
20:34:54 <JGR> There are two merge commits
20:36:20 <andythenorth> ok I just live with those
20:36:32 * andythenorth wanted a cleaner history, but I doubt anyone cares IRL
20:36:46 <andythenorth> only AIs in the future will care, when they do cultural archeology
20:36:53 <andythenorth> and they can work around it
20:43:08 <bigyihsuan> with git removing a commit is also a commit itself lol
20:44:11 <JGR> Reverting a commit is a different thing to removing it
20:45:46 *** gelignite has quit IRC (Ping timeout: 480 seconds)
20:47:29 *** ufo-piloot has joined #openttd
20:48:02 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
20:53:30 *** ChanServ sets mode: +v peter1138
20:55:44 <TallTyler> #8304 is ready for merge, if anyone wants to take a look (I approved it but don't want to unilaterally merge)
20:58:22 <LordAro> TallTyler: #10149 is out of date :p
21:00:17 <TallTyler> Oh, thought you meant something else 😛
21:02:03 <LordAro> just get petern to fix it
21:02:35 <TallTyler> Vehicle lists are always kinda ugly
21:06:15 <TallTyler> LordAro: #10093 was partly your idea in a discussion here, if you have any thoughts on my implementation 🙂
21:07:34 <LordAro> there was the issue brought up that if you upgrade with it set to TTD-mapgen, there's no obvious way of changing it
21:07:55 <LordAro> and also the fact that TGP kinda sucks
21:08:21 <TallTyler> Saveload to set it to Terragen when upgrading from an older version?
21:08:47 <LordAro> it's nothing to do with saveload though :p
21:08:55 <LordAro> not sure if we even can do that...
21:09:57 <TallTyler> Hmm, I will do some more thinking
21:10:13 <LordAro> i'll stick it in the PR so it doesn't get forgotten :)
21:11:04 <dP> LordAro: add to settings ui?
21:11:47 <dP> also, anything else on #10001?
21:12:00 <LordAro> dP: nothing from me, i approved it
21:12:07 <LordAro> but it's too big for me to merge it on my own
21:15:28 *** ufo-piloot_ has joined #openttd
21:15:42 *** ufo-piloot has quit IRC (Read error: Connection reset by peer)
21:15:42 *** ufo-piloot_ is now known as ufo-piloot
21:17:04 <andythenorth> competitive shit 🙂
21:27:27 *** geli has quit IRC (Quit: Stay safe!)
21:38:09 <petern> LordAro: Especially that variety bullshit
21:39:27 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:10:51 *** Montana_ has joined #openttd
22:26:48 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:36:56 <petern> #8304 looks fine as merged, IMHO.
22:37:33 <petern> The layout issue was not caused by that PR.
22:40:12 <petern> Oh I see, with OpenGFX it's bad, because OpenGFX is wrong.
continue to next day ⏵