Follow-up Comment #3, patch #1258 (project freeciv):

Why is this divided into 4 patches? It seems to me that patches 2 & 3 fix
serious bugs in patch 1, so patch 1 cannot be committed without them. Patch 4
is even more curious. Why does patch 1 set foggedborders setting SSET_RULES if
you already know that correct value is SSET_RULES_FLEXIBLE?

I do prefer small patches that do just one logical step. However, this is not
meant to introduce bugs. One logical step contains everything that is required
to make that step itself bugfree.

    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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