[7u6] Please review my fix for 7124247: [macosx] Implement	GraphicsDevice.setDisplayMode()
    Sergey Bylokhov 
    Sergey.Bylokhov at oracle.com
       
    Wed Jun 13 04:01:06 PDT 2012
    
    
  
Hi, Alexander.
Fix looks good.
09.06.2012 14:39, Alexander Zuev wrote:
> Hello,
>
>   slightly updated fix for CR 7124247: [macosx] Implement 
> GraphicsDevice.setDisplayMode()
>
>   Actually it is a backport to the 7u6 of the fix for jdk8 with minor 
> corrections - fixed situation with the getDisplayModes() when we 
> returned the wrong resolutions for the requested modes, fixed crash 
> when we repeatedly asked to set display mode in a quick sentence and 
> now setDisplayMode correctly throws an IllegalArgumentException 
> instead of NullPointerException if null is passed as a parameter.
>
>   CR description can be found here: 
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7124247
>
>   Webrev with the proposed change can be seen here: 
> http://cr.openjdk.java.net/~kizune/7124247/jdk7u6/webrev.02
>
> With best regards,
> Alex
-- 
Best regards, Sergey.
    
    
More information about the macosx-port-dev
mailing list