Log for #openttdcoop.devzone on 7th June 2011:
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) @
15:01:04  <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @
15:44:01  <Brot6> NewGRF Meta Language - Revision 1375:117c6353d899: Doc #2410: All remaining industry 60+x variables. (Hirundo) @
15:44:01  <Brot6> NewGRF Meta Language - Revision 1376:8b22c2ab743a: Remove #2410: Duplicate industry variable comp... (Hirundo) @
15:44:01  <Brot6> NewGRF Meta Language - Revision 1377:c191a974b9ec: Change #2410: Rename company_xx industry vars ... (Hirundo) @
15:44:32  <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @
15:46:28  <Brot6> NewGRF Meta Language - Bug #2410: Industry and industry tile 0x4X variables missing from document... (Hirundo) @
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 -
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) -
17:23:11  <Ammler> this error got old ;-)
17:23:37  <Brot6> sub-landscape: compile of r66 still failed (#2616) -
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) -
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) @
17:34:30  <Brot6> Example NewGRF Project - Bug #2656 (Closed): DevZone compile failed (planetmaker) @
17:34:32  <Brot6> newgrf_makefile: update from r290 to r294 done -
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) @
19:52:09  <Brot6> NewGRF Meta Language - Revision 1379:e6a309622d1a: Change: open output files as late as possible ... (yexo) @
19:52:09  <Brot6> NewGRF Meta Language - Revision 1380:4904cfea760c: Codechange: move SwitchRange to its own file i... (yexo) @
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) @
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) @
20:37:02  <Brot6> NewGRF Meta Language - Revision 1382:33ada870c160: Feature: Provide a useful implementation for i... (Hirundo) @
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) @
22:00:12  <Brot6> NewGRF Meta Language - Revision 1384:39c559c00af2: Feature: Support for non-constant parameters v... (Hirundo) @
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) @

Powered by YARRSTE version: svn-trunk