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