On Sat, 01 Mar 2014 10:03:41 +0100
Michael Vehrs <michael.bursc...@gmx.de> wrote:
> That should work. However, I suspect that we may be adding another 
> production type instead of replacing the old one.

Of course you are right, it just adds another production type.

> I'm afraid the semantics of spec loading were never clearly stated.

Generally it looks like elements have additive semantics.  I am hoping that
the serialization cleanup made this more consistent, although it was not a
goal at the time.  Attributes should be consistent because they are
controlled in FreeColXMLReader by the "preserve" or "extends" tags.  I
think we have optimized for the common case, which is good.  I'll check
the element stuff again and add some documentation to developer.tex.

Anyway I have made <production> elements respect the "delete" attribute, as
currently operates for Modifiers et al (git.c6115be).  With that in place
with git.0eb448d I can see horses limited to grain only in freecol-mode.

Finally, its done.  Modulo the user grumbling of course.

Cheers,
Mike Pope

Attachment: signature.asc
Description: PGP signature

------------------------------------------------------------------------------
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce.
With Perforce, you get hassle-free workflows. Merge that actually works. 
Faster operations. Version large binaries.  Built-in WAN optimization and the
freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.clktrk
_______________________________________________
Freecol-developers mailing list
Freecol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freecol-developers

Reply via email to