Warning Fixes from LJC Hack Session

Stuart Marks stuart.marks at oracle.com
Mon Feb 6 14:53:50 PST 2012


Michael,

Thanks for splitting up the patches and revising them in response to the review 
comments.

I'm not yet entirely sure how to proceed with pushing these changes. I think 
Chris assumed that I would push these changes. Since these are in awt, 
printing, and beans, the changes might need to be pushed through a different 
forest (2D?) than the forest in which I usually do work (TL). In addition it 
might benefit from some additional review by members of the appropriate teams.

Phil, you saw this go by earlier -- do you have a recommendation for how you'd 
like to proceed? I'm certainly happy to help out, but I'd end up pushing them 
into TL.

Oh, and Alan, Chris, Anthony, thanks for your reviews and comments.

s'marks


On 2/4/12 8:40 AM, Michael Barker wrote:
> Patch with BeanSupportContext changes removed.
>
> Mike.
>
> On Sat, Feb 4, 2012 at 3:48 PM, Chris Hegarty<chris.hegarty at oracle.com>  wrote:
>> I think you should just drop the changes to this file.
>>
>> -Chris.
>>
>> Michael Barker<mikeb01 at gmail.com>  wrote:
>>
>>>> Oh, good catch Alan, I missed this. This part of the change should be
>>>> reversed ( at least in the context of warnings cleanup ).
>>>
>>> Should I switch it to a wild card (protected transient ArrayList<?>
>>> bcmListeners) or just drop the modifications for that file?
>>>
>>> Mike.


More information about the jdk8-dev mailing list