Reviewer needed - more test cases added to regression java/lang/Double/

Pavel Tisnovsky ptisnovs at
Thu Mar 10 09:54:34 UTC 2011

Alan Bateman wrote:
> Pavel Tisnovsky wrote:
>> Hi,
>> I'd like to change regression test java/lang/Double/ to
>> check some issues described in bug #4511638 in addition to bug #4428022.
>> Webrev containing new version of the regression test (and diffs) is
>> available here:
>> New version of the test firstly check all numbers+their string form and
>> then throws an exception in case at least one number is converted
>> incorrectly (this means that all conversions are printed to stdout).
>> Can anybody please review this change?
>> Thank you in advance
>> Pavel Tisnovsky
> Pavel - I think it would be best to include the test with the changes
> for 4511638 as we can't add a test that always fails (or maybe you are
> looking to add it with the @ignore keyword so that it's not run
> automatically?)

Sorry I didn't know it is not possible to add a regression test which
always fails. Because 4511638 is not fixed yet (AFAIK), I'll prefer to
wait for this fix.


> -Alan

More information about the core-libs-dev mailing list