#70 Fix on lost equipment 0.8.3

closed-out-of-date
nobody
None
5
2010-11-16
2009-08-09
alimur
No

With very few changes is possible fix bug on lost equipment in combat model, with code from trunk (on 2009-07-26).
On test, I call it "0.8.4-alpha" ;-)

Changed files:
common/model/CombatModel.java
common/model/IndianSettlement.java
common/model/SimpleCombatModel.java

Discussion

  • alimur
    alimur
    2009-08-09

    Patch for lost equipment

     
    Attachments
  • alimur
    alimur
    2009-08-09

    While I'm sending diff file, I forgot to delete the reference to another changed file (Monarch.java), which is not very important :-))), from trunk: if (player.getTax() > MINIMUM_TAX_RATE + 10)

     
  • alimur
    alimur
    2009-08-10

    To give more details, clarifies that resolves the error in lock server on combat after the loss of horses. After many tests, I found that it does not crash again for this reason (which requires reconnect and ever waiting for a new turn to continue combat). In my proposal, regardless of the "getAutomaticEquipmen" and "TypeCountMap<EquipmentType>" which seems to be the difference with the official release 0.8.4, which I have been doubts in a fast tester session.

     
  • alimur
    alimur
    2009-08-17

    With these changes the game with single (European) player looks pretty stable... but now I've tested with multiple players: it is still somewhat unstable. Well, that's better than nothing :-D. I have learned a bit of java (which is not bad), but I fear that further research into the code of the program exceeds my ability to work (on weekend).
    Enjoy and best wishes

     
  • Mike Pope
    Mike Pope
    2010-11-16

    This patch may or may not have been accepted, but following the rework of the combat code is certainly out of date.

     
  • Mike Pope
    Mike Pope
    2010-11-16

    • status: open --> closed-out-of-date