Times are UTC Toggle Colours
21:02:11 *** tokai|noir has joined #openttd 21:02:11 *** ChanServ sets mode: +v tokai|noir 21:08:09 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1350938715199115365/image.png?ex=67d88f38&is=67d73db8&hm=65b2d027d83f67189352c50a33ea9f163f8bd254b76a4fdb399bb6642dad5ec6& 21:08:09 <andythenorth> I got 21:08:13 <andythenorth> but that was Chrome 😛 21:09:11 *** tokai has quit IRC 21:14:16 *** nielsm has quit IRC 21:17:27 <_zephyris> I got `ERR_ON_THE_SIDE_OF_CAUTION`, but that's just good advice. 21:21:29 <DorpsGek> [OpenTTD/OpenTTD] LC-Zorg commented on pull request #13835: Change: Close error message window on left click https://github.com/OpenTTD/OpenTTD/pull/13835#issuecomment-2727652928 21:23:58 <truebrain> Lol 21:30:29 <andythenorth> it's been a while 🙂 21:37:34 *** Wolf01 has quit IRC 22:04:46 <locosage> KuhnovicviaGitHub: hm, something like this would be quite fitting for cmclient 22:04:56 <locosage> with setting ofc 22:06:24 <xarick> how to backtrack? 22:06:51 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1350953487642660874/image.png?ex=67d89cfa&is=67d74b7a&hm=02b024d29ec804277b5c773868d1a88f88110d0ba401bc327c24a187c424a50f& 22:07:12 <xarick> green was faster 22:07:25 <xarick> so i want blue to move back one piece and pf again 22:08:48 <xarick> must think 22:15:46 *** HerzogDeXtEr has quit IRC 22:20:01 <DorpsGek> [OpenTTD/OpenTTD] TheMowgliMan opened issue #13837: [Bug]: Maximum speeds do not match https://github.com/OpenTTD/OpenTTD/issues/13837 22:20:36 *** kuka_lie has quit IRC 22:26:18 <DorpsGek> [OpenTTD/OpenTTD] James103 commented on issue #13837: [Bug]: Maximum speeds do not match https://github.com/OpenTTD/OpenTTD/issues/13837 22:32:35 <DorpsGek> [OpenTTD/OpenTTD] TheMowgliMan commented on issue #13837: [Bug]: Maximum speeds do not match https://github.com/OpenTTD/OpenTTD/issues/13837 22:33:53 <DorpsGek> [OpenTTD/OpenTTD] WenSimEHRP commented on issue #13837: [Bug]: Maximum speeds do not match https://github.com/OpenTTD/OpenTTD/issues/13837 22:35:00 <DorpsGek> [OpenTTD/OpenTTD] James103 opened issue #13838: [Bug]: Invalid parameter in error message when an industry is in the way https://github.com/OpenTTD/OpenTTD/issues/13838 22:35:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #13837: [Bug]: Maximum speeds do not match https://github.com/OpenTTD/OpenTTD/issues/13837 22:35:11 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #13837: [Bug]: Maximum speeds do not match https://github.com/OpenTTD/OpenTTD/issues/13837 23:00:39 <peter1138> Hmm, either I add a load of `std::move()`, or just copy the encoded string :S 23:01:11 <peter1138> Or. 23:01:15 <peter1138> Why does this work? 23:01:19 <peter1138> void CommandCost::AddCost(const CommandCost &ret) 23:01:20 <peter1138> { 23:01:21 <peter1138> ... 23:01:24 <peter1138> this->encoded_message = std::move(ret.encoded_message); 23:01:25 <peter1138> } 23:02:13 <_jgr_> The std::move will just do nothing there 23:05:09 <frosch123> std::move is just a cast. In this case it will cast to `const string &&`. Move assignment does not match the const, so overload resolution matches the copy assignment 23:05:51 <frosch123> This is C++ :) 23:13:13 <peter1138> Okay, can I make the function accept CommandCost &ret and CommandCost &&ret... :S 23:15:06 <frosch123> Either you make it a template. Or you make it a plain value parameter 23:15:59 <frosch123> The latter results in one copy one move for const &, and zero copy two moves for && 23:16:38 <frosch123> So one move more expensive than the optimum 23:17:32 <frosch123> Which may be preferable over moving the function into the header as template 23:20:12 <_jgr_> Even then, the compiler may well optimise that away for LTO release builds 23:28:24 <frosch123> I think there was a meme making fun of "C++03: pass stuff as const ref, it's the best" vs "C++11: pass stuff by value, it's the best"