Hi, On Thu, Nov 26, 2009 at 3:06 PM, Thomas Friedrichsmeier <[email protected]> wrote: > Hi, > > On Thursday 26 November 2009, Prasenjit Kapat wrote: >> (2) If the urls are stored, then >> we'll have to do an indexOf(...) search for the hostname combobox, >> everytime the settings dialog is loaded. > > not to worry. The main rule of thumb is: If each call of the code is a direct > result of a user action, then wasting a couple thousand CPU cycles is nothing. > Performance is often an issue for something that may run in a loop of some > sort, but rarely for anything else.
OK, using indexOf() the earlier bug is gone. But things are still hard coded! >> (2) I like the idea to give the user more readable choices, like the >> ones in the combobox (Country, State, City, Hostname), rather than the >> "choices" offered using the Tcl/Tk menu box. >> (3) Also, being able to store the choice is a plus point, I think. > > Both are good points. But perhaps both are solvable without hardcoding > anything: > > 1) In the settings, perhaps we could offer a "pick from list" button, which > quite simply calls chooseCRANmirror() in R. Hence we'd have a dropdown choice, > and still be able to save the result. > 2) We could replace chooseCRANmirror() to offer a nicer menu, with all the > additional info from getCRANmirrors(). > > Would that cover all wishes? Of course, that would be ideal! -- Prasenjit ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ RKWard-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/rkward-devel
