[threeten-develop] Field Sizes
Status: Alpha
Brought to you by:
scolebourne
From: Richard W. <ric...@gm...> - 2012-09-10 18:17:22
|
Hello all, I was just reviewing some of the source code and noted that a couple of the classes have fields with unnecessarily large field sizes for their primitive types. Month-of-year really doesn't need a full int, just a byte. At present its a short in LocalDate and an int in MonthDay and YearMonth. Day-of-month also only needs a byte and is a short in LocalDate and an int in Monthday. All of this is with respect to the ISO Calendaring System. Is there any particular reason for this choice, and would people be open to downsizing these fields to their appropriate sizes? regards, Richard |