Times are UTC Toggle Colours
00:11:50 *** FLHerne has quit IRC 00:45:36 *** Zuu has quit IRC 03:41:13 *** Supercheese has quit IRC 03:41:45 *** Supercheese has joined #openttd.dev 04:37:09 *** Supercheese has quit IRC 04:44:04 *** Supercheese has joined #openttd.dev 05:24:33 *** Supercheese has quit IRC 05:47:59 *** Supercheese has joined #openttd.dev 08:23:28 *** Zuu has joined #openttd.dev 08:23:29 *** ChanServ sets mode: +v Zuu 08:26:40 *** ntoskrnl has joined #openttd.dev 08:31:53 *** Zuu has quit IRC 08:45:31 *** Supercheese has quit IRC 08:52:27 *** andythenorth has joined #openttd.dev 08:52:27 *** ChanServ sets mode: +v andythenorth 08:55:54 *** fonsinchen has joined #openttd.dev 08:55:54 *** ChanServ sets mode: +v fonsinchen 09:26:03 *** Alberth has joined #openttd.dev 09:26:03 *** ChanServ sets mode: +v Alberth 09:30:01 *** andythenorth has quit IRC 09:30:44 *** andythenorth has joined #openttd.dev 09:30:44 *** ChanServ sets mode: +v andythenorth 10:03:54 *** ntoskrnl has quit IRC 10:16:35 *** ntoskrnl has joined #openttd.dev 11:21:54 *** FLHerne has joined #openttd.dev 11:43:37 *** frosch123 has joined #openttd.dev 11:43:37 *** ChanServ sets mode: +v frosch123 12:18:22 <Alberth> regarding the 'inline' problem http://bugs.openttd.org/task/5375 http://www.tt-forums.net/viewtopic.php?p=1025338#p1025338 the other option to s/inline/INLINE/ is of course to remove the #define 12:19:18 <Alberth> but I don't know whether the Windows build likes that :p 12:20:07 <frosch123> well, iirc the point of FORCEINLINE was that yapf was significantly faster 12:20:26 <frosch123> replacing all inline with INLINE is certainly wrong 12:20:51 <frosch123> either the normal 'inline' works, or we need to reintro 'FORCEINLINE' for stuff were we really want it 12:20:57 <Alberth> I fully agree on not wanting INLINE :) 12:20:58 <frosch123> i hope SmatZ has a clue :) 12:22:04 <frosch123> many 'inline' are for 'static inline' functions in header files, and the only point of 'inline' is to suppress the 'function is unused' warning 12:23:39 <Alberth> yapf has getters like const VehicleType * GetVehicle() const { return m_veh; } 12:23:53 <Alberth> perhaps removing such stuff helps already 12:24:18 <frosch123> i would prefer FORCEINLINE above removing accessors .) 12:24:25 <frosch123> s/above/over/ 12:46:54 *** FLHerne has quit IRC 13:23:15 *** Zuu has joined #openttd.dev 13:23:15 *** ChanServ sets mode: +v Zuu 14:34:57 <SmatZ> hello :-) 14:35:33 <frosch123> summoning completed :) 14:35:43 <SmatZ> ;-) 14:36:45 <SmatZ> the problem with gcc is that sometimes, it won't inline a simple getter/setter because of reaching some limit, or because of wrongly guessing the effect of inlining.. I suppose MSVC has the same problem, so that's why FORCEINLINE was introduced 14:37:58 <SmatZ> I tried to do a systematic using of inline/FORCEINLINE 14:38:05 <SmatZ> but I wasn't really successful 14:38:33 <frosch123> well, FORCEINLINE was removed :) not it looks like we have to reintroduce it, just where :p 14:38:42 <frosch123> s/not/now/ 14:41:39 * Alberth prefers as few places as possible 14:43:26 <frosch123> well, all of pathfinders? all of map accessors? all of core/ ? 14:44:07 <frosch123> all of blitter/ ? :p 14:44:18 <frosch123> all of smallmap? 14:44:43 <frosch123> if you want to profile all of that stuff, that means a lot of boring work :p 14:47:16 *** FLHerne has joined #openttd.dev 14:47:39 <SmatZ> yup :) 14:51:31 <SmatZ> in the end, I decided not to touch using of inline/FORCEINLINE, and let the compiler do its work... 14:56:17 <Alberth> that seems the only sane way :) 14:56:35 <frosch123> hmm, who removed FORCEINLINE then? 14:57:11 <frosch123> ah, truebrain broke it all :p 14:58:45 <frosch123> http://paste.openttdcoop.org/show/1959/ <- usage of FORCEINLINE before it was remoed 14:59:34 <frosch123> maybe we could just use that list, and remove the things which obviously do not need it 14:59:49 <frosch123> like newgrf_gui :p 16:26:28 *** ntoskrnl11 has joined #openttd.dev 16:26:45 *** Zuu_ has joined #openttd.dev 16:33:45 *** ntoskrnl has quit IRC 16:33:50 *** Zuu has quit IRC 17:15:31 *** Alberth has quit IRC 17:17:02 *** FLHerne has quit IRC 17:33:25 *** andythenorth has quit IRC 17:54:24 *** andythenorth has joined #openttd.dev 17:54:25 *** ChanServ sets mode: +v andythenorth 18:00:56 *** FLHerne has joined #openttd.dev 18:02:29 *** andythenorth has quit IRC 19:17:41 <frosch123> http://devs.openttd.org/~frosch/diffs/diffsettings/ <- first diff is unrelated, rest is about removal of difficulty settings 19:25:10 *** andythenorth has joined #openttd.dev 19:25:10 *** ChanServ sets mode: +v andythenorth 19:36:25 *** andythenorth has quit IRC 19:52:04 *** ntoskrnl11 has quit IRC 19:55:28 *** andythenorth has joined #openttd.dev 19:55:28 *** ChanServ sets mode: +v andythenorth 20:18:23 *** Supercheese has joined #openttd.dev 21:03:20 *** andythenorth has quit IRC 21:18:50 *** fonsinchen has quit IRC 22:59:19 *** frosch123 has quit IRC 23:19:59 *** FLHerne has quit IRC