[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up
Follow-up Comment #5, bug #13841 (project freeciv): Next I would create ticket about using genlist to store unlimited number of global worklists. There still is no such patch, right? (Just making sure I'm not just too blind to find the ticket) ___ Reply to this item at: http://gna.org/bugs/?13841 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up
Follow-up Comment #4, bug #13841 (project freeciv): *_worklist() - worklist_*() rename and universal_kind_name() change already have tickets of their own. Next I would create ticket about using genlist to store unlimited number of global worklists. ___ Reply to this item at: http://gna.org/bugs/?13841 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up
Follow-up Comment #1, bug #13841 (project freeciv): Please try to split your patches to smaller separate tickets which each do just one thing. That way it's much easier to confirm them. ___ Reply to this item at: http://gna.org/bugs/?13841 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up
Follow-up Comment #2, bug #13841 (project freeciv): I'm not sure I could make separated patches for those changes because all of them are linked. ___ Reply to this item at: http://gna.org/bugs/?13841 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up
Follow-up Comment #3, bug #13841 (project freeciv): I try to make more detailed splitting proposal when I have time to look this patch in more detail (or maybe I'll end admitting that it's not splittable :-) Anyway, obvious first patch would be init_worklist() - worklist_init(), copy_worklist() - worklist_copy() rename. ___ Reply to this item at: http://gna.org/bugs/?13841 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up
Update of bug #13841 (project freeciv): Assigned to:None = cazfi ___ Reply to this item at: http://gna.org/bugs/?13841 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev