<Swing Dev> [9] Review Request for 8129838: NPE in java.awt.Choice

Semyon Sadetsky semyon.sadetsky at oracle.com
Thu Jul 30 06:22:49 UTC 2015


Hi Alexander,

- I have introduced a private method for those 2 duplicated lines you've 
found.
- One iteration will require to rework the initial logic.

http://cr.openjdk.java.net/~ssadetsky/8129838/webrev.01/

--Semyon


On 7/29/2015 2:39 PM, Alexander Scherbatiy wrote:
>
>   - Could you move the duplicated code to one method?
>   - Is it possible to find bounds using only one iteration over 
> GraphicsDevice/GraphicsConfiguration loop.
>
>  Thanks,
>  Alexandr.
>
> On 7/29/2015 12:51 PM, Semyon Sadetsky wrote:
>> Hello,
>>
>> Please review fix for JDK9:
>> bug: https://bugs.openjdk.java.net/browse/JDK-8129838
>> webrev: http://cr.openjdk.java.net/~ssadetsky/8129838/webrev.00/
>>
>> This regression from 7072653. Swing components used as AWT components 
>> peers in OSX do not receive graphics configuration so NPE is thrown.  
>> The solution is to obtain it directly from the graphics environment.
>>
>> --Semyon
>>
>




More information about the swing-dev mailing list