Changes between Initial Version and Version 1 of Ticket #3606, comment 6


Ignore:
Timestamp:
Dec 31, 2015, 12:03:48 PM (8 years ago)
Author:
mimo

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #3606, comment 6

    initial v1  
    1 Thanks for the new patch. There is nonetheless something which does not look nice with this approach (sorry, i didn't think about it yesterday when I proposed it). It is that the concept of idle unit is really a UnitAI concept, which does not mean anything for all structures and trees. So maybe it is better to make IdleRegenRate optional (and then you would have to test its existence each time you use it), or if anybody has a better idea ?
     1Thanks for the new patch. There is nonetheless something which does not look nice with this approach (sorry, i didn't think about it yesterday when I proposed it). It is that the concept of idle unit is really a UnitAI concept, which does not mean anything for all structures and trees. So maybe it is better to make IdleRegenRate optional (and then you would have to test its existence each time you use it), or if anybody has a better idea ?
     2
     3Edit: after discussion on irc with sanderd17, having it optional should be good enough
    24
    35Otherwise, concerning the patch itself: