LGTM
looks like this crept in in the massive r8946. In addition to being
moved, the parse method was changed from
public Double parse(CharSequence object) throws ParseException {
- try {
- return Double.valueOf(object.toString());
- } catch (NumberFormatException e) {
- throw new ParseException(e.getMessage(), 0);
- }
- }
to
<<<<<
public Double parse(CharSequence object) throws ParseException {
+ if ("".equals(object.toString())) {
+ return null;
+ }
+
try {
- return Double.valueOf(object.toString());
+ return
Math.rint(NumberFormat.getDecimalFormat().parse(object.toString()));
} catch (NumberFormatException e) {
throw new ParseException(e.getMessage(), 0);
}
On 2010/12/02 22:45:38, rchandia wrote:
Gah! I meant:
http://gwt-code-reviews.appspot.com/1170801
El 2 de diciembre de 2010 17:40, Chris Conroy
<mailto:[email protected]> escribió:
> that's a link to this review :P
>
>
> On Thu, Dec 2, 2010 at 5:39 PM, <mailto:[email protected]> wrote:
>
>> This is actually a repost of tboryer's patch originally submitted
at:
>>
>>
>> http://gwt-code-reviews.appspot.com/1181802
>>
>> On 2010/12/02 22:33:23, rchandia wrote:
>>
>>
>>
>>
>> http://gwt-code-reviews.appspot.com/1181802/show
>>
>
>
http://gwt-code-reviews.appspot.com/1181802/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
