Code review request for 6253144: Long narrowing conversion should describe the algorithm used and implied "risks"

joe.darcy at oracle.com joe.darcy at oracle.com
Thu Jun 23 22:08:12 UTC 2011



On 6/22/2011 8:52 AM, Chris Hegarty wrote:
>
>
> On 06/22/11 04:47 PM, Joe Darcy wrote:
>> David Holmes wrote:
>>> Hi Joe,
>>>
>>> Joe Darcy said the following on 06/18/11 11:38:
>>>> Please review this (somewhat tedious) change to make the behavior of
>>>> the Number subtypes in the JDK more explicit:
>>>>
>>>> 6253144: Long narrowing conversion should describe the algorithm used
>>>> and implied "risks"
>>>> http://cr.openjdk.java.net/~darcy/6253144.0/
>>>>
>>>> David, how are changes to AtomicInteger and AtomicLong managed?
>>>
>>> Normally they would go into Doug Lea's CVS for jsr166, we (Chris
>>> Hegarty) would pull them over and then push to OpenJDK. It can work
>>> the other way but the sync's can get messier.
>>>
>>
>> Chris,
>>
>> Off-list, Mike approved this set of changes and I'd like to get them
>> pushed once the matching ccc is approved. (The ccc request is needed
>> since the long-standing behavior of the non-abstract methods on number
>> is being specified.)
>>
>> How would you like to handle updates to the Atomic classes?
>
> I think in this case it should be fine to make the changes in OpenJDK 
> first. Then I can create a patch for the Atomic changes based on Dougs 
> CVS.
>
> I'll double check this with Doug, but unless you hear otherwise let's 
> assume we can do this. In fact, if Doug is watching he may be able to 
> move faster than us!
>
> -Chris.
>

Hello.

FYI, fix pushed to JDK 8 tl:
    http://hg.openjdk.java.net/jdk8/tl/jdk/rev/151756a4037b

Thanks,

-Joe



More information about the core-libs-dev mailing list