Follow-up Comment #7, bug #23444 (project freeciv):

> package_unit(punit, ...) should be package_unit(ptrans, ...)

Indeed.

> Probably this check should just be removed (and the
> client_has_player() one too?) Not tested this.

I think this check is correct for the S2_5 branch because the transport are
supposed to be known by the client before their cargo (it is linked with
former recursive unit info sending).

But, the behaviour is quite different in S2_6 branch and TRUNK (see patch
#5382 which removed notably these tests).

> (The comment about send_unit_info_to_onlookers() is now wrong.)

Indeed.


    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?23444>

_______________________________________________
  Message posté via/par Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to