<AWT Dev> [8] Please review fix for 7175704: [macosx] "8" PIT: NPE in GetDisplayMode fullscreen test

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Tue Oct 9 03:31:54 PDT 2012


09.10.2012 14:25, Alexander Zuev wrote:
> Sergey,
>
>   the text message is not specified in the documentation and the new 
> one is already used on MacOS X
> in the jdk7 so i would let it be as it is for consistency.
I don't see a reason not to unify it across tolkits. Why we should use 
different text on different platforms?
> The check in obtaining matching display mode is already performed in 
> the native code:
> In CGraphicsDevice.m :
>
>     CGDisplayModeRef closestMatch = getBestModeForParameters(allModes, 
> (int)w, (int)h, (int)bpp, (int)refrate);
>     if (closestMatch != NULL) {
>      ...
>     } else {
>         [JNFException raise:env as:kIllegalArgumentException 
> reason:"Invalid display mode"];
>     }
>
> With best regards,
> Alexander Zuev
>
> On 10/9/12 14:12, Sergey Bylokhov wrote:
>> Hi, Alexander
>> On win32 anx X11 we have another exception text message "Invalid 
>> display mode", also there is additional check in 
>> getMatchingDisplayMode() is it unnecessary on the macosx?
>>
>> 09.10.2012 12:35, Alexander Zuev wrote:
>>> Hello,
>>>
>>>   please review my fix for CR 7175704: [macosx] "8" PIT: NPE in 
>>> GetDisplayMode fullscreen test
>>>
>>>   The NPE happens because we do not check the passed argument to 
>>> null before using it. The correct exception according to the 
>>> specification should be IllegalArgumentException.
>>>
>>> Bug description is: 
>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7175704
>>> Webrev can be found here: 
>>> http://cr.openjdk.java.net/~kizune/7175704/webrev.00
>>>
>>> With best regards,
>>> Alex
>>
>>
>


-- 
Best regards, Sergey.




More information about the awt-dev mailing list