Great, thanks Anthony! -DrD-
> Thanks Artem. > > David: I can push the fix as soon as you get a release team approval for this > fix to go to 7u-dev. > > -- > best regards, > Anthony > > On 06/06/13 17:10, Artem Ananiev wrote: >> >> The changes looks fine to me. For JDK8, the fix can be simplified by >> removing X11-specific part of jawt_md.h on Mac, since we're going to >> remove XAWT support on that platform in JDK8 anyway. >> >> Thanks, >> >> Artem >> >> On 6/6/2013 4:40 PM, Anthony Petrov wrote: >>> I've already reviewed this fix, and it looks fine to me. We still need >>> another reviewer. Adding awt-dev@. >>> >>> Anyone please? >>> >>> -- >>> best regards, >>> Anthony >>> >>> On 06/05/13 20:52, David DeHaven wrote: >>>> >>>> Proposed fix for getting the correct jawt_md.h header in the JDK. I >>>> reused the XAWT flag to enable the X11 drawing surface struct. It is >>>> disabled by default as just about every application running on Mac >>>> will use the Cocoa interface. Apparently the X11 interface is still >>>> operable on Mac and it's used for verification, so I don't want to >>>> completely remove it just yet. I also changed several #include >>>> statements to use the JDK headers instead of the deprecated JavaVM >>>> framework headers. >>>> >>>> Since I'm not a jdk7u-dev committer, Anthony Petrov has graciously >>>> agreed to sponsor this patch. I will be kicking off a JPRT run when I >>>> can steal another couple minutes, and figure out the magic >>>> incantations needed to include the AWT tests. >>>> >>>> >>>> Webrev: >>>> http://cr.openjdk.java.net/~ddehaven/7181710/jdk.0/ >>>> >>>> Issue: >>>> http://bugs.sun.com/view_bug.do?bug_id=7181710 >>>> >>>> -DrD- >>>>
