Changes between Version 2 and Version 3 of Ticket #1718, comment 11


Ignore:
Timestamp:
Dec 9, 2012, 11:12:11 PM (11 years ago)
Author:
historic_bruno

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1718, comment 11

    v2 v3  
    66   * It sounds nice, but I'd rather wait until we actually want another transforming behavior before going that route (It seems like a more generic `Transform` component means more very specific pack logic in `UnitAI`).
    77   * But we should at least consider factoring out whatever they share into global helper functions.
    8  * There is also duplication in the siege entity templates, which is mostly unavoidable since the templates don't allow multiple inheritance. For instance, packed entities duplicate the `Attack` stats of their unpacked counterparts because they need to know a.) if they can attack, b.) the target range, etc. If we update one, we have to remember to update the other.
    9   * One slight way of improving this: have a single generic siege template for each type, then have a civ-specific "common" template inherit from that, and then packed/unpacked templates inherit from the common template. Attack stats, etc would go in the common template. Like this:
     8 * ~~There is also duplication in the siege entity templates, which is mostly unavoidable since the templates don't allow multiple inheritance. For instance, packed entities duplicate the `Attack` stats of their unpacked counterparts because they need to know a.) if they can attack, b.) the target range, etc. If we update one, we have to remember to update the other.~~
     9  * ~~One slight way of improving this: have a single generic siege template for each type, then have a civ-specific "common" template inherit from that, and then packed/unpacked templates inherit from the common template. Attack stats, etc would go in the common template. Like this:~~
    1010   {{{
    1111                                                                                         /-- athen_mechanical_siege_oxybeles_packed