Log for #openttdcoop.devzone on 15th June 2010:
Times are UTC Toggle Colours
00:31:22  *** KenjiE20 has quit IRC
06:44:32  *** ODM has joined #openttdcoop.devzone
07:33:52  <Brot6> #openttdcoop NewGRF package - Bug #1009 (New): removed too many obsolete newgrfs (planetmaker) @
07:33:52  <Brot6> #openttdcoop NewGRF package - Bug #542: Remove OpenGFX newgrfs (planetmaker) @
07:35:38  <Brot6> #openttdcoop NewGRF package - Bug #550 (Closed): Remove ECS from George (planetmaker) @
09:53:19  *** KenjiE20 has joined #openttdcoop.devzone
13:32:04  <Brot6> NFO Meta Language - Feature #969: Parse #file and #line lines that are written by a preprocessor (yexo) @
13:36:30  <Brot6> NFO Meta Language - Feature #969: Parse #file and #line lines that are written by a preprocessor (yexo) @
14:52:14  <Brot6> NFO Meta Language - Revision 299:4665232e95d9: Feature: support for conditional expressions in ac... (yexo) @
15:03:21  *** Seberoth has joined #openttdcoop.devzone
16:12:06  *** frosch123 has joined #openttdcoop.devzone
16:15:48  <Brot6> NFO Meta Language - Revision 300:af1ae084ac48: Change: allow expressions (instead of only IDs) as... (yexo) @
16:18:58  <Brot6> nml: update from r292 to r300 done -
16:19:04  <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r545), 32bpp-extra (r36), airportsplus (r50), bros (r12), comic-houses (r70), firs (r910), fish (r375), heqs (r320), newgrf_makefile (ERROR r99), nmts (r16), nutracks (r69), ogfxplus (r27), opengfx (r459), openmsx (r57), opensfx (r94), snowlinemod (r12), swedishrails (r25), test (ERROR r47), worldairlinersset (r643)
17:26:11  *** Seberoth has quit IRC
17:27:31  *** Seberoth has joined #openttdcoop.devzone
17:32:38  *** Alberth has joined #openttdcoop.devzone
17:46:15  <Brot6> NFO Meta Language - Revision 301:3778d42fa25b: Fix: print action3 mappings in the same order as t... (yexo) @
17:59:34  <Brot6> NFO Meta Language - Feature #969: Parse #file and #line lines that are written by a preprocessor (Alberth) @
18:06:48  <Brot6> NFO Meta Language - Feature #995: Handle errors more gracefully (Alberth) @
18:10:31  <Brot6> NFO Meta Language - Feature #995: Handle errors more gracefully (yexo) @
18:13:23  <Alberth> hai
18:14:18  <Alberth> can you explain the warning thingie in #1008? where/when would such a warning be given?
18:15:21  <Yexo> nml checks the version where all used features are supported and determines you need at least openttd r19000 for a feature that is used
18:15:38  * Alberth nods
18:15:40  <Yexo> when the nml doesn't have a "minimum_openttd_version(1, 0, 0, 19000)" or higher statement then a warning is given
18:16:14  <Alberth> ah, to the nml file author. that makes sense
18:16:47  <Yexo> yes, as planetmaker said "But in order to elaborate on it: I don't think NML should go and decide for the programmer which minimum version requirement a newgrf should have."
18:18:00  <Alberth> I read that differently, as in 'nml should not bother about minimum version', and then I got lost.
18:18:09  <Alberth> thanks for the clarification
18:20:35  <Alberth> About #995 and #969, I think they are seperate issues, except that they both start with position reporting by ply. The only thing we should synchronize imho is that a solution with positions for one bug is also usable by the other bug.
18:21:21  <Alberth> ie #995 can perfectly be done with a position-class containing only a line number or so.
18:22:32  <Yexo> yes, but if the Position-class is done then #969 is done with the line_directives.diff patch I added there
18:23:26  <Hirundo> <- Is this really easier / better than using action9?
18:25:17  <Yexo> Hirundo: action7/9 can't compare parameter with eachother
18:25:33  <Hirundo> action6 should fix that
18:25:36  <Yexo> so it'd be action6 to write param2 value, then action7/9 to compare param1 with written value
18:25:45  <Yexo> then an actionD to set the target to 1
18:25:54  <Yexo> before that all you need an actionD to set the target to 0
18:26:47  <Yexo> you're right, here action7/9 might be better but we need code very much like this anyway to support the same operators in action2 code
18:27:44  <Hirundo> Action2 has op 12
18:28:48  <Yexo> hmm, I forgot about that one
18:29:43  <Yexo> ok, I do agree that using action7/9 would be better
18:30:02  <Hirundo> though this works, and working code is better than no code whatsoever
18:30:51  <Yexo> using action7/9 will introduce yet another cyclic dependency
18:32:44  <Hirundo> Could fix that, by splitting the business logic of actionD/7/9 (get_tmp_parameteter) to a new file that imports Action7/9/D
18:33:35  <Yexo> yep, indeed
18:39:35  <Yexo> Hirundo: how do you plan to solve #974
18:40:57  <Hirundo> I don't have a complete plan yet
18:40:59  <Hirundo> first step, which is mostly finished, is to count the bytes inside the Output classes
19:17:10  <Alberth> regression test fails
19:17:48  <Alberth> nml/actions/", line 63, in parse_graphics_block
19:18:01  <Alberth> TypeError: 'NoneType' object is not iterable
19:25:55  <Yexo> fixed
19:26:53  <Brot6> NFO Meta Language - Revision 302:3e01cae1bc33: Fix (r301): no graphics blocks should be an empty ... (yexo) @
19:48:29  <Brot6> NFO Meta Language - Revision 303:a1e7d5a70f2c: Fix (r294): One reduce_expr() was left. (Alberth) @
19:57:28  <Brot6> NFO Meta Language - Revision 304:112ababa53e9: Codechange: Moving some classes from ast to action1. (Alberth) @
19:57:28  <Brot6> NFO Meta Language - Revision 305:d3b6a7e81314: Codechange: Moving ParameterAssignment from ast to... (Alberth) @
19:57:28  <Brot6> NFO Meta Language - Revision 306:4e0fa08f10cf: Codechange: Moving SwitchRange class from ast to a... (Alberth) @
19:57:32  <Brot6> NFO Meta Language - Revision 307:42e3f83f34d3: Codechange: Move a few classes from ast to real_sp... (Alberth) @
19:57:53  <PeterT> why does Brot6 say 3 revisions in the same second?
19:58:20  <Alberth> last 4 all got pushed together
19:58:50  <Alberth> so the real question is, why is the last one 4 seconds late?
19:59:14  <PeterT> didn't know you could push 4 different revisions together ;-)
19:59:25  <PeterT> err...wrong emoticon
19:59:27  <PeterT> O_o
19:59:56  <Alberth> with mercurial and git, you can, since you first commit locally, then push all changes that are not at the server
20:00:35  <PeterT> ah, git ,3
20:00:37  <PeterT> *<3
20:00:48  <Alberth> mercurial, actually :p
20:01:11  <Alberth> git has too many settings
20:01:15  <PeterT> yeah
20:01:21  <PeterT> but git's branches, wow.
20:01:32  <Alberth> mercurial also has branches
20:02:04  <Alberth> I think they are more or less equal
20:10:33  <PeterT> either way, they are both slow with cloning
20:10:37  <Brot6> NFO Meta Language - Revision 308:be8dbdbd084d: Codechange: Explicitly import expression module in... (Alberth) @
20:11:59  <Alberth> jay, we got a 6.42/10 from pylint for :D
20:26:03  <Brot6> FIRS Industry Replacement Set - Feature #1010 (New): Clustering for mines (andythenorth) @
20:26:18  <planetmaker> good evening :-)
20:26:29  <andythenorth> hi hi
20:27:07  <planetmaker> uh... busy NML evening :-)
20:27:07  <Yexo> Alberth: yes, and -7.43/10 for (yes, a negative score)
20:27:13  <Yexo> evening planetmaker
20:27:32  <Brot6> FIRS Industry Replacement Set - Feature #1011 (New): Clustering for forests (andythenorth) @
20:27:44  * planetmaker calles 'update_repos' :-)
20:27:59  <Alberth> planetmaker: our leader has made lots of decisions :)
20:28:11  <planetmaker> good leader we have then :-)
20:29:05  * Alberth agrees
20:29:14  <Brot6> FIRS Industry Replacement Set - Feature #1012 (New): Oil wells should be oil field - high product... (andythenorth) @
20:29:46  <Yexo> planetmaker: did you already write some constants for those railtype numbers? if not I'll do that now
20:30:04  <planetmaker> I did, but I had to leave before I could commit
20:30:11  <planetmaker> let me get you the paste for review
20:31:41  <planetmaker> <-- that worked for me, but I haven't made sure enough it has no side effects
20:32:15  <Alberth> Yexo: I tried making better last weekend, but failed due to the imports
20:32:40  <Brot6> NFO Meta Language - Revision 309:167b9e182367: Codechange: Explicit module import for several act... (Alberth) @
20:32:58  <Yexo> planetmaker: that works, but that breaks using cargo labels in vehicle action3's
20:33:08  <planetmaker> I feared something like that
20:33:19  <Yexo> but with that list of IDs it's easy to implement
20:33:42  <planetmaker> giving it another name doesn't do the job. How is it proper?
20:34:48  <Yexo> writing a function that returns the proper list (either cargo_number for vehicle/station features or the list in that pastebin for railtypes) and use that instead of always [cargo_numbers]
20:36:27  * andythenorth reads some nml source
20:37:40  <planetmaker> hm... I guess you know already how to do that, Yexo :-)
20:37:59  <Alberth> good night all
20:38:15  <planetmaker> g'night Alberth
20:38:52  *** Alberth has left #openttdcoop.devzone
20:39:54  <Brot6> FIRS Industry Replacement Set - Feature #977 (New): Rework Aluminium Plant (andythenorth) @
20:40:15  <Yexo> planetmaker: thanks for the list with constants :)
20:40:36  <planetmaker> that's the no-brainer :-P
20:41:08  <Brot6> NFO Meta Language - Revision 310:7a4893da6ce5: Feature: constants for use in railtype action3 (ba... (yexo) @
21:02:21  *** frosch123 has quit IRC
22:24:29  <Brot6> Swedish Rails - Revision 26:8a270c3fa811: Feature: Normal and electrical rails use railtypes (planetmaker) @
22:31:32  *** ODM has quit IRC
22:42:46  <Brot6> Swedish Rails - Revision 27:7454303aab67: Fix: Don't forget to actually add the source files (planetmaker) @
23:00:44  <Brot6> Swedish Rails - Revision 28:380e0e6eec51: Fix: Add simple level crossings (planetmaker) @
23:24:24  <Brot6> Swedish Rails - Revision 29:a61806623969: Doc: Update repo URL and some minor improvements (planetmaker) @
23:27:36  *** Seberoth has quit IRC
23:28:55  <Brot6> Swedish Rails - Feature #989 (Closed): railtypes support for OpenTTD (planetmaker) @
23:28:55  <Brot6> Swedish Rails - Feature #988 (Closed): support for tropical climate (planetmaker) @
23:30:32  *** KenjiE20 has quit IRC
23:30:52  <Brot6> Swedish Rails - Feature #987 (Closed): Improved snowy TTD track sprites (planetmaker) @
23:33:15  <Brot6> Swedish Rails - Feature #1013 (New): rail type level crossing sprites (planetmaker) @
23:35:42  <Brot6> Swedish Rails - Feature #1014 (New): ttdp - style level crossings (planetmaker) @
23:35:42  <Brot6> Swedish Rails - Bug #1015 (New): tunnel sprites with OpenGFX-style ground sprites (planetmaker) @
23:37:23  <Brot6> Swedish Rails - Feature #1016 (New): depot sprites (planetmaker) @

Powered by YARRSTE version: svn-trunk