On Thu, Nov 20, 2008 at 10:23 AM, John Tamplin <[EMAIL PROTECTED]> wrote:
> On Thu, Nov 20, 2008 at 10:04 AM, Emily Crutcher <[EMAIL PROTECTED]> wrote: > >> For efficiency and code clearity I would still be inclined to support >> the singleton version as well, but adding the Date... version as an option >> to the plural versions seems like a terrific idea. I don't think it matters >> if we use calendar/getCalendar, but we should probably pick only one for >> consistancy! >> > > You can still call the varargs version with a single value, as in > addStyleToVisibleDates(styleName, date), and the extra cost should be > minimal. I think the extra weight of additional API surface area is more > costly. > Well, we can always add the single-arg one back in, so if the compiler actually generates reasonably efficent code for this, I think you're probably right. > > -- > John A. Tamplin > Software Engineer (GWT), Google > > > > > -- "There are only 10 types of people in the world: Those who understand binary, and those who don't" --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---