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:


  Nachricht geschickt von/durch Gna!

Freeciv-dev mailing list

Reply via email to