Hi Brian -
Thanks for your review!
I think your point about adding !expOverflow to that conditional makes
perfect sense. We're only looking to account for expVal > expLimit where
decExp would be adjusted downward. Please adjust as appropriate.
Cheers,
SR
Sandipan
"00.01e326";
+ double expected = Double.parseDouble("1e308");
+double value = Double.parseDouble(text);
+if (Double.compare(expected, value) != 0) {
+throw new RuntimeException("String [" + text +
+ "] was in
Hi Brian/all,
Fix for the above-mentioned bug is complete. If you (or anyone else as
appropriate) could please review the changes and let me know of your
comments that would be great!
Link for the webrev:
http://www.sandipan.net/public/webrevs/8043740/webrev.00/
The patch is inlined below my si
Thanks Brian -
My concern is that, despite the unassigned status, someone just like
me could be working on it. Since I don't have a Jira login I cannot
update the status to communicate this to others.
Cheers,
SR
Sandipan Razzaque | www.sandipan.net
On Wed, May 28, 2014 at 10:36 AM,
Hi all,
Ready to contribute another patch - I intend to try and fix the following:
https://bugs.openjdk.java.net/browse/JDK-8043740
Any thoughts/objections and/or is anyone already working on it?
Cheers,
SR
Sandipan Razzaque | www.sandipan.net
Hi all,
Please find below the webrev for bug 8037343.
http://www.sandipan.net/public/webrevs/8037343/webrev.00
Thanks in advance -
Cheers,
SR
Sandipan Razzaque | www.sandipan.net
chance this weekend.
Cheers,
SR
Sandipan Razzaque | www.sandipan.net
From: Remi Forax
To: core-libs-dev@openjdk.java.net
Subject: Re: JDK-8037382 Sequence and SequenceSet interface
Message-ID: <5370.8010...@univ-mlv.fr>
Content-Type: text/plain; charset=UTF-8; format=flowed
On 05/1
ot wish to duplicate
effort.)
* Are there any objections to the changes already laid out in the ticket?
If all is well, I shall get started with a patch for review.
Cheers,
SR
https://bugs.openjdk.java.net/browse/JDK-8037382
Sandipan Razzaque | www.sandipan.net
