Follow-up Comment #18, bug #21115 (project freeciv):
And I've completed the review of my rebase, with the following results:
(1) was mitigated with patch #4451, but has other issues. Bug #21991 raised
for that, but not a dependency of this bug, as the issue is more general than
just present==FALSE.
(2) is ideally a no-op given empty nreqs. Included in the wider patch #4679
(which is mostly irrelevant to present==FALSE, except for one comment change,
and the underlying motivation)
During preparation of patch #4679, found another case of dependency on
"nreqs", rather than wider requirement processing, raised as bug #21992 (which
this depends upon).
(3) patch #4451 addressed this in a better way (although not textually near my
earlier changes)
(4) root issue raised as bug #21982 (dependency of this bug)
I did not find anything else in my prior notes that appears to remain
unaddressed by the current codebase.
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?21115>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
