[Freeciv-Dev] [patch #3260] Show only airports in goto dialog
Update of patch #3260 (project freeciv): Planned Release: 2.5.0 = ___ Reply to this item at: http://gna.org/patch/?3260 ___ 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] [patch #3260] Show only airports in goto dialog
Follow-up Comment #3, patch #3260 (project freeciv): There was reply on freeciv-dev: http://mail.gna.org/public/freeciv-dev/2012-06/msg00474.html Please add comments to tickets in the future instead of sending to mailing list directly. For a long time I wondered what are you talking about ability to airlift just now as opposed to being able to airlift in principle. I didn't say anything about being able to airlift now. Now I guess (fell free to correct me) that you misunderstood city has positive EFT_AIRLIFT value. Value of the airlift effect is the number of units city can airlift / turn, not the capacity remaining this turn. ___ Reply to this item at: http://gna.org/patch/?3260 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
Re: [Freeciv-Dev] [patch #3260] Show only airports in goto dialog
On Fri, Jun 29, 2012 at 1:34 AM, Marko Lindqvist no-reply.invalid-addr...@gna.org wrote: Do you plan to produce updated/fixed patch? I like the idea of this patch. I wasn't particularly, but I am currently on/near HEAD so... maybe. At first I thought your comments made a lot of sense to me, but then after a while they became slightly less obviously sensible :} Let me explain... I'm not sure that what I (!) wanted was a means to shorten the list to only those cities that can airlift a given unit *right now*; what I wanted rather was to limit the list to only those cities that could, _in principle_, airlift a unit. The checkbox next to each city already shows if a city can airlift right now. Why would I want to see cities that can't airlift during the current turn? Because then at least I can send the Freight to that city while I can still remember the purpose for having built it. (It gets hard to remember these things when you have 600+ cities and a commensurate number of units hanging around.) (By 2018 it would be nice if I could just tell the Freight Get yerself to Timbuktu and the client would figure out what to do to make that happen; then I wouldn't have to remember at all.) Granted, the criticism that the patch should look for an effect and not a particular building, remains valid. ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3260] Show only airports in goto dialog
Follow-up Comment #1, patch #3260 (project freeciv): Why you want to insist that city must have *building* like Airport? Checking if city has positive EFT_AIRLIFT value instead of checking if it has building providing such effect would make code simpler and it would also list cities where effect is provided by something else than building, and to not to list cities with building that allows airlifting but missing some other prerequisite. ___ Reply to this item at: http://gna.org/patch/?3260 ___ 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] [patch #3260] Show only airports in goto dialog
URL: http://gna.org/patch/?3260 Summary: Show only airports in goto dialog Project: Freeciv Submitted by: berndj Submitted on: Tue Apr 17 18:04:06 2012 Category: client-gtk-2.0 Priority: 5 - Normal Status: None Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: ___ Details: This patch adds a toggle to show only airports, which is useful if you have 600 cities and only one or two airports per continent. ___ File Attachments: --- Date: Tue Apr 17 18:04:06 2012 Name: 0002-Filter-cities-by-whether-they-have-an-airport.patch Size: 4kB By: berndj Applies on top of S2_2@20614 http://gna.org/patch/download.php?file_id=15591 ___ Reply to this item at: http://gna.org/patch/?3260 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
