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:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to