Changeset: 85fd2ae0a845
Author:mchung
Date: 2013-11-05 17:33 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/85fd2ae0a845
8022208: Intermittent test failures in java/lang/Thread/ThreadStateTest.java
6944188: ThreadMXBean/ThreadStateTest.java fails intermittently
Reviewed-by: d
On 05/11/13 20:49, Chris Hegarty wrote:
On 5 Nov 2013, at 20:39, Michael McMahon wrote:
On 05/11/13 19:38, Chris Hegarty wrote:
Another installment of checks for return values from
GetFieldID, and friends, to follow up on last weeks work [1].
http://cr.openjdk.java.net/~chegar/netNullChecks/w
> On 5 Nov 2013, at 20:47, roger riggs wrote:
>
> H Michael,
>
> If I remember rightly, CHECK_NULL tests for an exception and conditionally
> does an early return from the function. So testing just before an
> unconditional return is unnecessary.
>
> (These look ok to me; not a Reviewer)
Th
> On 5 Nov 2013, at 20:39, Michael McMahon wrote:
>
>> On 05/11/13 19:38, Chris Hegarty wrote:
>> Another installment of checks for return values from
>> GetFieldID, and friends, to follow up on last weeks work [1].
>>
>> http://cr.openjdk.java.net/~chegar/netNullChecks/webrev/
>>
>> There are
H Michael,
If I remember rightly, CHECK_NULL tests for an exception and conditionally
does an early return from the function. So testing just before an
unconditional return is unnecessary.
(These look ok to me; not a Reviewer)
Roger
On 11/5/2013 3:39 PM, Michael McMahon wrote:
On 05/11/13
On 05/11/13 19:38, Chris Hegarty wrote:
Another installment of checks for return values from
GetFieldID, and friends, to follow up on last weeks work [1].
http://cr.openjdk.java.net/~chegar/netNullChecks/webrev/
There are more cleanups to come along the same lines, but I'd like to
keep these c
Another installment of checks for return values from
GetFieldID, and friends, to follow up on last weeks work [1].
http://cr.openjdk.java.net/~chegar/netNullChecks/webrev/
There are more cleanups to come along the same lines, but I'd like to
keep these changes small and confined for now.
-Chr
Changeset: d5b3f83ffc40
Author:psandoz
Date: 2013-11-05 12:08 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d5b3f83ffc40
8027712: DistinctOpTest fails for unordered test
Reviewed-by: henryjen, alanb
!
test/java/util/stream/test/org/openjdk/tests/java/util/stream/DistinctOp
Ineexof(char) sounds like as fast and simpler?
Sent from my iPhone
On Nov 5, 2013, at 8:55 AM, Peter Levart wrote:
> On 11/04/2013 07:12 PM, robert.fi...@oracle.com wrote:
>> Changeset: 51b002381b35
>> Author:rfield
>> Date: 2013-11-04 10:12 -0800
>> URL: http://hg.openjdk.java.n
