Follow-up Comment #1, patch #3726 (project freeciv):

Reading the patch alone (not checking existing context from sources) it seems
that even after noticed failure you still continue to try to do everything
inside that particular function as if everything succeeded. At least executing
secfile_save() after secfile creation has already failed seems like wrong
thing to do.

    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


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

Reply via email to