Times are UTC Toggle Colours
02:56:30 *** Lakie has quit IRC 14:26:58 *** Lakie has joined #openttdcoop.devzone 14:28:35 *** ODM has joined #openttdcoop.devzone 15:00:38 <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @ http://dev.openttdcoop.org/issues/2410#change-6792 15:01:04 <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @ http://dev.openttdcoop.org/issues/2410#change-6792 15:44:01 <Brot6> NewGRF Meta Language - Revision 1375:117c6353d899: Doc #2410: All remaining industry 60+x variables. (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/117c6353d899 15:44:01 <Brot6> NewGRF Meta Language - Revision 1376:8b22c2ab743a: Remove #2410: Duplicate industry variable comp... (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/8b22c2ab743a 15:44:01 <Brot6> NewGRF Meta Language - Revision 1377:c191a974b9ec: Change #2410: Rename company_xx industry vars ... (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/c191a974b9ec 15:44:32 <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @ http://dev.openttdcoop.org/issues/2410#change-6793 15:46:28 <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @ http://dev.openttdcoop.org/issues/2410#change-6793 15:47:08 <Yexo> Hirundo: I'd like to discuss the advantages/disadvantages of writing nml files before or after pre_process again 16:02:41 <Hirundo> After giving last discussion some thought, I think it's best to do it before 16:03:19 <Hirundo> So basically all reduce() calls need to be made from the pre_process stage 16:03:31 <Yexo> currently that isn't done 16:03:49 <Yexo> that also means that when the input is "1+1" we'll output "1+1", not "2" 16:04:16 <Hirundo> Yes, but does that really matter? 16:04:21 <Yexo> not really 16:04:48 <Hirundo> the nml->nml output is basically a no-op anyway 16:04:54 <Yexo> indeed 16:05:14 <Yexo> do we in general want any errors thrown from ast::__init__ or only from pre_process? 16:05:20 <Yexo> and if so, in which case? 16:05:39 <Hirundo> IMO there may be some basic error checks in init, like number of parameters 16:06:00 <Hirundo> Just the stuff you need to parse the constructor arguments into a sane structure 16:06:00 <Yexo> agreed 16:06:07 * Hirundo resumes cooking 16:39:17 *** frosch123 has joined #openttdcoop.devzone 16:59:09 <Hirundo> Yexo: We could do a review/sanitization of the whole AST, fixing both this and adding comments/regression coverage where needed 16:59:33 <Yexo> I'm fixing this problem right now 17:00:41 <Hirundo> nice :) 17:00:59 <Yexo> adding comments / regression coverage is another good project :) 17:01:07 <Hirundo> I'm off to sports now, I'll be back later tonight 17:01:16 <Yexo> have fun :) 17:01:22 <planetmaker> is "much sweat" a good wish then? :-) 17:01:26 <planetmaker> anyway, enjoy ;-) 17:10:15 <Brot6> nml: update from r1374 to r1377 done - http://bundles.openttdcoop.org/nml/nightlies/r1377 17:17:26 <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r750), 32bpp-extra (r40), ai-admiralai (r75), ai-aroai (r39), ailib-common (r21), ailib-direction (r17), ailib-list (r32), ailib-string (r29), ailib-tile (r16), airportsplus (r93), basecosts (r25), belarusiantowns (r8), bros (r52), chips (r141), comic-houses (r71), firs (r1996), fish (r653), frenchtowns (r6), german-townnames (r34), grfcodec (r830), grfpack (r279), heqs 17:17:26 <Brot6> (r605), indonesiantowns (r41), manindu (r7), metrotrackset (r56), narvs (r37), newgrf_makefile (ERROR r293), nml (r1377), nutracks (r201), ogfx-industries (r119), ogfx-landscape (r69), ogfx-rv (r107), ogfx-trains (r241), ogfx-trees (r49), opengfx (r673), openmsx (r97), opensfx (r97), smts (r19), snowlinemod (r49), spanishtowns (r10), sub-landscape (ERROR r66), sub-opengfx (ERROR r666), swedishrails (r202), swisstowns (r22), 17:17:28 <Brot6> transrapidtrackset (r15), ttdviewer (r34), ttrs (r36), worldairlinersset (r672) 17:20:26 <Brot6> newgrf_makefile: compile of r293 still failed (#2656) - http://bundles.openttdcoop.org/newgrf_makefile/nightlies/ERROR/r293 17:23:11 <Ammler> this error got old ;-) 17:23:37 <Brot6> sub-landscape: compile of r66 still failed (#2616) - http://bundles.openttdcoop.org/sub-landscape/nightlies/ERROR/r66 17:24:12 <Ammler> hmm, and this, shall we remove those subrepo tests? 17:24:25 <Brot6> sub-opengfx: compile of r666 still failed (#2586) - http://bundles.openttdcoop.org/sub-opengfx/nightlies/ERROR/r666 17:24:52 <planetmaker> Ammler, don't remove the repos... the test has not been done really ;-) 17:25:16 <Brot6> Following repos rebuilds successful without any difference to earlier nightlies builds: airportsplus (Diffsize: 12), belarusiantowns (Diffsize: 30), frenchtowns, german-townnames, indonesiantowns (1 errors) (Diffsize: 1), manindu (Diffsize: 1), narvs, ogfx-industries, ogfx-landscape (Diffsize: 21), ogfx-rv, ogfx-trains (Diffsize: 813), spanishtowns (Diffsize: 1), swedishrails (Diffsize: 385), swisstowns 17:26:00 <planetmaker> hm... what does the newgrf framework fail on? 17:26:17 <Ammler> scripts/Makefile.common:12: *** missing separator. Stop. 17:26:25 <planetmaker> ah, drat 17:26:59 <Ammler> I guess, the building worked, the check failed 17:28:10 <planetmaker> seems like 17:28:44 <planetmaker> how was the check performed? 17:29:42 <planetmaker> nvm. Building the thing 17:30:52 <Ammler> planetmaker: ./scripts/check 17:34:30 <Brot6> Example NewGRF Project - Revision 294:43d80525512b: Fix #2656: Missing separator in conditional e... (planetmaker) @ http://dev.openttdcoop.org/projects/newgrf-makefile/repository/revisions/43d80525512b 17:34:30 <Brot6> Example NewGRF Project - Bug #2656 (Closed): DevZone compile failed (planetmaker) @ http://dev.openttdcoop.org/issues/2656#change-6794 17:34:32 <Brot6> newgrf_makefile: update from r290 to r294 done - http://bundles.openttdcoop.org/newgrf_makefile/nightlies/r294 19:01:04 *** andythenorth has joined #openttdcoop.devzone 19:01:45 *** andythenorth has joined #openttdcoop.devzone 19:50:45 <Hirundo> The "much sweat" part certainly succeeded... 25 degrees in there 19:52:09 <Brot6> NewGRF Meta Language - Revision 1378:f4a80a9994eb: Change: create NML output before the pre_proce... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/f4a80a9994eb 19:52:09 <Brot6> NewGRF Meta Language - Revision 1379:e6a309622d1a: Change: open output files as late as possible ... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/e6a309622d1a 19:52:09 <Brot6> NewGRF Meta Language - Revision 1380:4904cfea760c: Codechange: move SwitchRange to its own file i... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/4904cfea760c 19:58:52 <Hirundo> Yexo: I think you've missed a 'hg add' 20:00:48 <Brot6> NewGRF Meta Language - Revision 1381:13561a045b4e: Fix r1380: forgot to hg add a file (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/13561a045b4e 20:00:54 <Yexo> I did indeed 20:16:45 <Hirundo> Yexo: wrt. industry tile var 43 (relative position)... 20:17:11 <Hirundo> I suggest to make separate variables for X and Y, and a third that contains both (lower 16 bits, 0xYYXX) 20:17:42 <Hirundo> The latter might be quite useful when selecting sprites based on location within the industry 20:18:42 <Yexo> agreed 20:32:21 *** andythenorth has quit IRC 20:32:35 *** andythenorth has joined #openttdcoop.devzone 20:37:01 <Brot6> NewGRF Meta Language - Bug #2410 (Closed): Industry and industry tile 0x4X variables missing from... (Hirundo) @ http://dev.openttdcoop.org/issues/2410#change-6795 20:37:02 <Brot6> NewGRF Meta Language - Revision 1382:33ada870c160: Feature: Provide a useful implementation for i... (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/33ada870c160 20:41:14 <Hirundo> Next job: Implementing var 7B and varact2 shifting operators in NML 20:42:39 <Yexo> are the varact2 shifting operators supported in openttd 1.1? 20:43:44 <Hirundo> I assumed they are, but I'll check to be sure 20:44:40 <Hirundo> They are 20:47:28 <Hirundo> Actually, they're already in NML :) 20:50:22 <Yexo> even better :) 20:51:44 <Yexo> var 7B is more tricky 21:11:28 *** andythenorth has quit IRC 21:12:15 *** andythenorth has joined #openttdcoop.devzone 21:36:12 *** andythenorth has left #openttdcoop.devzone 21:42:05 *** frosch123 has quit IRC 21:43:02 *** ODM has quit IRC 22:00:12 <Brot6> NewGRF Meta Language - Revision 1383:5680062f3e65: Codechange: Rework the way variable arguments ... (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/5680062f3e65 22:00:12 <Brot6> NewGRF Meta Language - Revision 1384:39c559c00af2: Feature: Support for non-constant parameters v... (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/39c559c00af2 22:13:33 <Hirundo> goodnight 22:15:41 <planetmaker> good night Hirundo 22:15:50 <planetmaker> lovely work today :-) 22:16:05 <Yexo> nice work indeed :) 22:18:05 <planetmaker> well both of you :-) 22:53:07 *** Lakie has quit IRC 23:25:40 <Brot6> NewGRF Meta Language - Revision 1385:c20c22247f13: Codechange: OptionParser doesn't raise an Exce... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/c20c22247f13