Follow-up Comment #6, patch #1296 (project freeciv):

>  if (ptile) {
>    /* Is it useful? */
>    packet->x = ptile->x;
>    packet->y = ptile->y;
>   }

I think the reset of the coordinates (x,y) within the packet is needed, as
the same packet struct could also be used for offline events.

> Why is this not simple strncpy()? 

I do most of my coding as learning-by-doing. Sorry if I did not choose the
right function or if there are bad code sections. Here I did select the first
function which did that I want. You did delete the block completely and I
think its a better solution.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?1296>

_______________________________________________
  Nachricht geschickt von/durch Gna!
  http://gna.org/


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

Reply via email to